Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

The utility function govc.RetrieveVMNameFromIP() is not working properly when guestOS flips its IP address as docker IP #1284

Closed
shaominchen opened this issue May 23, 2017 · 4 comments

Comments

@shaominchen
Copy link
Contributor

In PR #1278 we introduced a testing utility function govc.RetrieveVMNameFromIP(), which is supposed to get the VM name for the given IP by calling govmomi. During test, it turns out the function is not working properly with Photon VM - there are some issues dealing with Photon VM's guest IP.

The usage of this function has been temporarily commented out in PR #1276.

@lipingxue
Copy link
Contributor

I hit this issue on Ubuntu OS too, so it looks like not specific to Photon OS.

@govint
Copy link
Contributor

govint commented Jun 6, 2017 via email

@shuklanirdesh82 shuklanirdesh82 changed the title The utility function govc.RetrieveVMNameFromIP() is not working properly with Photon OS The utility function govc.RetrieveVMNameFromIP() is not working properly when guestOS flips its IP address as docker IP Jun 7, 2017
@shuklanirdesh82 shuklanirdesh82 added P1 and removed P0 labels Jun 13, 2017
@shaominchen shaominchen modified the milestones: 0.16 , 0.17 Jun 20, 2017
@shuklanirdesh82
Copy link
Contributor

shuklanirdesh82 commented Jun 26, 2017

@ashahi1

Can you please take care of this issue?

  1. uncomment the verification code ( added as part of [Automation Test] Add 2 basic test cases #1276) and invoke CI 3-4 times > if CI passed merge the change and close this PR

  2. incase of observing failure let's investigate further about the issue .. (https://github.com/vmware/docker-volume-vsphere/blob/master/tests/utils/inputparams/testparams.go#L136 ... our GetTestConfig is consuming it and our tests are heavily dependent on and not observed any issue related with reported one so far)

moving to 0.16 for performing action item-#1

@ashahi1
Copy link
Contributor

ashahi1 commented Jun 28, 2017

No longer seeing the original issue of RetrieveVMNameFromIP method not returning vm name. Tried 3-4 times on CI but original issue not reproducible.
Uncommented the part that was commented out in PR #1276 and merged as part of PR #1487.

Closing this issue now.

@ashahi1 ashahi1 closed this as completed Jun 28, 2017
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

6 participants