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

Tenancy: Tenant rm command needs updates #688

Closed
kerneltime opened this issue Nov 3, 2016 · 0 comments
Closed

Tenancy: Tenant rm command needs updates #688

kerneltime opened this issue Nov 3, 2016 · 0 comments

Comments

@kerneltime
Copy link
Contributor

  1. The command I typed below listing volumes to be deleted should not work.
  2. The help needs updating.
[root@localhost:~] 
[root@localhost:~] /usr/lib/vmware/vmdkops/bin/vmdkops_admin.py tenant rm tenant1 --remove-volumes storage3,storage2,storage4
tenant rm succeeded
[root@localhost:~] ls /vmfs/volumes/datastore1/dockvols/tenant1/storage2
storage2-9a71d73b5c60ee6e.vmfd  storage2-flat.vmdk              storage2.vmdk
[root@localhost:~] ls /vmfs/volumes/vsanDatastore/dockvols/tenant1/
.0c558557-03e0-3b0c-6cc4-000c29acb565.lck  storage3-95f889e1f5f5f2b0.vmfd             storage4-c9f772672565851f.vmfd
.17588557-d9db-6d1e-9a44-000c29acb565.lck  storage3.vmdk                              storage4.vmdk
[root@localhost:~] /usr/lib/vmware/vmdkops/bin/vmdkops_admin.py tenant ls
Uuid                                  Name     Description  Default_datastore  VM_list  
------------------------------------  -------  -----------  -----------------  -------  
5afa4767-ecd8-489e-9d97-5c46ae50d3fc  DevAppX               default_ds         ubuntu   

[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
[root@localhost:~] 
@pdhamdhere pdhamdhere modified the milestone: 0.9 Nov 4, 2016
pdhamdhere added a commit that referenced this issue Nov 8, 2016
* Minor bug fixes in multi-tenancy
* remove extra newline
* No auth check needed for list command
* No auth check needed for GET command
* Admin CLI usability improvements
* Address Liping's review comments and also fixed  #675 & #688
* Addressed review comments (nits)
* Fix CLI unit tests
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

2 participants