<div dir="ltr">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">2. Abort live migration in queued state:<span> <br></span></span><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">-<span> </span></span><a href="https://blueprints.launchpad.net/nova/+spec/abort-live-migration-in-queued-status" rel="noreferrer" target="_blank" style="color:rgb(17,85,204);font-size:small">https://blueprints.launchpad.net/nova/+spec/abort-live-migration-in-queued-status</a><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><span> <br></span></span><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">-<span> </span></span><a href="https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+(status:open+OR+status:merged)" rel="noreferrer" target="_blank" style="color:rgb(17,85,204);font-size:small">https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+(status:open+OR+status:merged)</a><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><span> <br></span></span><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">- Weekly Progress: Review is going and it is in nova runway this week. In API office hour, we discussed about doing the compute service version checks on<span> </span></span><a href="http://compute.api.py/" rel="noreferrer" target="_blank" style="color:rgb(17,85,204);font-size:small">compute.api.py</a><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><span> </span>side than on rpc side. Dan has point of doing it on rpc side where migration status can changed to running. We decided to further discussed it on patch.<span> </span></span></blockquote><div><br></div><div>This is my own defence, Dan's point seems to be that the actual rpc version pin could be set to be lower than the can_send_version even when the service version is new enough, so he thinks doing it in rpc is better. </div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 12, 2018 at 9:15 AM Ghanshyam Mann <<a href="mailto:gmann@ghanshyammann.com">gmann@ghanshyammann.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All, <br>
<br>
Please find the Nova API highlights of this week. <br>
<br>
Weekly Office Hour: <br>
=============== <br>
We had more attendees in this week office hours. <br>
<br>
What we discussed this week: <br>
- Discussion on API related BP. Discussion points are embedded inline with BP weekly progress in next section. <br>
- Triage 1 new bug and Alex reviewed one in-progress <br>
<br>
Planned Features : <br>
============== <br>
Below are the API related features for Rocky cycle. Nova API Sub team will start reviewing those to give their regular feedback. If anythings missing there feel free to add those in etherpad- <a href="https://etherpad.openstack.org/p/rocky-nova-priorities-tracking" rel="noreferrer" target="_blank">https://etherpad.openstack.org/p/rocky-nova-priorities-tracking</a> <br>
<br>
1. Servers Ips non-unique network names : <br>
- <a href="https://blueprints.launchpad.net/nova/+spec/servers-ips-non-unique-network-names" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/nova/+spec/servers-ips-non-unique-network-names</a> <br>
- Spec Merged<br>
- <a href="https://review.openstack.org/#/q/topic:bp/servers-ips-non-unique-network-names+(status:open+OR+status:merged)" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/topic:bp/servers-ips-non-unique-network-names+(status:open+OR+status:merged)</a> <br>
- Weekly Progress: Spec is merged. I am in contact with author about code update (sent email last night). If no response till this week, i will push the code update for this BP. <br>
<br>
2. Abort live migration in queued state: <br>
- <a href="https://blueprints.launchpad.net/nova/+spec/abort-live-migration-in-queued-status" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/nova/+spec/abort-live-migration-in-queued-status</a> <br>
- <a href="https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+(status:open+OR+status:merged)" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+(status:open+OR+status:merged)</a> <br>
- Weekly Progress: Review is going and it is in nova runway this week. In API office hour, we discussed about doing the compute service version checks on <a href="http://compute.api.py" rel="noreferrer" target="_blank">compute.api.py</a> side than on rpc side. Dan has point of doing it on rpc side where migration status can changed to running. We decided to further discussed it on patch. <br>
<br>
3. Complex anti-affinity policies: <br>
- <a href="https://blueprints.launchpad.net/nova/+spec/complex-anti-affinity-policies" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/nova/+spec/complex-anti-affinity-policies</a> <br>
- <a href="https://review.openstack.org/#/q/topic:bp/complex-anti-affinity-policies+(status:open+OR+status:merged)" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/topic:bp/complex-anti-affinity-policies+(status:open+OR+status:merged)</a> <br>
- Weekly Progress: Good review progress. In API office hour, we discussed on 2 points-<br>
1. whether request also need to have flat format like response. IMO we need to have flat in both request and response. Yikun need more opinion on that. <br>
<br>
2. naming fields to policy_* as we are moving these new fields in flat format. I like to have policy_* for clear understanding of attributes by their name. This is not concluded <br>
and alex will give feedback on patch.<br>
Discussion is on patch for consensus on naming things. <br>
<br>
4. Volume multiattach enhancements: <br>
- <a href="https://blueprints.launchpad.net/nova/+spec/volume-multiattach-enhancements" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/nova/+spec/volume-multiattach-enhancements</a> <br>
- <a href="https://review.openstack.org/#/q/topic:bp/volume-multiattach-enhancements+(status:open+OR+status:merged)" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/topic:bp/volume-multiattach-enhancements+(status:open+OR+status:merged)</a> <br>
- Weekly Progress: mriedem mentioned in last week status mail that he will continue work on this. <br>
<br>
5. API Extensions merge work <br>
- <a href="https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-rocky" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-rocky</a> <br>
- <a href="https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/api-extensions-merge-rocky" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/api-extensions-merge-rocky</a> <br>
- Weekly Progress: I did not get chance to push more patches on this. I will target this one before next office hour. <br>
<br>
6. Handling a down cell<br>
- <a href="https://blueprints.launchpad.net/nova/+spec/handling-down-cell" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/nova/+spec/handling-down-cell</a><br>
- Spec mriedem mentioned in previous week ML is merged - <a href="https://review.openstack.org/#/c/557369/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/557369/</a><br>
<br>
Bugs: <br>
==== <br>
Triage 1 new bug and Alex reviewed one in-progress. I did not do my home work of doing review on in-progress patches (i will accommodate that in next week) <br>
<br>
This week Bug Progress: <br>
Critical: 0->0 <br>
High importance: 2->3 <br>
By Status:<br>
New: 1->0<br>
Confirmed/Triage: 30-> 31<br>
In-progress: 36->36<br>
Incomplete: 4->4<br>
=====<br>
Total: 70->71<br>
<br>
NOTE- There might be some bug which are not tagged as 'api' or 'api-ref', those are not in above list. Tag such bugs so that we can keep our eyes. <br>
<br>
Ref: <a href="https://etherpad.openstack.org/p/nova-api-weekly-bug-report" rel="noreferrer" target="_blank">https://etherpad.openstack.org/p/nova-api-weekly-bug-report</a> <br>
<br>
-gmann <br>
<br>
<br>
<br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>