Closed ansijain closed 3 years ago
Hi @ansijain. Thanks for your PR.
I'm waiting for a kubevirt member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/ok-to-test @ansijain Thank you for your PR and for catching the issue with masquerade versus bridge. I had meant to alter vm.yaml to switch it over to masquerade, but apparently my change didn't make it into my PR somehow. (I previously thought it was a CI issue, but just checked the file's blame log, and cannot find my change)
Since you caught this, would you be okay with altering your PR to flip the network type in vm.yaml? Would you see any reason not to start with a masquerade network type in lab1?
/ok-to-test @ansijain Thank you for your PR and for catching the issue with masquerade versus bridge. I had meant to alter vm.yaml to switch it over to masquerade, but apparently my change didn't make it into my PR somehow. (I previously thought it was a CI issue, but just checked the file's blame log, and cannot find my change)
Since you caught this, would you be okay with altering your PR to flip the network type in vm.yaml? Would you see any reason not to start with a masquerade network type in lab1?
Well I can change that as well but I am not sure if that yaml is used anywhere where bridge network is required.
/test kubevirt-io-presubmit-link-checker
/retest
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cwilkers
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest One more try, but I think we have a systemic issue with CI.
Signed-off-by: ansijain ansi.jain@india.nec.com
What this PR does / why we need it: Added yaml file for migration lab that uses masquerade n/w instead of bridge Also updated the migration lab with new yaml file
Does this PR fix any issue? _(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged)_:Special notes for your reviewer: