Difference between revisions of "Nova/VersionedNotificationTransformation"

Revision as of 13:18, 2 June 2016

Transform every legacy notification to the new versioned notification framework.

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

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.unpause	open
instance.power_on	open
instance.power_off	open
instance.reboot	open
instance.shelve	open
instance.shelve_offload	open
instance.unshelve	open
instance.snapshot	open
instance.soft_delete	open
instance.suspend	open
instance.resume	open
instance.restore	open
instance.trigger_crash_dump	open
instance.unrescue	open
instance.create	open			Some of the instance.create.<phase> notifications has extra payload
instance.rebuild	open
instance.rebuild.scheduled	open			we might need to change the event_type
instance.resize	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	open
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

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

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

@@ Line 91: / Line 91: @@
 |}
-The complex instance.<action> notifications:
+The complex instance.<action> notifications that needs payload extension.
 {| class="wikitable"
 |-
 ! event type !! status !! taken by !! review !! comments
 |-
-| Example || Example || Example || Example || Example
+| instance.update || needs patch || gibi ||  ||
 |-
-| Example || Example || Example || Example || Example
+| instance.exists || open ||  ||  ||
 |-
-| Example || Example || Example || Example || Example
+| instance.rescue || open ||  ||  ||
 |-
-| Example || Example || Example || Example || Example
+| instance.volume.attach || open ||  ||  ||
-|-
-| 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
 |}