Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Tenant cli improvement #675

Closed
kerneltime opened this issue Nov 1, 2016 · 4 comments
Closed

Tenant cli improvement #675

kerneltime opened this issue Nov 1, 2016 · 4 comments

Comments

@kerneltime
Copy link
Contributor

CLI should promote name to a first class citizen of the cli.

vmdkops_admin.py tenant create <tenant>
vs
vmdkops_admin.py tenant create --name <tenant>

This is also applicable to rm
cc @lipingxue @msterin

@msterin
Copy link
Contributor

msterin commented Nov 1, 2016

we are currently trying to be consistent with docker volume create --name and related 'docker volume rm ' an IMO should stay such

@kerneltime
Copy link
Contributor Author

Wokie..

@kerneltime
Copy link
Contributor Author

We should then make create and rm symmetric

[root@localhost:~]  /usr/lib/vmware/vmdkops/bin/vmdkops_admin.py tenant rm -h
usage: vmdkops_admin.py tenant rm [-h] [--remove-volumes REMOVE_VOLUMES] name

positional arguments:
  name                  The name of the tenant

optional arguments:
  -h, --help            show this help message and exit
  --remove-volumes REMOVE_VOLUMES
                        BE CAREFUL: Removes this tenant volumes when removing
                        a tenant

@msterin
Copy link
Contributor

msterin commented Nov 2, 2016

Pls add similar docker volume command to the issue , and let's make it consistent

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

3 participants