-
Notifications
You must be signed in to change notification settings - Fork 81
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
[RFR][VirtualCenter] Added method set hot add for cpu and memory #365
Conversation
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.
looks good at first sight but I'd like you to do some changes.
Besides I suppose you're going to use these new methods in tests, right ? So, please create appropriate PR which uses this PR. You just need to temporarily update requirements to refer to your repo and branch. This will facilitate quicker review since we won't need to pull your PR and test it locally.
Signed-off-by: Nikhil Dhandre <ndhandre@redhat.com>
Signed-off-by: Nikhil Dhandre <ndhandre@redhat.com>
@niyazRedhat can you please add test related to changes?. If you need help let me know. |
@digitronik Sure |
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.
well done !
Signed-off-by: Nikhil Dhandre <ndhandre@redhat.com>
Signed-off-by: Nikhil Dhandre ndhandre@redhat.com