-
Notifications
You must be signed in to change notification settings - Fork 1.3k
server: fix network access for addNicToVirtualMachine API #5184
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
server: fix network access for addNicToVirtualMachine API #5184
Conversation
weizhouapache
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.
code lgtm
|
@shwstppr re-target to 4.15 ? |
New NICs shouldn't added to VM with network from admin account, same as deployment. Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
e4831ff to
e12a7e9
Compare
|
@weizhouapache done @blueorangutan package |
|
@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔️ el7 ✔️ el8 ✔️ debian. SL-JID 481 |
|
@blueorangutan test |
|
@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-1210)
|
Pearl1594
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. verified behavior - add NIC to VM behaviour aligns with deployment workflow
Description
New NICs shouldn't be added to VM with Shared and Isolated networks from admin account, same as deployment.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
Tested:
Networks
Current behaviour for NIC/networks (works for all networks):
Current deployment behaviour (works for L2 network, fails for Isolated and Shared networks)
Current attach data disk behaviour (fails)
After changes add NIC/network behaviour (fails for Isolated and shared networks but works for L2 networks)