You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2022/11/03 22:07:57 UTC

[GitHub] [cloudstack] weizhouapache commented on pull request #6868: Set root volume as destroyed when destroying a VM

weizhouapache commented on PR #6868:
URL: https://github.com/apache/cloudstack/pull/6868#issuecomment-1302729629

   > I think you are missing a (not necessarily new) use-case where the root disk is expunged and the VM restored. both with and without this setting this will go wrong. You just made it a bit more explicit with your code.
   
   Right. Because vm can be recovered, the root disk should not be removed.
   If root disk is set to Destroyed, it will be cleaned after a period, then the vm cannot be recovered.
   
   If you want to exclude destroyed vm in resource count, please look at the methods for resource calculation. @JoaoJandre 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org