<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014/1/9 Russell Bryant <span dir="ltr"><<a href="mailto:rbryant@redhat.com" target="_blank">rbryant@redhat.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 01/08/2014 09:53 AM, John Garbutt wrote:<br>
> On 8 January 2014 10:02, David Xie <<a href="mailto:david.scriptfan@gmail.com">david.scriptfan@gmail.com</a>> wrote:<br>
>> In nova/compute/api.py#2289, function resize, there's a parameter named<br>
>> flavor_id, if it is None, it is considered as cold migration. Thus, nova<br>
>> should skip resize verifying. However, it doesn't.<br>
>><br>
>> Like Jay said, we should skip this step during cold migration, does it make<br>
>> sense?<br>
><br>
> Not sure.<br>
><br>
>> On Wed, Jan 8, 2014 at 5:52 PM, Jay Lau <<a href="mailto:jay.lau.513@gmail.com">jay.lau.513@gmail.com</a>> wrote:<br>
>>><br>
>>> Greetings,<br>
>>><br>
>>> I have a question related to cold migration.<br>
>>><br>
>>> Now in OpenStack nova, we support live migration, cold migration and<br>
>>> resize.<br>
>>><br>
>>> For live migration, we do not need to confirm after live migration<br>
>>> finished.<br>
>>><br>
>>> For resize, we need to confirm, as we want to give end user an opportunity<br>
>>> to rollback.<br>
>>><br>
>>> The problem is cold migration, because cold migration and resize share<br>
>>> same code path, so once I submit a cold migration request and after the cold<br>
>>> migration finished, the VM will goes to verify_resize state, and I need to<br>
>>> confirm resize. I felt a bit confused by this, why do I need to verify<br>
>>> resize for a cold migration operation? Why not reset the VM to original<br>
>>> state directly after cold migration?<br>
><br>
> I think the idea was allow users/admins to check everything went OK,<br>
> and only delete the original VM when the have confirmed the move went<br>
> OK.<br>
><br>
> I thought there was an auto_confirm setting. Maybe you want<br>
> auto_confirm cold migrate, but not auto_confirm resize?<br>
<br>
</div></div>I suppose we could add an API parameter to auto-confirm these things.<br>
That's probably a good compromise.<br></blockquote><div>OK, will use auto-confirm to handle this.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
>>> Also, I think that probably we need split compute.api.resize() to two<br>
>>> apis: one is for resize and the other is for cold migrations.<br>
>>><br>
>>> 1) The VM state can be either ACTIVE and STOPPED for a resize operation<br>
>>> 2) The VM state must be STOPPED for a cold migrate operation.<br>
><br>
> We just stop the VM them perform the migration.<br>
> I don't think we need to require its stopped first.<br>
> Am I missing something?<br>
<br>
</div>Don't think so ... I think we should leave it as is.<br></blockquote><div>OK, will leave this as it is for now. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
--<br>
Russell Bryant<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>