GaOrtiga commented on PR #8556:
URL: https://github.com/apache/cloudstack/pull/8556#issuecomment-1946485515

   @rohityadavcloud 
   > * Would this also deletion of systemvmtemplate current in use?
   > * If that's the case, can we add a check so it won't allow deleting 
systemvmtemplate in use
   
   It will not allow the deletion of the current template unless both the 
`isSystem` and the `forced` flag are passed. 
   
   > * Does it need some UI changes, so admins can delete systemvmtemplates 
(not in use) from the UI?
   
   Given some of the previous discussions regarding this implementation, mainly 
about security, I believe this PR should focus solely on the backend. If, in 
the future, we decide that UI changes are beneficial, we can make a new PR to 
implement it.


-- 
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-unsubscr...@cloudstack.apache.org

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

Reply via email to