Difference between revisions of "Nova/VersionedNotificationTransformation"
< Nova
m (→Tasks) |
|||
(80 intermediate revisions by 15 users not shown) | |||
Line 1: | Line 1: | ||
+ | = This work has been completed = | ||
+ | |||
+ | The information below is kept here for historical reasons | ||
+ | |||
== Goal == | == Goal == | ||
Transform every legacy notification to the new versioned notification framework. | Transform every legacy notification to the new versioned notification framework. | ||
Line 4: | Line 8: | ||
== How to do it == | == How to do it == | ||
* Check the notification devref that describes the new framework: http://docs.openstack.org/developer/nova/notifications.html | * Check the notification devref that describes the new framework: http://docs.openstack.org/developer/nova/notifications.html | ||
− | * Check the existing | + | * Check the existing transformation patches that are merged: http://burndown.peermore.com/nova-notification/ |
− | * | + | * Select a transformation from the table on http://burndown.peermore.com/nova-notification/ You can take a patch with status TODO or you can check if a patch that is INPROGRESS needs help. |
− | |||
* Propose a transformation patch based on the examples. | * Propose a transformation patch based on the examples. | ||
** Make sure that you create a new sample file under doc/notificaiton_samples/ | ** Make sure that you create a new sample file under doc/notificaiton_samples/ | ||
** Make sure that you add a notification sample test under nova/tests/functional/notification_sample_test/ | ** Make sure that you add a notification sample test under nova/tests/functional/notification_sample_test/ | ||
− | * If you got stuck you can ask questions from | + | * If you got stuck you can ask questions from gibi on IRC or join the weekly notification subteam meeting https://wiki.openstack.org/wiki/Meetings/NovaNotification |
== Tasks == | == Tasks == | ||
− | + | Please use the https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-rocky topic for the transformation commits | |
+ | |||
+ | |||
+ | <big> [http://burndown.peermore.com/nova-notification/ The burndown chart is updated hourly from gerrit. ] </big> | ||
+ | You can see what to transform and what to review there. | ||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | The complex instance.<action> notifications that needs payload extension. | + | There are three types of transformations: |
− | + | * category: instance-payload: The easy ones that uses the same payload as instance.delete | |
− | + | * category: extended-instance-payload: The complex instance.<action> notifications that needs payload extension. | |
− | + | * category: specific-payload: The rest that needs specific payload | |
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | + | If something doesn't show up on the burndown then please ping gibi on IRC. | |
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− |
Latest revision as of 09:21, 10 December 2018
This work has been completed
The information below is kept here for historical reasons
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 transformation patches that are merged: http://burndown.peermore.com/nova-notification/
- Select a transformation from the table on http://burndown.peermore.com/nova-notification/ You can take a patch with status TODO or you can check if a patch that is INPROGRESS needs help.
- Propose a transformation patch based on the examples.
- Make sure that you create a new sample file under doc/notificaiton_samples/
- Make sure that you add a notification sample test under nova/tests/functional/notification_sample_test/
- If you got stuck you can ask questions from gibi on IRC or join the weekly notification subteam meeting https://wiki.openstack.org/wiki/Meetings/NovaNotification
Tasks
Please use the https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-rocky topic for the transformation commits
The burndown chart is updated hourly from gerrit.
You can see what to transform and what to review there.
There are three types of transformations:
- category: instance-payload: The easy ones that uses the same payload as instance.delete
- category: extended-instance-payload: The complex instance.<action> notifications that needs payload extension.
- category: specific-payload: The rest that needs specific payload
If something doesn't show up on the burndown then please ping gibi on IRC.