-
Notifications
You must be signed in to change notification settings - Fork 117
Detail on changing primary storage scope added to adminguide and concepts #395
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…conceptsandterminology
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abh1sar |
@weizhouapache you mean the cluster->zone change scope form in the adminguide? |
@andrijapanicsb please review |
@abh1sar |
@abh1sar |
@weizhouapache resolved conflicts and added Cluster->Zone UI screenshot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to update this image if the dialog has changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, updated the image.
@blueorangutan docbuild |
@andrijapanicsb a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress. |
I don't see the other image, but I've asked BO to build docs again, and if both images are visible, I'm LGTM with the rest, nice explanation. |
QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/395. (QA-JID 226) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
source/adminguide/storage.rst
Outdated
Storage Pool, it cannot be disconnected from the hosts. In this case the Scope | ||
change operation will error out. Such VMs need to be stopped or migrated first. | ||
|
||
This feature is tested and supported for the following Hypervisor-Protocol- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feature is tested and supported for the following Hypervisor-Protocol- | |
This feature is tested and supported for the following Hypervisor and storage combinations: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
I made a few proposals to make it more readable from the end user (user doesn't necessarily care about "protocol" and "provider" - simply saying "KVM with NFS" is enough - again - just a proposal from my side, to simplify it. |
@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/395. (QA-JID 227) |
@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/395. (QA-JID 229) |
@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/395. (QA-JID 234) |
@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/395. (QA-JID 237) |
Documentation for Feature PR : apache/cloudstack#8875
📚 Documentation preview 📚: https://cloudstack-documentation--395.org.readthedocs.build/en/395/