Skip to content

vm: note on kvm dynamic scaling#283

Merged
shwstppr merged 4 commits intoapache:4.17from
shapeblue:add-note-kvm-dynscaling
Aug 26, 2022
Merged

vm: note on kvm dynamic scaling#283
shwstppr merged 4 commits intoapache:4.17from
shapeblue:add-note-kvm-dynscaling

Conversation

@shwstppr
Copy link
Copy Markdown
Contributor

Related apache/cloudstack#6626

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Copy Markdown
Contributor Author

@blueorangutan docbuild

@blueorangutan
Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

Doc build preview: http://qa.cloudstack.cloud/docs/WIP-PROOFING/pr/283. (SL-JID 263)

Comment thread source/adminguide/virtual_machines.rst Outdated
@DaanHoogland
Copy link
Copy Markdown
Contributor

@shwstppr this is in the chapter on vm instance creation. Is it required to put this in some prerequisites chapter? (This is a genuine question but it seems this remark is a bit after the fact on this location)

shwstppr and others added 2 commits August 22, 2022 13:29
@shwstppr
Copy link
Copy Markdown
Contributor Author

@shwstppr this is in the chapter on vm instance creation. Is it required to put this in some prerequisites chapter? (This is a genuine question but it seems this remark is a bit after the fact on this location)

@DaanHoogland can you please suggest the appropriate location where the note can be added?

@shwstppr shwstppr requested a review from yadvr August 22, 2022 08:21
@DaanHoogland
Copy link
Copy Markdown
Contributor

@shwstppr this is in the chapter on vm instance creation. Is it required to put this in some prerequisites chapter? (This is a genuine question but it seems this remark is a bit after the fact on this location)

@DaanHoogland can you please suggest the appropriate location where the note can be added?

I don´t know @shwstppr , It is a genuine question. I am thinking of the quick installation guide, the paragraph on KVM install (http://docs.cloudstack.apache.org/en/latest/quickinstallationguide/qig.html#kvm-setup-and-installation) or the Installation guide (http://docs.cloudstack.apache.org/en/latest/installguide/hypervisor/kvm.html#system-requirements-for-kvm-hypervisor-hosts)

It is certainly not wrong to mention it here, but also something I would like to know as an operator before choosing and installing hypervisors.

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Copy Markdown
Contributor Author

@DaanHoogland I've added a note in http://docs.cloudstack.apache.org/en/latest/installguide/hypervisor/kvm.html#system-requirements-for-kvm-hypervisor-hosts as well. I guess it is okay to have it at both places?

@DaanHoogland
Copy link
Copy Markdown
Contributor

@DaanHoogland I've added a note in http://docs.cloudstack.apache.org/en/latest/installguide/hypervisor/kvm.html#system-requirements-for-kvm-hypervisor-hosts as well. I guess it is okay to have it at both places?

yes, I agree

@shwstppr shwstppr requested a review from Pearl1594 August 26, 2022 04:53
Copy link
Copy Markdown
Contributor

@Pearl1594 Pearl1594 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - may be worth adding error message that is seen on Qemu end that inidicates qemu's inability to support dynamic scaling

@shwstppr shwstppr merged commit fb0d37c into apache:4.17 Aug 26, 2022
weizhouapache pushed a commit to weizhouapache/cloudstack-documentation that referenced this pull request Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants