govc: Add capability to add and remove standalone host to/from datacenter. about govmomi HOT 8 CLOSED

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 ...
And HostSystem.DisconnectHost: govc host.disconnect -dc x ...

from govmomi.

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.

$ govc datacenter.addstandalonehost -dc=somedatacenter -hostname=somehost ...
$ govc host.destroy -dc=somedatacenter -hostname=something

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!

from govmomi.

Ah cool, in that case I'd probably go with "datacenter.host.add" or even just "host.add".

from govmomi.

Agreed. I like host.add. Will do! Thanks for the feedback!

from govmomi.

@bitwisekaizen Hi Gavin, did you end up working on this? If not, somebody else can pick it up.

from govmomi.

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.

from govmomi.

Thanks for the update.

from govmomi.

We have added both 'host.add' (e01555f) and 'cluster.add' (5154339) commands to govc.

from govmomi.