DK101010 commented on PR #6012:
URL: https://github.com/apache/cloudstack/pull/6012#issuecomment-1278665418

   > @DK101010 the systemvm iso has been deprecated so no need to use it 
anymore. I think the approach you are taking is fine, only one comment is that 
CloudStack would examine the system VM template checksum and compare it to the 
ones on the .ini file and in case they do not match it will attempt to download 
a new template
   
   @nvazquez @DaanHoogland 
   It is all fine, after the update failed I removed my whole environment and 
build a clean cs instance to prevent such things. 
   I checked the agent.zip and cloud-scripts.tgz to find the needed files but 
no luck. I'm still the opinion that a new template is needed or the agent 
initializing is wrong, but in that case we have bad source code in the main 
branch and I don't believe that.
   
   I would feel better if I could test it again in my own environment but this 
PR have extensive test to get sure that this feature works. so theoretically it 
could be merged or we move the merge to 4.19 :-/


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