<div dir="ltr">Thank you very much for the review and updates during the weekends.</div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jul 14, 2018 at 4:05 AM Matt Riedemann <<a href="mailto:mriedemos@gmail.com">mriedemos@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 7/11/2018 9:03 PM, Zhenyu Zheng wrote:<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>
>     <<a href="https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+%28status:open+OR+status:merged%29" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+%28status:open+OR+status:merged%29</a>><br>
>     - Weekly Progress: Review is going and it is in nova runway this<br>
>     week. In API office hour, we discussed about doing the compute<br>
>     service version checks <a href="http://oncompute.api.py" rel="noreferrer" target="_blank">oncompute.api.py</a> <<a href="http://compute.api.py/" rel="noreferrer" target="_blank">http://compute.api.py/</a>>side<br>
>     than on rpc side. Dan has point of doing it on rpc side where<br>
>     migration status can changed to running. We decided to further<br>
>     discussed it on patch.<br>
> <br>
> <br>
> This is my own defence, Dan's point seems to be that the actual rpc <br>
> version pin could be set to be lower than the can_send_version even when <br>
> the service version is new enough, so he thinks doing it in rpc is better.<br>
<br>
That series is all rebased now and I'm +2 up the stack until the API <br>
change, where I'm just +1 since I wrote the compute service version <br>
checking part, but I think this series is ready for wider review.<br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt<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>