-
Notifications
You must be signed in to change notification settings - Fork 120
Added Limitations section to Extensions #547
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
shwstppr
left a comment
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.
Some comments
|
|
||
| - Affinity Groups. | ||
|
|
||
| - Usage and Billing. |
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.
Usage should be supported for VM and network. Only storage events can't be tracked I think
Migration should be added?
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.
Right. I'll remove Usage and add Migration.
|
|
||
| - Reinstall Instance. | ||
|
|
||
| - Migrate Instance to another Host. |
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.
I don't think migration would work correctly right now. I mean, it won't sync the host change back in CloudStack.
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.
Correct
|
|
||
| - Affinity Groups. | ||
|
|
||
| - Migrate Instance. |
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 could we also add host capacity & utilisation? Also do we support multiple nics (post-deployment or on initial deployment?)
It may be exhaustive to list what we don't support, perhaps we can say at the end specifically what is supported and that all the list of things (above) are some of the general things that we don't support?
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, I'll add host capacity and utilisation.
multiple nics are supported during initial deployment only.
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.
@rohityadavcloud I have changed the wording so that it doesn't appear to be an exhaustive list. I haven't added a list of what is supported, as I think the documentation already has that info.
|
@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/547. (QA-JID 412) |
|
|
||
| - User Data and Metadata services. | ||
|
|
||
| - SSH key injection. |
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.
userdata and sshkey injections will work right! as these are VR related operations and we do support all networking via VR. cc @shwstppr @rohityadavcloud
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.
Ideally they should, but I think there are some issues with userdata and metadata services via VR. I don't know the exact root cause though.
rajujith
left a comment
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
|
@abh1sar as an other way, is it better to mention the supported operations (VM deployment excluding data disk provisioning, VM power operations and network operations via VR) and all others as not supported ? cc @shwstppr @rohityadavcloud |
IMO it is less confusing for users, if we upfront mention some common limitations. If we just mention what is supported, then what's not supported is open to interpretation. The documentation already contains details on what all operations users can do. But I can add the list here as well, if you think that would be better. |
harikrishna-patnala
left a comment
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
📚 Documentation preview 📚: https://cloudstack-documentation--547.org.readthedocs.build/en/547/