Skip to content
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

Update vendor/govmomi to release 0.11.2 #2977

Merged
merged 1 commit into from
Nov 2, 2016
Merged

Conversation

dougm
Copy link
Member

@dougm dougm commented Nov 2, 2016

Fixes #2968 (NSX-T)

Avoid possible VirtualMachine.Device NPE seen in #2818

Fix for #2878 (docker image update still required)

@dougm
Copy link
Member Author

dougm commented Nov 2, 2016

Seems github suppresses vendor/ changes? Diff here w/o all the clicking: vmware/govmomi@v0.10.0...v0.11.2

@emlin
Copy link
Contributor

emlin commented Nov 2, 2016

please hold on this fix. It will remove the dangling vm from ci, but I'm still using that for some test.

@emlin
Copy link
Contributor

emlin commented Nov 2, 2016

I'm fine with it now, thanks.
LGTM 👍

@hmahmood
Copy link
Contributor

hmahmood commented Nov 2, 2016

lgtm

@vburenin
Copy link
Contributor

vburenin commented Nov 2, 2016

LGTM

@dougm
Copy link
Member Author

dougm commented Nov 2, 2016

.

Fixes vmware#2968 (NSX-T)

Avoid possible VirtualMachine.Device NPE seen in vmware#2818

Fix for vmware#2878 (docker image update still required)
@dougm
Copy link
Member Author

dougm commented Nov 2, 2016

.

@dougm dougm merged commit 64d1802 into vmware:master Nov 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants