slavkap commented on PR #11177: URL: https://github.com/apache/cloudstack/pull/11177#issuecomment-3073096971
For me, the code looks good, but my only concern here is that the configuration `create.full.clone` should come from the management service as an API parameter or a property of a volume/VM, or template (I'm not sure exactly where). This is in case the administrators want some volumes to be full cloned and others not. In that case, they should change the value of the configuration every time from the agent.properties file, and restart the agent service -- 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
