[ci] Repository contains Horizon package with too high version

Bug #1529220 reported by Ivan Udovichenko
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Invalid
High
Igor Yozhikov

Bug Description

http://perestroika-repo-tst.infra.mirantis.net/mos-repos/ubuntu/8.0/pool/main/h/horizon/

'''
horizon-doc_8.0.0-5~u14.04+mos45_all.deb 23-Dec-2015 21:58 1195116
horizon_8.0.0-5~u14.04+mos45.debian.tar.gz 23-Dec-2015 21:58 13229
horizon_8.0.0-5~u14.04+mos45.dsc 23-Dec-2015 21:58 3101
horizon_8.0.0.orig.tar.gz 23-Dec-2015 21:58 3880611
openstack-dashboard_8.0.0-5~u14.04+mos45_all.deb 23-Dec-2015 21:58 2654214
python-django-horizon_8.0.0-5~u14.04+mos45_all.deb 23-Dec-2015 21:58 299900
'''

Meanwhile repository contains version in d/changelog :
horizon (2:8.0.0~b3-5~u14.04+mos2) mos8.0; urgency=medium

What is the reason?

Tags: area-mos
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

The reason is the package version was set from upstream git tag. Please keep change log version in sync with upstram one

Changed in fuel:
assignee: Fuel build team (fuel-build) → MOS Packaging Team (mos-packaging)
Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :

Please take a closer look at version 2:8.0.0-_5_~u14.04+mos45
This bug is about revision and not about version from tag.
Could you please prevent such behavior in future ?

Changed in fuel:
assignee: MOS Packaging Team (mos-packaging) → Fuel build team (fuel-build)
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

The revision of the package was taken from spec file (except mosNNN part)
> horizon (2:8.0.0~b3-5~u14.04+mos2) mos8.0; urgency=medium

Changed in fuel:
assignee: Fuel build team (fuel-build) → MOS Packaging Team (mos-packaging)
Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :

And I took another version [1]:
horizon (2:8.0.0-4~u14.04+mos1) mos8.0; urgency=medium

The version you took is higher now [2].
Do you see the difference ?

[1] https://review.fuel-infra.org/#/c/14825/5/trusty/debian/changelog
[2] https://packaging-ci.infra.mirantis.net/job/8.0-pkg-install-ubuntu/2001/

Changed in fuel:
assignee: MOS Packaging Team (mos-packaging) → Fuel build team (fuel-build)
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

You've downgraded revision part of version string with this CR from "-5~u14.04" to "-4~u14.04"

Changed in fuel:
assignee: Fuel build team (fuel-build) → Ivan Udovichenko (iudovichenko)
Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :

Please be more precise:
"~b3-5~u14.04" to "-4~u14.04"

Do you still think that this behavior is valid ?

Changed in fuel:
assignee: Ivan Udovichenko (iudovichenko) → Fuel build team (fuel-build)
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

No, revision part starts from "-", not from "~"
So you've got downgrade from 5 to 4

Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :

$ dpkg --compare-versions 2:8.0.0~b3-5~u14.04+mos2 lt 2:8.0.0-4~u14.04+mos1; echo $?
0

Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :
Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :

If a major change (new version of the software being packaged) occurs, the version number is changed to reflect the new software version, and debian package release number is reset to 1.

Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :

Previous comment contains quote from the document you've provided.

Roman Vyalov (r0mikiam)
Changed in fuel:
assignee: Fuel build team (fuel-build) → Igor Yozhikov (iyozhikov)
Revision history for this message
Roman Vyalov (r0mikiam) wrote :

Igor please confirm this bug . I dont understand, why do you want to downgrade the version for horizon package

tags: added: area-mos
Changed in fuel:
status: Confirmed → Incomplete
Changed in fuel:
status: Incomplete → Won't Fix
Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

Looks like the conclusion was this is actually Invalid...

Changed in fuel:
status: Won't Fix → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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