Allow copy of templates from secondary storages of same or other zone when adding a new secondary storage #611
Conversation
…y added secondary storage
| | | when adding a new Secondary Storage, instead of downloading them from the source URL. | | | ||
| +----------------------------------------------+-------------------------------------------------------------------------------------------------------------+-----------+ | ||
|
|
||
| This setting is enabled by default and can be configured globally or at zone level. |
There was a problem hiding this comment.
when creating a zone it is enabled by default, so preventing it will be hard right? A race against the machine.
There was a problem hiding this comment.
good point Daan. When creating a zone, it uses the global setting (not zone level). I also want to keep the default value to false to keep it backward compatible. But the first PR is already merged apache/cloudstack#10363, I can do that change in my subsequent PR apache/cloudstack#12296
One another point to note, if the setting is enabled, first it attempts to copy from secondary storage, if it fails then it fall back to old behavior.
There was a problem hiding this comment.
I do not mind enabling by default, just something to be aware of (and maybe document here). If useful to alternate we may wat to add a switch to the zone wizard (i.e set it before enabling the zone)
|
@blueorangutan docbuild |
|
@abh1sar a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress. |
|
QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/611. (QA-JID 470) |
abh1sar
left a comment
abh1sar
left a comment
There was a problem hiding this comment.
LGTM.
some nitpicks regarding capitalisations.
|
thank you @abh1sar I've addressed all of them |
4 participants
This document is in relation to the PRs apache/cloudstack#10363 and apache/cloudstack#12296
📚 Documentation preview 📚: https://cloudstack-documentation--611.org.readthedocs.build/en/611/