-
Notifications
You must be signed in to change notification settings - Fork 913
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
make processing of mac addresses case insensitive #2510
Conversation
Thx @rconde01 ! Would it be ok to ask for a unit/integration test, e.g. by using the simulator package? |
@embano1 I'll look at adding that |
@rconde01, VMware has approved your signed contributor license agreement. |
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.
Thanks @rconde01 , regarding the test against vcsim, you'll need #2511
With that, the test could be as simple as the diff below, or adding a new sequence of customize, power on and wait.ip
modified govc/test/vm.bats
@@ -7,7 +7,7 @@ load test_helper
id=/DC0/vm/DC0_H0_VM0
- mac=00:50:56:83:3a:5d
+ mac=00:50:56:83:3A:5D
run govc vm.customize -vm $id -mac $mac -ip 10.0.0.1 -netmask 255.255.0.0 -type Linux
assert_success
I added a test...let me know if there's anything else i need to do. |
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, thanks @rconde01 !
Description
Fixes WaitForNetIP when the vm contains a manual mac address which may contain uppercase characters.
Closes: #2509
Type of change
Please mark options that are relevant:
not work as expected)
Checklist:
this project