[openstack-dev] [nova] Notification update week 11
Hi, Here is the status update / focus settings mail for w11. Bugs No new bug and no changes in the existing bugs from last week report http://lists.openstack.org/pipermail/openstack-dev/2018-March/127992.html Versioned notification transformation - We already have some patches proposed to the rock bp. Let's go and review them. https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-rocky+status:open Introduce instance.lock and instance.unlock notifications - The bp https://blueprints.launchpad.net/nova/+spec/trigger-notifications-when-lock-unlock-instances is approved. Implementation patch exists but still needs work https://review.openstack.org/#/c/526251/ Add the user id and project id of the user initiated the instance action to the notification - The bp https://blueprints.launchpad.net/nova/+spec/add-action-initiator-to-instance-action-notifications is approved. Implementation patch exists but still needs work https://review.openstack.org/#/c/536243/ Add request_id to the InstanceAction versioned notifications The bp https://blueprints.launchpad.net/nova/+spec/add-request-id-to-instance-action-notifications is approved and assigned to Keving_Zheng. Patch has been proposed https://review.openstack.org/#/c/551982/ and needs review. Sending full traceback in versioned notifications - Based on a short investigation it seems that it was a conscious decision not to include the full traceback. See details in the ML post http://lists.openstack.org/pipermail/openstack-dev/2018-March/128105.html I will file a specless bp to add the full traceback if nobody objects in the ML thread. Add versioned notifications for removing a member from a server group - The specless bp https://blueprints.launchpad.net/nova/+spec/add-server-group-remove-member-notifications is proposed and it looks good to me. Factor out duplicated notification sample - https://review.openstack.org/#/q/topic:refactor-notification-samples+status:open No open patches, but I would like to progress with this through the Rocky cycle. Weekly meeting -- The next meeting will be held on 13th of Marc on #openstack-meeting-4 https://www.timeanddate.com/worldclock/fixedtime.html?iso=20180313T17 Cheers, gibi __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
[openstack-dev] [nova] notification update week 11
Hi, Here is the status update / focus setting mail about notification work for week 11. Bugs [Medium] https://bugs.launchpad.net/nova/+bug/1657428 The instance notifications are sent with inconsistent timestamp format. Fix is ready for the cores to review https://review.openstack.org/#/c/421981 [Undecided] https://bugs.launchpad.net/nova/+bug/1673375 ValueError: Circular reference detected" in send_notification. Fix proposed https://review.openstack.org/#/c/446948/ . Versioned notification transformation - No new goal setting, let's try to focus on to finish the goals of the last to weeks Patches that just need a second core: * https://review.openstack.org/#/c/382959 Transform instance.reboot notification * https://review.openstack.org/#/c/411791/ Transform instance.reboot.error notification * https://review.openstack.org/#/c/401992/ Transform instance.volume_attach notification * https://review.openstack.org/#/c/408676/ Transform instance.volume_detach notification Searchlight integration --- changing Searchlight to use versioned notifications ~~~ https://blueprints.launchpad.net/searchlight/+spec/nova-versioned-notifications bp is a hard dependency for the integration work. Searchlight team promised to provide a list of notifications needed to be transformed so that they can switch to the versioned interface. We will prioritize the transformation work on the nova side accordingly. bp additional-notification-fields-for-searchlight ~ Patches needs review: https://review.openstack.org/#/q/label:Code-Review%253E%253D1+status:open+branch:master+topic:bp/additional-notification-fields-for-searchlight The BlockDeviceMapping addition to the InstancePayload has been discussed on the ML and on the weekly meeting. Implementation is onging. Other items --- Short circuit notification payload generation ~ New oslo messaging has been released with the needed addition and global requirements has been bumped. Work needs to be continued on the nova patch https://review.openstack.org/#/c/428260/ Weekly meeting -- The notification subteam holds it's weekly meeting on Tuesday 17:00 UTC on openstack-meeting-4 so the next meeting will be held on 21th of March https://www.timeanddate.com/worldclock/fixedtime.html?iso=20170321T17 Cheers, gibi __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev