<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 9, 2014 at 5:00 PM, ž¤ĒLĒG <span dir="ltr"><<a href="mailto:lilinguo8212@gmail.com" target="_blank">lilinguo8212@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Oh, I see. Thank you very much.<br>
It's just hard coded for attaching volume and swapping volume.<br>
<br>
How to deal the bp:<br>
<a href="https://blueprints.launchpad.net/nova/+spec/add-delete-on-termination-option" target="_blank">https://blueprints.launchpad.net/nova/+spec/add-delete-on-termination-option</a><br>
?<br>
<br></blockquote><div><br></div><div>So I think the only thing left in your bp would be adding a delete on terminate option when attaching a volume to an existing server and the novaclient changes. So I'd cleanup the blueprint and then set the milestone target to icehouse-3 which will trigger it to get it reviewed. Perhaps consider whether its reasonable to just apply this to the V3 API rather than doing an enhancement for both the V2 and V3 API.<br>
<br></div><div>Regards,<br><br></div><div>Chris<br></div><div></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2014/1/9 Christopher Yeoh <<a href="mailto:cbkyeoh@gmail.com">cbkyeoh@gmail.com</a>>:<br>
<div class="HOEnZb"><div class="h5">> On Thu, Jan 9, 2014 at 2:35 PM, ž¤ĒLĒG <<a href="mailto:lilinguo8212@gmail.com">lilinguo8212@gmail.com</a>> wrote:<br>
>><br>
>> Hi Chris,<br>
>> Thanks for you reply.<br>
>><br>
>> It's not only hard coded for swap volumes. In function<br>
>> '_create_instance' which for creating instance of nova/compute/api.py,<br>
>> the '_prepare_image_mapping' function will be called. And it hard code<br>
>> to True, too.<br>
>><br>
>> values = block_device.BlockDeviceDict({<br>
>>                 'device_name': bdm['device'],<br>
>>                 'source_type': 'blank',<br>
>>                 'destination_type': 'local',<br>
>>                 'device_type': 'disk',<br>
>>                 'guest_format': guest_format,<br>
>>                 'delete_on_termination': True,<br>
>>                 'boot_index': -1})<br>
>><br>
><br>
> Just before that in _prepare_image_mapping is:<br>
><br>
>             if virtual_name == 'ami' or virtual_name == 'root':<br>
>                 continue<br>
><br>
>             if not block_device.is_swap_or_ephemeral(virtual_name):<br>
>                 continue<br>
><br>
><br>
> Chris<br>
><br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<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>
><br>
<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>