-
Notifications
You must be signed in to change notification settings - Fork 120
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
Merged
Merged
Changes from 4 commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
72de4ac
Details about changing primary storage scope added to adminguide and …
abh1sar 4fa38a9
Pull 395: Merge 4.19
abh1sar 5fe384c
Pull 395: Added screenshot for cluster to zone scope change
abh1sar d7fd333
Pull 395: Updated screenshot for cluster to zone scope change
abh1sar cd1fa24
Pull 395: updated image after change in dialog
abh1sar ebe9fe4
Pull 395: simplified hypervisor-storage combinations
abh1sar 553801c
Pull 395: Added info that this might be a long running operation
abh1sar 88f78a6
Pull 395: Added info about listAffectedVmsForstorageScopeChange api
abh1sar a52298a
Merge branch '4.19' into scope-419
abh1sar File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -185,6 +185,38 @@ ensure that the protocol is set to "Filesystem". | |||||
|
|
||||||
| |adding-local-pool-via-ui.png| | ||||||
|
|
||||||
| Changing the Scope of the Primary Storage | ||||||
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||||||
| Scope of a Primary Storage can be changed from Zone-wide to Cluster-wide | ||||||
| and vice versa when the Primary Storage is in Disabled state. | ||||||
| An action button is displayed in UI for each Primary Storage in Disabled state. | ||||||
|
|
||||||
| |change-storage-pool-scope-via-ui.png| | ||||||
|
|
||||||
| Scope change from Cluster to Zone will connect the Primary Storage to all Hosts | ||||||
| of the zone running the same hypervisor as set on the storage pool. | ||||||
|
|
||||||
| |change-storage-pool-scope-to-zone.png| | ||||||
|
|
||||||
| Scope change from Zone to Cluster will disconnect the Primary Storage from all | ||||||
| Hosts that were previously connected to the Primary Storage and are not a part | ||||||
| of the specified Cluster. So, if there are running VMs on such hosts using this | ||||||
| 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- | ||||||
|
||||||
| This feature is tested and supported for the following Hypervisor-Protocol- | |
| This feature is tested and supported for the following Hypervisor and storage combinations: |
Contributor
Author
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
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
abh1sar marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.