Thank you very much for the review and updates during the weekends. On Sat, Jul 14, 2018 at 4:05 AM Matt Riedemann <mriedemos at gmail.com> wrote: > On 7/11/2018 9:03 PM, Zhenyu Zheng wrote: > > 2. Abort live migration in queued state: > > - > https://blueprints.launchpad.net/nova/+spec/abort-live-migration-in-queued-status > > - > https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+(status:open+OR+status:merged) > > < > https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+%28status:open+OR+status:merged%29 > > > > - 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 oncompute.api.py <http://compute.api.py/>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. > > > > > > 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. > > That series is all rebased now and I'm +2 up the stack until the API > change, where I'm just +1 since I wrote the compute service version > checking part, but I think this series is ready for wider review. > > -- > > Thanks, > > Matt > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20180716/1f9bdb1f/attachment.html>