[openstack-dev] [nova]notification update week 29

Matt Riedemann mriedemos at gmail.com
Mon Jul 17 19:32:03 UTC 2017


On 7/17/2017 2:36 AM, Balazs Gibizer wrote:
> Hi,
> 
> Here is the status update / focus setting mail about notification work
> for week 29.
> 
> Bugs
> ----
> [Undecided] https://bugs.launchpad.net/nova/+bug/1684860 Versioned
> server notifications don't include updated_at
> The fix https://review.openstack.org/#/c/475276/ is in focus but 
> comments needs to be addressed.
> 
> [Low] https://bugs.launchpad.net/nova/+bug/1696152 nova notifications
> use nova-api as binary name instead of nova-osapi_compute
> Agreed not to change the binary name in the notifications. Instead we
> make an enum for that name to show that the name is intentional.
> Patch needs review:  https://review.openstack.org/#/c/476538/
> 
> [Undecided] https://bugs.launchpad.net/nova/+bug/1702667 publisher_id of 
> the versioned instance.update notification is not consistent with other 
> notifications
> The inconsistency of publisher_ids was revealed by #1696152. Patch needs 
> review: https://review.openstack.org/#/c/480984
> 
> [Undecided] https://bugs.launchpad.net/nova/+bug/1699115 api.fault
> notification is never emitted
> Still no response on the ML thread about the way forward.
> http://lists.openstack.org/pipermail/openstack-dev/2017-June/118639.html
> 
> [Undecide] https://bugs.launchpad.net/nova/+bug/1700496 Notifications
> are emitted per-cell instead of globally
> Fix is to configure a global MQ endpoint for the notifications in cells 
> v2. Patch looks good from notification perspective but affects other 
> part of the system as well: https://review.openstack.org/#/c/477556/
> 
> 
> Versioned notification transformation
> -------------------------------------
> The last week's merge conflicts are mostly cleaned up and there is 11 
> patches that are waiting for core revew:
> https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/versioned-notification-transformation-pike+label:Code-Review%253E%253D%252B1+label:Verified%253E%253D1+AND+NOT+label:Verified%253C0+AND+NOT+label:Code-Review%253C0 
> 
> 
> If you are affraid of the long list then here is a short list of live 
> migration related transformations to look at:
> * https://review.openstack.org/#/c/480214/
> * https://review.openstack.org/#/c/420453/
> * https://review.openstack.org/#/c/480119/
> * https://review.openstack.org/#/c/469784/
> 
> 
> Searchlight integration
> -----------------------
> bp additional-notification-fields-for-searchlight
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> The BDM addition has been merged.
> 
> As a last piece of the bp we are still missing the Add tags to 
> instance.create Notification https://review.openstack.org/#/c/459493/ 
> patch but that depends on supporting tags and instance boot 
> https://review.openstack.org/#/c/394321/ which is getting closer to be 
> merged. Focus is on these patches.
> 
> There are a set of follow up patches for the BDM addition to optimize 
> the payload generation but these are not mandatory for the functionality 
> https://review.openstack.org/#/c/483324/
> 
> 
> Instability of the notification sample tests
> --------------------------------------------
> Multiple instability of the sample test was detected last week. The nova 
> functional test fails intermittenly at least for two distinct reasons:
> * https://bugs.launchpad.net/nova/+bug/1704423 _test_unshelve_server 
> intermittently fails in functional versioned notification tests
> Possible solution found, fix proposed and it only needs a second +2: 
> https://review.openstack.org/#/c/483986/
> * https://bugs.launchpad.net/nova/+bug/1704392 
> TestInstanceNotificationSample.test_volume_swap_server fails with 
> "testtools.matchers._impl.MismatchError: 7 != 6"
> Patch that improves logging of the failure has been merged 
> https://review.openstack.org/#/c/483939/ and detailed log now available 
> to look at 
> http://logs.openstack.org/82/482382/4/check/gate-nova-tox-functional-ubuntu-xenial/38a4cb4/console.html#_2017-07-16_01_14_36_313757 
> 
> 
> 
> Small improvements
> ~~~~~~~~~~~~~~~~~~
> * https://review.openstack.org/#/c/428199/ Improve assertJsonEqual
> error reporting
> * https://review.openstack.org/#/q/topic:refactor-notification-samples
> Factor out duplicated notification sample data
> This is a start of a longer patch series to deduplicate notification
> sample data. The third patch already shows how much sample data can be
> deleted from nova tree. We added a minimal hand rolled json ref
> implementation to notification sample test as the existing python json
> ref implementations are not well maintained.
> 
> 
> Weekly meeting
> --------------
> The notification subteam holds it's weekly meeting on Tuesday 17:00 UTC
> on openstack-meeting-4. The next meeting will be held on 18th of July.
> https://www.timeanddate.com/worldclock/fixedtime.html?iso=20170718T170000
> 
> Cheers,
> gibi
> 
> 
> 
> 
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

What do you want to do with this blueprint?

https://blueprints.launchpad.net/nova/+spec/json-schema-for-versioned-notifications

I don't know if all of the dependencies are done, and it looks like the 
Nova changes are pretty stale. Should we just defer this to Queens?

-- 

Thanks,

Matt



More information about the OpenStack-dev mailing list