You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2020/05/07 08:47:47 UTC

[GitHub] [cloudstack] nvazquez opened a new pull request #3595: [WIP DO NOT MERGE] KVM dynamic VM scaling

nvazquez opened a new pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595


   ## Description
   Allow dynamic VM scaling for KVM
   
   Fixes: #3594 
   
   Changes:
   - When deploying VM, the VM vCPU max number is set to the host CPU number. This allows virsh to set vCPU numbers and avoid the error:
   ````
   invalid argument: requested vcpus is greater than max allowable vcpus for the live domain: N > M
   ```` 
   
   ## Types of changes
   - [x] Enhancement (improves an existing feature and functionality)
   
   ## Screenshots (if appropriate):
   
   ## How Has This Been Tested?
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on pull request #3595: [WIP DO NOT MERGE] KVM dynamic VM scaling

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595#issuecomment-738734580


   @nvazquez Is this worth moving forward for 4.16 or should we close and restart?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on pull request #3595: [WIP DO NOT MERGE] KVM dynamic VM scaling

Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595#issuecomment-963442883


   Sorry, yes let's close it in favour of the PR referenced above


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] nvazquez closed pull request #3595: [WIP DO NOT MERGE] KVM dynamic VM scaling

Posted by GitBox <gi...@apache.org>.
nvazquez closed pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on pull request #3595: [WIP DO NOT MERGE] KVM dynamic VM scaling

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595#issuecomment-863919295


   @nvazquez (cc @rhtyd ) please considder #4878, it sugests to solve the same thing. Let's see if we need to merge the two or choose either.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #3595: [WIP DO NOT MERGE] KVM dynamic VM scaling

Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595#issuecomment-643067266


   Okay @nvazquez keep me posted when this is ready for review. Thanks.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on pull request #3595: [WIP DO NOT MERGE] KVM dynamic VM scaling

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595#issuecomment-938496635


   @nvazquez can we close this?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org