<div id="geary-body"><div><div>Hi,</div><div><br></div><div>Here is the status update / focus settings mail for 2018 w3.</div><div><br></div><div>Bugs</div><div>----</div><div>[High] <a href="https://bugs.launchpad.net/nova/+bug/1742935TestServiceUpdateNotificationSample">https://bugs.launchpad.net/nova/+bug/1742935TestServiceUpdateNotificationSample</a> fails intermittently: u'host2' != u'host1': path: root.payload.nova_object.data.host</div><div>The openstack-tox-functional (and functional-py35) test environment was totally broken during last Friday. Sorry for that. The patch that caused the break has been reverted <a href="https://review.openstack.org/#/c/533190/">https://review.openstack.org/#/c/533190/</a></div><div>A follow up bug has been opened (see next) to avoid similar break in the future.</div><div><br></div><div>[High] <a href="https://bugs.launchpad.net/nova/+bug/1742962">https://bugs.launchpad.net/nova/+bug/1742962</a> nova functional test does not triggered on notification sample only changes</div><div>During the zuul v3 migration the project-config generated based on the zuul v2 jobs. It contained a proper definition of when nova wants to trigger the functional job. Unfortunately this job definition does not override the openstack-tox-functional job definition from the openstack-zuul-jobs repo. This caused that the openstack-tox-functional (and functional-py35) jobs were not triggered for certain commits. The fix is to create a nova specific tox-functional job in tree. Patches has been proposed:</div><div>* <a href="https://review.openstack.org/#/c/533210/">https://review.openstack.org/#/c/533210/</a> Make sure that functional test triggered on sample changes</div><div>* <a href="https://review.openstack.org/#/c/533608/">https://review.openstack.org/#/c/533608/</a> Moving nova functional test def to in tree</div><div>In general we have to review all nova jobs in the project-config and move those in-tree that try to override parameters of the job definitions in openstack-zuul-jobs repo.</div><div><br></div><div>[High] <a href="https://bugs.launchpad.net/nova/+bug/1737201">https://bugs.launchpad.net/nova/+bug/1737201</a> TypeError when</div><div>sending notification during attach_interface</div><div>Fix merged to master. Backports have been proposed:</div><div>* Pike: <a href="https://review.openstack.org/#/c/531745/">https://review.openstack.org/#/c/531745/</a></div><div>* Queens: <a href="https://review.openstack.org/#/c/531746/">https://review.openstack.org/#/c/531746/</a></div><div><br></div><div>[High] <a href="https://bugs.launchpad.net/nova/+bug/1739325">https://bugs.launchpad.net/nova/+bug/1739325</a> Server operations</div><div>fail to complete with versioned notifications if payload contains unset</div><div>non-nullable fields</div><div>Patch has been proposed: <a href="https://review.openstack.org/#/c/529194/">https://review.openstack.org/#/c/529194/</a></div><div>Dan left feedback on it and I accept his comment that this is mostly papering over a problem that we don't fully understand how can happen in the first place. In the other hand I don't know how can we figure out what happend. So if somebody has an idea then don't hesistate to tell me.</div><div><br></div><div>[Low] <a href="https://bugs.launchpad.net/nova/+bug/1742688">https://bugs.launchpad.net/nova/+bug/1742688</a> test_live_migration_actions notification sample test fails intermittently with 'notification instance.live_migration_rollback.start hasn't been received'</div><div>It seems that test execution in CI is a lot slower than before and it makes the 1 second timeout in the notification test too small. Fix is on the gate: <a href="https://review.openstack.org/#/c/532816">https://review.openstack.org/#/c/532816</a></div><div><br></div><div>[Low] <a href="https://bugs.launchpad.net/nova/+bug/1487038">https://bugs.launchpad.net/nova/+bug/1487038</a></div><div>nova.exception._cleanse_dict should use</div><div>oslo_utils.strutils._SANITIZE_KEYS</div><div>Old abandoned patches exist but need somebody to pick them up:</div><div>* <a href="https://review.openstack.org/#/c/215308/">https://review.openstack.org/#/c/215308/</a></div><div>* <a href="https://review.openstack.org/#/c/388345/">https://review.openstack.org/#/c/388345/</a></div><div><br></div><div>Versioned notification transformation</div><div>-------------------------------------</div><div>Here are the patches ready to review:</div><div>* <a href="https://review.openstack.org/#/c/385644">https://review.openstack.org/#/c/385644</a> Transform rescue/unrescue instance notifications</div><div>Needs only a second +2</div><div>* <a href="https://review.openstack.org/#/c/403660">https://review.openstack.org/#/c/403660</a> Transform instance.exists notification</div><div>* <a href="https://review.openstack.org/#/c/410297">https://review.openstack.org/#/c/410297</a> Transform missing delete notifications</div><div>* <a href="https://review.openstack.org/#/c/476459">https://review.openstack.org/#/c/476459</a> Send soft_delete from context manager</div></div><div><br></div><div><div>Introduce instance.lock and instance.unlock notifications</div><div>-----------------------------------------------------------</div><div>A specless bp has been proposed to the Rocky cycle</div><div><a href="https://blueprints.launchpad.net/nova/+spec/trigger-notifications-when-lock-unlock-instances">https://blueprints.launchpad.net/nova/+spec/trigger-notifications-when-lock-unlock-instances</a></div><div>Some preliminary discussion happened in an earlier patch</div><div><a href="https://review.openstack.org/#/c/526251/">https://review.openstack.org/#/c/526251/</a></div><div><br></div><div>Add the user id and project id of the user initiated the instance action to the notification</div><div>--------------------------------------------------------------------------------------------</div><div>A new bp has been proposed <a href="https://blueprints.launchpad.net/nova/+spec/add-action-initiator-to-instance-action-notifications">https://blueprints.launchpad.net/nova/+spec/add-action-initiator-to-instance-action-notifications</a></div><div>As the user who initiates the instance action (e.g. reboot) could be different from the user owning the instance it would make sense to include the user_id and project_id of the action initiatior to the versioned instance action notifications as well.</div><div><br></div><div>Factor out duplicated notification sample</div><div>-----------------------------------------</div><div><a href="https://review.openstack.org/#/q/topic:refactor-notification-samples+status:open">https://review.openstack.org/#/q/topic:refactor-notification-samples+status:open</a></div><div>We have to be carefull to approve these type of commits until the solution for <a href="https://bugs.launchpad.net/nova/+bug/1742962">https://bugs.launchpad.net/nova/+bug/1742962</a> merged as functional tests could be broken silently.</div><div><br></div><div>Weekly meeting</div><div>--------------</div><div>There will not be a meeting this week. The next meeting will be held on 23th of January.</div><div><a href="https://www.timeanddate.com/worldclock/fixedtime.html?iso=20180123T170000">https://www.timeanddate.com/worldclock/fixedtime.html?iso=20180123T170000</a></div><div><br></div><div>Cheers,</div><div>gibi</div></div><div><br></div></div>