-
Notifications
You must be signed in to change notification settings - Fork 904
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
govc: Add capability to add and remove standalone host to/from datacenter. #191
Comments
Sounds great @gavingray . Maybe stick with the MO type . method name for the cli names, in this case: The HostSystem.ReconnectHost would be: govc host.reconnect -dc x ... |
I think that reconnect and disconnect are fundamentally different things that what I'm talking about to do here. AddStandaloneHost is a method on the host folder and Destroy is a method on the ComputeResource. Reconnect and Disconnect are used on a HostSystem. Disconnect leaves the host in the inventory and Reconnect assumes that the HostSystem is in the disconnected state. That said, if you'd like to stick closely with the MO paradigms, it would be something more like this.
I think that "addstandalonehost" is pretty wordy though. I think that "addhost" would probably suffice. I have most of the work here done on the govmomi side. Will just need to add tests and the proper govc implementation. Just let me know what you think and I'll get a PR out stat! |
Ah cool, in that case I'd probably go with "datacenter.host.add" or even just "host.add". |
Agreed. I like host.add. Will do! Thanks for the feedback! |
@bitwisekaizen Hi Gavin, did you end up working on this? If not, somebody else can pick it up. |
I actually have a changed shelved for this. I got sidetracked for a while, but I should be able to put it up in the next couple of days. Hoping to get more active in the next few weeks. |
Thanks for the update. |
Doesn't look like we have the ability to add or remove a host to a datacenter from govc or from datacenter.go. Perhaps we could have something like this:
I'm working on this now to support my work on the Terraform vSphere plugin. Would be happy to contribute it back upstream if the usage above looks reasonable to you guys.
The text was updated successfully, but these errors were encountered: