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