DaanHoogland commented on pull request #5884:
URL: https://github.com/apache/cloudstack/pull/5884#issuecomment-1021153912


   @utchoang I think your fix is good for destroy VM calls. It does not adress 
the generic race condition that is in the UI on any delete action. I have seen 
this on different types of offerings and it could also happen on for instance 
allert, event, account, user, network, ...
   
   That said, I could not test this in the QA environment, so I'd need to 
install a dedicated environment. If someone else manages to test this in the qa 
env, please.
   
   code looks good but as said, the solution only applies to the destroyVM API.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to