Fuel for OpenStack

rpmvercmp.rb - incorrect result of comparing due to package issues in 5.0

Bug #1362071 reported by Ihor Kalnytskyi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Dmitry Ilyin
Fuel for OpenStack 5.0.2

Bug Description

The puppet runs `yum` with downgrade option since we have incorrect versioning for ceilometer-* packages in 5.0.
Here the list of packages:

    openstack-ceilometer-alarm
    openstack-ceilometer-api
    openstack-ceilometer-central
    openstack-ceilometer-collector
    openstack-ceilometer-common
    openstack-ceilometer-compute
    python-ceilometer

The package version in Fuel 5.0 is "2014.1.fuel5.1-mira0".
The package version in Fuel 5.0.2 is "2014.1.1-fuel5.0.2.mira4"

Additional puppet logs: http://xsnippet.org/360145/raw/

See original description
Revision history for this message
Dmitry Ilyin (idv1985) wrote :

It's not actually incorrect because there is logic to make fuel5.1 version newver then fuel5.0.2 regardless actual package version
sh compare.sh 2014.1.fuel5.1-mira0 2014.1.1-fuel5.0.2.mira4
* Native Puppet
2014.1.fuel5.1-mira0 > 2014.1.1-fuel5.0.2.mira4
* Puppet RPMVERCMP
2014.1.fuel5.1-mira0 > 2014.1.1-fuel5.0.2.mira4
* Native RPM
0:2014.1.fuel5.1-mira0 < 0:2014.1.1-fuel5.0.2.mira4
* Native DEB
2014.1.fuel5.1-mira0 > 2014.1.1-fuel5.0.2.mira4

Well... I didn't forsee that someone would name 5.0 package with fuel5.1

Revision history for this message
Dmitry Ilyin (idv1985) wrote :

I can remove fuel version comparsion and it will help

sh compare.sh 2014.1.fuel5.1-mira0 2014.1.1-fuel5.0.2.mira4
* Native Puppet
2014.1.fuel5.1-mira0 > 2014.1.1-fuel5.0.2.mira4
* Puppet RPMVERCMP
2014.1.fuel5.1-mira0 < 2014.1.1-fuel5.0.2.mira4
* Native RPM
0:2014.1.fuel5.1-mira0 < 0:2014.1.1-fuel5.0.2.mira4
* Native DEB
2014.1.fuel5.1-mira0 > 2014.1.1-fuel5.0.2.mira4

Most likely it's safe because it was added just to ensure the correct results.

Ihor Kalnytskyi (ikalnytskyi)
description: updated
Revision history for this message
Dmitry Ilyin (idv1985) wrote :

No, we can't remove fuel version check. I'm afraid we'll have to rename out 5.0.2 packages to 2014.1.1-fuel5.1rel5.0.2.mira4

sh compare.sh 2014.1.1-fuel5.1rel5.0.2.mira4 2014.1.fuel5.1-mira0
* Native Puppet
2014.1.1-fuel5.1rel5.0.2.mira4 < 2014.1.fuel5.1-mira0
* Puppet RPMVERCMP
2014.1.1-fuel5.1rel5.0.2.mira4 > 2014.1.fuel5.1-mira0
* Native RPM
0:2014.1.1-fuel5.1rel5.0.2.mira4 > 0:2014.1.fuel5.1-mira0
* Native DEB
2014.1.1-fuel5.1rel5.0.2.mira4 < 2014.1.fuel5.1-mira0

Revision history for this message
Dmitry Ilyin (idv1985) wrote :

We have created packages named like 2014.1.1-fuel5.1rel5.0.2.mira4 they will be added to 5.0.2 mirrors shortly.

Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/5.1.x
Revision history for this message
Ihor Kalnytskyi (ikalnytskyi) wrote :

Verified on tarball #484.

Changed in fuel:
milestone: 5.1 → 5.0.2
no longer affects: fuel/5.0.x
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.