Difference between revisions of "Nova/VersionedNotificationTransformation"
< Nova
(→Tasks) |
(→Tasks) |
||
| Line 19: | Line 19: | ||
|- | |- | ||
| instance.delete || subteam review || gibi || https://review.openstack.org/#/c/313654 || This shall be the base of for the other instance.<action> transformation | | instance.delete || subteam review || gibi || https://review.openstack.org/#/c/313654 || This shall be the base of for the other instance.<action> transformation | ||
| − | |||
| − | |||
|- | |- | ||
| instance.shutdown || open || || || | | instance.shutdown || open || || || | ||
| Line 27: | Line 25: | ||
|- | |- | ||
| instance.create_ip || open || || || | | instance.create_ip || open || || || | ||
| + | |- | ||
| + | | instance.delete_ip || open || || || | ||
|- | |- | ||
| instance.evacuate || open || || || | | instance.evacuate || open || || || | ||
| Line 33: | Line 33: | ||
|- | |- | ||
| instance.pause || open || || || | | instance.pause || open || || || | ||
| + | |- | ||
| + | | instance.power_on || open || || || | ||
|- | |- | ||
| instance.power_off || open || || || | | instance.power_off || open || || || | ||
| Line 50: | Line 52: | ||
| instance.unrescue || open || || || | | instance.unrescue || open || || || | ||
|- | |- | ||
| − | | instance. | + | | instance.create || open || || || Some of the instance.create.<phase> notifications has extra payload |
| − | |||
| − | |||
| − | |||
| − | |||
|- | |- | ||
| instance.rebuild.scheduled || open || || || we might need to change the event_type | | instance.rebuild.scheduled || open || || || we might need to change the event_type | ||
| Line 77: | Line 75: | ||
|- | |- | ||
| instance.rebuild || open || || || | | instance.rebuild || 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.pre || open || || || we might want to change the event_type | ||
Revision as of 13:07, 2 June 2016
Goal
Transform every legacy notification to the new versioned notification framework.
How to do it
- Check the notification devref that describes the new framework: http://docs.openstack.org/developer/nova/notifications.html
- Check the existing new versioned notification example: https://review.openstack.org/#/c/245678/
- Check the example transformation patch: https://review.openstack.org/#/c/313654/
- Select a task for the below table and add your IRC handle to table. (Please take only one at a time)
- Propose a transformation patch based on the examples.
- If you got stuck you can ask questions from rlrossit and gibi on IRC or join the weekly notification subteam meeting https://wiki.openstack.org/wiki/Meetings/NovaNotification
Tasks
The easy ones that uses the same payload as instance.delete:
| event type | status | taken by | review | comments |
|---|---|---|---|---|
| instance.delete | subteam review | gibi | https://review.openstack.org/#/c/313654 | This shall be the base of for the other instance.<action> transformation |
| instance.shutdown | open | |||
| instance.scheduled | open | |||
| instance.create_ip | open | |||
| instance.delete_ip | open | |||
| instance.evacuate | open | |||
| instance.migration | open | |||
| instance.pause | open | |||
| instance.power_on | open | |||
| instance.power_off | open | |||
| instance.reboot | open | |||
| instance.shelve | open | |||
| instance.snapshot | open | |||
| instance.soft_delete | open | |||
| instance.suspend | open | |||
| instance.trigger_crash_dump | open | |||
| instance.unrescue | open | |||
| instance.create | open | Some of the instance.create.<phase> notifications has extra payload | ||
| instance.rebuild.scheduled | open | we might need to change the event_type | ||
| instance.resize.prep | open | we might need to change the event_type | ||
| instance.resize | open | |||
| instance.restore | open | |||
| instance.resume | open | |||
| instance.shelve_offload | open | |||
| instance.unpause | open | |||
| instance.finish_resize | open | we might want to change the even_type | ||
| instance.resize.confirm | open | |||
| instance.resize.revert | open | |||
| instance.rebuild | 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.unshelve | open | |||
| instance.live_migration.post.dest | open | we might want to change the event_type |
The complex instance.<action> notifications:
| event type | status | taken by | review | comments |
|---|---|---|---|---|
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
The rest:
| event type | status | taken by | review | comments |
|---|---|---|---|---|
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
| Example | Example | Example | Example | Example |
