[Bug 1243550] Review Request: openstack-aodh - OpenStack Telemetry Alarming

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1243550

Haïkel Guémar <karlthered at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |pkilambi at redhat.com
             Blocks|                            |1243533
                   |                            |(RDO-LIBERTY-REVIEWS)
              Flags|                            |needinfo?(pkilambi at redhat.c
                   |                            |om)



--- Comment #3 from Haïkel Guémar <karlthered at gmail.com> ---
## Blockers


* I'll need your help to properly understand ceilometer/aodh relationship and
figure out the proper upgrade path.

I pretty much get that openstack-aodh-{notifier,evaluator} services replace
services with the same in ceilometer.
Problem is that openstack-ceilometer-{notifier,evaluator} were shipped in a
single openstack-ceilometer-alarm package, previously.

=> You already obsoletes openstack-ceilometer-alarm but it's unversioned,
should be:
Provides: openstack-ceilometer-alarm = 2:%{version}-%{release}
Obsoletes: openstack-ceilometer-alarm < last n-v-r with epoch of
openstack-ceilometer-alarm

As ceilometer switched to semantic versioning, we need to carry epoch from
ceilometer in our provides/obsoletes. No need to add epoch to the package
though

=> Problem is that your main package and  doesn't require
openstack-aodh-{notifier,evaluator} and hence the upgrade path is still
incorrect.
So either we create a subpackage openstack-aodh-alarm-compat requiring all the
packages providing features from openstack-ceilometer-alarm, so we just 


Relevant guidelines
[1]
https://fedoraproject.org/wiki/Upgrade_paths_%E2%80%94_renaming_or_splitting_packages#.28n:m.29_Many_to_many_replacement

If you're aware of other similar features mapping in ceilometer/aodh, let me
know so we could figure out the upgrade path.

* use versioned python macros

* I exempt this package from newer python guidelines as we need to make sure
that the upgrade path from ceilometer alarm is ok before renaming stuff.
Moreover, openstack-xxx packages will be retired from F24 at a later point.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1243533
[Bug 1243533] (RDO) Tracker: Review requests for new RDO Liberty packages
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component