-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
provider/vsphere: provide host
to provisioner connections
#5558
Conversation
} else { | ||
d.SetConnInfo(map[string]string{ | ||
"host": ip.(string), | ||
}) |
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.
It looks like this will call d.SetConnInfo
only when the waitForNetworkingActive
code path is not executed. Is this the desired behavior? Seems like we might want to SetConnInfo
after the boot waiting as well?
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.
That's right! I'll fix it soon!
0a534f2
to
2194b38
Compare
2194b38
to
71d93d2
Compare
@phinze I fixed the behavior and rebased to master branch. |
👍 This has been driving me crazy. Glad I stumbled across this. Would love to see this get implemented. Have to roll this out across some big datacenters and would like to now use work arounds. Great work. |
This LGTM - still working on getting a vSphere environment on our side to be able to properly test these PRs, but in the meantime this looks like a pretty straightforward win. 👍 |
provider/vsphere: provide `host` to provisioner connections
Thank you! |
@phinze I have vSphere 6 at home. Let me know if you need some help. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Provide
host
to provisioner connections to enable to use provisioner with vsphere provider.We cannot use remote-exec provisioner with vsphere provider for now, because vsphere provider doesn't provide
host
to provisioner connections. And the debug log is following:Workaround: