Jump to: navigation, search

Difference between revisions of "Nova/VersionedNotificationTransformation"

(Tasks)
(Tasks)
Line 20: Line 20:
 
Please use the https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-ocata topic for the transformation commits
 
Please use the https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-ocata topic for the transformation commits
  
 +
There are three types of transformations:
 +
* The easy ones that uses the same payload as instance.delete
 +
* The complex instance.<action> notifications that needs payload extension.
 +
* The rest that needs specific payloads.
  
The easy ones that uses the same payload as instance.delete:
+
The burndown chart is updated hourly from gerrit https://vntburndown-gibi.rhcloud.com/index.html . You can see what to transform and what to review there.
  
 
{| class="wikitable"
 
{| class="wikitable"

Revision as of 12:58, 5 October 2016

Goal

Transform every legacy notification to the new versioned notification framework.

How to do it

Tasks

The master is now open for Ocata so work can continue.

Please use the https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-ocata topic for the transformation commits

There are three types of transformations:

  • The easy ones that uses the same payload as instance.delete
  • The complex instance.<action> notifications that needs payload extension.
  • The rest that needs specific payloads.

The burndown chart is updated hourly from gerrit https://vntburndown-gibi.rhcloud.com/index.html . You can see what to transform and what to review there.

event type status taken by review comments
instance.shutdown in progress siva_krishnan https://review.openstack.org/#/c/329214
instance.create_ip move to Ocata antal https://review.openstack.org/#/c/332876/ renamed to add_fixed_ip to be more precise
instance.snapshot move to Ocata syjulian https://review.openstack.org/#/c/330128/
instance.reboot move to Ocata macsz https://review.openstack.org/#/c/330224/
instance.power_off move to Ocata xenogear https://review.openstack.org/#/c/330761/
instance.scheduled
instance.delete_ip open rename it to remove_fixed_ip
instance.evacuate open
instance.migration open
instance.unpause in progress MikeG451 https://review.openstack.org/380668
instance.reboot.error open
instance.shelve_offload in progress vancsics https://review.openstack.org/#/c/377239/
instance.unshelve open jinli
instance.soft_delete in progress stewie925
instance.resume in progress stewie925 https://review.openstack.org/#/c/375784/
instance.trigger_crash_dump open
instance.unrescue open
instance.create open Some of the instance.create.<phase> notifications has extra payload
instance.rebuild in progress xavvior
instance.rebuild.scheduled open we might need to change the event_type
instance.resize.error open
instance.resize.prep open we might need to change the event_type
instance.finish_resize open we might want to change the even_type
instance.resize.confirm in progress MikeG451
instance.resize.revert open
instance.live.migration.abort open we might need to change the even_type to make it consistent
instance.live_migration.pre open we might want to change the event_type
instance.live_migration.rollback.dest open we might want to change the event_type
instance.live_migration._rollback open we might want to change the event_type
instance.live_migration._post open we might want to change the even_type
instance.live_migration.post.dest open we might want to change the event_type
instance.delete merged gibi https://review.openstack.org/#/c/313654 This shall be the base of for the other instance.<action> transformation
instance.pause merged sneti https://review.openstack.org/#/c/329141/
instance.power_on merged zhs https://review.openstack.org/#/c/329746/
instance.shelve merged vancsics https://review.openstack.org/#/c/332696/
instance.suspend merged pumaranikar https://review.openstack.org/#/c/329089
instance.restore merged zhs https://review.openstack.org/#/c/331972/
instance.resize merged aunnam https://review.openstack.org/#/c/329255/

The complex instance.<action> notifications that needs payload extension.

event type status taken by review comments
instance.update merged gibi https://review.openstack.org/#/c/325923/ This shall be the example for creating instance.<action> notification with extended InstancePayload
instance.exists open
instance.rescue open
instance.volume.attach open

The rest:

event type status taken by review comments
compute.exception merged laszloh https://review.openstack.org/#/c/310700
aggregate.addhost open
aggregate.create open
aggregate.delete open
aggregate.removehost open
aggregate.updatemetadata open
aggregate.updateprop open
api.fault open
libvirt.error open
compute_task.build_instances open
compute_task.migrate_server open
compute_task.rebuild_server open
HostAPI.power_action open
HostAPI.set_enabled open
HostAPI.set_maintenance open
keypair.create spec exists h-eguchi spec: https://review.openstack.org/#/c/300942
keypair.delete spec exists h-eguchi spec: https://review.openstack.org/#/c/300942
keypair.import spec exists h-eguchi spec: https://review.openstack.org/#/c/300942
network.floating_ip.allocate open
network.floating_ip.associate open
network.floating_ip.deallocate open
network.floating_ip.disassociate open
scheduler.select_destinations open
servergroup.addmember open
servergroup.create open
servergroup.delete open
volume.usage open
metrics.update open sudipto


Follow up tasks