-
Notifications
You must be signed in to change notification settings - Fork 915
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
Add storage profile related api and govc helpers #3466
Conversation
Example of using % govc vm.create -place -net "VM Network" -cluster $cluster -iso library:/govc/ubuntu-14.04.6-server-amd64/*.iso ubuntu
Target: /test-vpx-1717673857-26202-wcp.wcp-sanity/host/test-vpx-1717673857-26202-wcp.wcp-sanity-cluster
Datastore: /test-vpx-1717673857-26202-wcp.wcp-sanity/datastore/sharedVmfs-0
Pool: /test-vpx-1717673857-26202-wcp.wcp-sanity/host/test-vpx-1717673857-26202-wcp.wcp-sanity-cluster/Resources
Host: /test-vpx-1717673857-26202-wcp.wcp-sanity/host/test-vpx-1717673857-26202-wcp.wcp-sanity-cluster/10.182.172.85 Note that adding |
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.
LGTM with a minor comment left. Thanks again for adding these supports in both api and govc!
- add pbm.ProfileMap for finding storage profiles by name
…mmands - The '-cluster', '-library' and '-storage' flags can be specified by name, rather than ID - The namespace.info command outputs cluster, library and storage by name, rather than ID
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.
LGTM!
api: add pbm.DatastoreMap for finding compatible Datastores
govc: add vm.create '-profile' and '-place' flags
api: add storage_specs field to Namespace related structures
govc: add '-storage' flag to namespace.create and namespace.update commands