[openstack-dev] [nova] otification update week 41

Balazs Gibizer balazs.gibizer at ericsson.com
Mon Oct 9 11:00:21 UTC 2017


Hi,

Here is the status update / focus settings mail for w41.

Bugs
----
[Medium] https://bugs.launchpad.net/nova/+bug/1699115 api.fault
notification is never emitted
It seems that rackspace does not use api.fault notification [2][3] So 
the agreement seems to be to remove the dead code.
Patch is proposed and needs a second +2:
* https://review.openstack.org/#/c/505164/ Remove dead code of 
api.fault notification sending

[2] 
http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-09-26.log.html#t2017-09-26T19:57:03
[3] 
http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-09-27.log.html#t2017-09-27T08:30:57


[Undecided] https://bugs.launchpad.net/nova/+bug/1718226 bdm is 
wastefully loaded for versioned instance notifications
Patch series needs a second +2: 
https://review.openstack.org/#/q/topic:bug/1718226

[High] https://bugs.launchpad.net/nova/+bug/1706563
TestRPC.test_cleanup_notifier_null fails with timeou
[High] https://bugs.launchpad.net/nova/+bug/1685333 Fatal Python error:
Cannot recover from stack overflow. - in py35 unit test job
The first bug is just a duplicate of the second. It seems the TetRPC
test suite has a way to end up in an infinite recusion.
Two compeeting patches were proposed that might help us getting at 
least logs from the failing test cases but there is no decision which 
patch should be merged.
* https://review.openstack.org/#/c/507253/
* https://review.openstack.org/#/c/507239/

[Undecided] https://bugs.launchpad.net/nova/+bug/1721670 Build 
notification in conductor fails to send due to InstanceNotFound Edit
Patch has been proposed: https://review.openstack.org/#/c/509967/

[Undecided] https://bugs.launchpad.net/nova/+bug/1721843 Unversioned 
notifications not being sent
Regression in the legacy notifications introduced when the short 
cutting of the versioned notification payload generation was added. 
Only affects compute.instance.update legacy notification.


Versioned notification transformation
-------------------------------------
The interface.attach / detach notifications has been merged. \o/
Here are the 3 patches for this week:
* https://review.openstack.org/#/c/396210 Transform aggregate.add_host 
notification
* https://review.openstack.org/#/c/396211 Transform 
aggregate.remove_host notification
* https://review.openstack.org/#/c/443764 use context mgr in 
instance.delete

Just a reminder that the versioned notification burndown chart is 
available on a new address: 
http://burndown.peermore.com/nova-notification/


Small improvements
------------------

* 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.
Takashi pointed out in the review that the current proposal actually 
changes the content of the sample appearing in our documentations. The 
reason is that some fields of the common sample fragment is overridden 
only during the functional test run and not during the doc generation. 
We agreed with Matt to try to make the override in a more clever way 
that applies to both the functional test and the doc generation.
The series needs to be updated.

Weekly meeting
--------------
Next subteam meeting will be held on 10th of October, Tuesday 17:00 UTC 
on openstack-meeting-4.
https://www.timeanddate.com/worldclock/fixedtime.html?iso=20171010T170000


Cheers,
gibi






More information about the OpenStack-dev mailing list