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: [email protected]?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
