<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 9, 2014 at 9:25 AM, 黎林果 <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">Hi All,<br>
<br>
   Attach a volume when creating a server, the API contains<br>
'block_device_mapping', such as:<br>
"block_device_mapping": [<br>
            {<br>
                "volume_id": "<VOLUME_ID>",<br>
                "device_name": "/dev/vdc",<br>
                "delete_on_termination": "true"<br>
            }<br>
]<br>
<br>
It allows the option 'delete_on_termination', but in the code it's<br>
hardcoded to True. Why?<br></blockquote><div><br></div><div>I don't think it does hardcode it to true. The API appears to be passing it down<br>correctly. I can see one case of delete_on_termination being set to true, though thats<br>
for ephemeral or swap volumes.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    Another situation, attach a volume to an exists server, there is<br>
not the option 'delete_on_termination'.<br>
<br>
  Should we add the 'delete_on_termination' when attach a volume to an<br>
exists server or modify the value from the params?<br>
<br></blockquote><div><br></div><div>I think adding delete_on_termination option when attaching a volume to an existing server<br></div><div>is reasonable. Perhaps add it to the v3 API?<br></div><div><br></div><div>Regards,<br>
<br>Chris<br><br></div></div></div></div>