weizhouapache commented on pull request #4641:
URL: https://github.com/apache/cloudstack/pull/4641#issuecomment-919115487


   > @weizhouapache I'm just being risk-averse :) I see that checks are only 
postponed - but I worry if not doing checks at the time of initial connection 
causes any failures? For example, you allow the agent to connect and it starts 
doing some storage related operation, and later the post-connect thread/check 
runs and finds out we shouldn't have connected, or it fails but since the agent 
is connected, agent continues to process new commands? (a case can be the 
template/volume scanning logic times out, but the connect is still connected so 
agent gets some new operation say re-download template from a URL which is 
wrong/404 etc and it ends up overwriting existing files?)
   
   @rhtyd these are very good questions. 
   @ravening are you ok to close this pr ? the issue this pr aims to fix is, 
SSVM will be Up only if all imags/volumes are checked which I think it is not a 
major issue.
   


-- 
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