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

Corrects host.shutdown #939

Merged
merged 3 commits into from
Nov 27, 2017
Merged

Corrects host.shutdown #939

merged 3 commits into from
Nov 27, 2017

Conversation

1995parham
Copy link
Contributor

host.shutdown expects HOST as an argument.

host.shutdown expects HOST as an argument.
@vmwclabot
Copy link
Member

@1995parham, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.

@dougm
Copy link
Member

dougm commented Nov 25, 2017

Thanks @1995parham , the USAGE.md doc is actually generated by running make doc. Can you add a Usage method to host/shutdown.go similar to: https://github.com/vmware/govmomi/blob/master/govc/dvs/add.go#L59-L61

@1995parham
Copy link
Contributor Author

Thanks @dougm for your reply, I add usage method for host shutdown command.

@dougm dougm merged commit adf4530 into vmware:master Nov 27, 2017
@dougm
Copy link
Member

dougm commented Nov 27, 2017

@1995parham thanks!

@vmwclabot
Copy link
Member

@1995parham, VMware has rejected your signed contributor license agreement. The change has already been merged, but it will be backed out by the project maintainers if the agreement is not resigned in a timely manner. Click here to resign the agreement.

@vmwclabot
Copy link
Member

@1995parham, VMware has rejected your signed contributor license agreement. The change has already been merged, but it will be backed out by the project maintainers if the agreement is not resigned in a timely manner. Click here to resign the agreement.

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.

3 participants