summaryrefslogtreecommitdiff
path: root/cinderclient
Commit message (Collapse)AuthorAgeFilesLines
* Revert "Update cinderclient to skip the additional GET during create"1.0.8Mike Perez2014-02-041-0/+1
| | | | | | | | | This reverts commit 47014971b09f0334e5cc3b414bef31784f88a37d. Conflicts: doc/source/index.rst Change-Id: Ica1e526b2c181d1ce72183b7293d21119371c822
* Fixed image_name from image-name in upload-to-imageAndrew Kerr2014-01-311-1/+1
| | | | | | | | | This fix addresses an issue where the variable name for image_name was incorrect, causing the image_name parameter to be sent as null when using "cinder upload-to-image" from cli with v2 of the API Change-Id: I62f8bbb4397526fb291a46777d9025318da8e5b3 Closes-Bug: 1274552
* Merge "Sync up with oslo-incubator"Jenkins2014-01-3011-265/+383
|\
| * Sync up with oslo-incubatorSwapnil Kulkarni2014-01-2611-265/+383
| | | | | | | | | | | | | | | | | | | | | | | | | | | | be81d6b Cleanup unused log related code e53fe85 strutils bool_from_string, allow specified default 12bcdb7 Remove vim header 12d3bbc Add method quote_plus in module py3kcompat.urlutils afdbc0a Fix E501 in individual openstack projects Oslo Version: 8dee4b6 Merge "remove extra newlines that eventlet seems to add" Thu, 23 Jan 2014 14:26:42 +0530 (08:56 +0000) Change-Id: I7348e797a4a78ec20dff77ebdbb41bb8c6068b34
* | Add shell tests for snapshot_deleteEric Harney2014-01-274-0/+14
| | | | | | | | | | | | | | | | snapshot_delete does not have a shell test covering it. Closes-Bug: #1257747 Change-Id: I22252f5025c11a7c115aa75e07c78424eec75ba1
* | Merge "Remove RAX-specific auth in cinderclient"Jenkins2014-01-271-15/+1
|\ \ | |/ |/|
| * Remove RAX-specific auth in cinderclientSwapnil Kulkarni2014-01-251-15/+1
| | | | | | | | | | Change-Id: Idd36694a322f467ba5e8c1f58bfac701bff8f3f8 Closes-Bug: #966329
* | Merge "Add volume retype command"Jenkins2014-01-265-5/+50
|\ \
| * | Add volume retype commandAvishay Traeger2014-01-255-5/+50
| | | | | | | | | | | | | | | | | | | | | Add volume retype to Cinder client. Cinder code: https://review.openstack.org/#/c/44881/ Change-Id: Ied4902da531386f744403962e4a8ddfb7587c4ef
* | | Merge "Unit tests for limits.py in cinderclient/v1 and v2"Jenkins2014-01-252-4/+284
|\ \ \ | |/ / |/| |
| * | Unit tests for limits.py in cinderclient/v1 and v2Igor A. Lukyanenkov2014-01-132-4/+284
| | | | | | | | | | | | Change-Id: I962161ac657d9ec676f3b03ab17138ed1911ec35
* | | Merge "disable/enable a service has no output"Jenkins2014-01-2510-16/+56
|\ \ \
| * | | disable/enable a service has no outputJay Lau2014-01-2110-16/+56
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When disable/enable a cinder service, there is no output, this caused end user does not know if the service was disabled/enabled successfully. It is better add some output for the command just like nova to tell end user the service status. Change-Id: I9d440ab7437c268b17627e255f152db0b3bf4f53 Closes-Bug: #1261120
* | | Merge "Fix 'search_opts' unexpected keyword argument error"Jenkins2014-01-254-4/+4
|\ \ \ | |_|/ |/| |
| * | Fix 'search_opts' unexpected keyword argument errorXiao Chen2014-01-204-4/+4
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When executing 'encryption-type-create' command using resource name, it will use the findall() in base.py, then error will occur because list() in findall() used 'search_opts' argument which list() in VolumeEncryptionTypeManager did not have. Some other command like 'transfer-delete' met same issue. This patch will add 'search_opts' argument in list() function in child class to avoid the bug. Closes-Bug: #1248519 Related-Bug: #1252665 Change-Id: If1cd8b812af21335e5d4351766539a927a8fbdf4
* | Merge "Add encryption-type-delete to cinderclient"Jenkins2014-01-2410-6/+58
|\ \
| * | Add encryption-type-delete to cinderclientPeter Hamilton2013-11-2010-6/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This modification adds support for deleting volume encryption types. The encryption-type-delete command accepts a single argument, the name or ID of the volume type whose associated encryption type should be deleted. The encryption type may only be deleted if the associated volume type is unused. blueprint encrypt-cinder-volumes Change-Id: Ie189e7ec91f5a8adc9d3496911de2811ad40f282
* | | Remove copyright from empty filesAlexander Ignatov2014-01-163-44/+0
| |/ |/| | | | | | | | | | | | | | | According to policy change in HACKING: http://docs.openstack.org/developer/hacking/#openstack-licensing empty files should no longer contain copyright notices. Change-Id: I357adea1d295cdc11a12194ad7116d73a8560122 Closes-Bug: #1262424
* | Update cinderclient to skip the additional GET during createSwapnil Kulkarni2014-01-081-1/+0
| | | | | | | | | | | | | | | | | | With change https://review.openstack.org/64945 cinder V2 API now returns volume details. Updated do_create in cinderclient to remove additional GET call. Change-Id: I2a36f2fa865b52f228bad44e0a9c67d49228321e Partial-Bug: #1265893
* | Fix RateLimit.__repr__ - self.method is undefinedIgor A. Lukyanenkov2013-12-314-2/+50
| | | | | | | | | | | | | | | | Using self.verb instead of self.method, unit tests added to cover this fix. Closes-bug: #1258489 Change-Id: I4a84179ff96c50b230afcaea7a19fb54613577fb
* | Fix inappropriate comment for volume update apiZhenguo Niu2013-12-271-1/+1
| | | | | | | | | | Change-Id: Id7a436cc864772bd2662fce0ef80d647c9e3dcf0 Closes-Bug: #1264415
* | Merge "Fix broken argument name in v2 volume extend routine"Jenkins2013-12-121-1/+1
|\ \
| * | Fix broken argument name in v2 volume extend routineJon Bernard2013-12-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new_size attribute was misspelled, causing the volume extend operation to fail with: ERROR: 'Namespace' object has no attribute 'new_size' This happens only with the v2 API, v1 works fine. Change-Id: Icb6b5008dad229f21e7906d9b5342d41f5e760cd Closes-Bug: #1259552
* | | Reset-state and snapshot-reset-state for multiple objectsEric Harney2013-12-046-12/+124
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows a user to call reset-state or snapshot-reset-state on a list of objects. The behavior is modeled after a similar change to delete multiple volumes. $ cinder reset-state good_volume good_volume_2 <no output> Error behavior is as follows: One success/one failure: $ cinder reset-state good_volume asdf Reset state for volume asdf failed: No volume with a name or ID of 'asdf' exists. One failure: $ cinder reset-state asdf ERROR: Reset state for volume asdf failed: No volume with a name or ID of 'asdf' exists. Two failures: $ cinder reset-state asdf qwert Reset state for volume asdf failed: No volume with a name or ID of 'asdf' exists. Reset state for volume qwert failed: No volume with a name or ID of 'qwert' exists. ERROR: Unable to reset the state for any of the specified volumes. Related-Bug: 1241941 Closes-Bug: 1256069 Change-Id: Id0a36fb7de0d69be0dac98ea04e4708775250b7a
* | Add search_opts into the method list() for VolumeTypeManagerVincent Hou2013-12-032-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Since the method base.ManagerWithFind.findall has been modified by adding the search_opts for all tenants support, the parameter search_opts should also be added to all the manager classes, which can be called whithin this method. volumetypemanager does not have this parameter for its method list(). Adding search_opts will resolve this issue. Closes-Bug: #1252665 Change-Id: I526500625f1b5483cb5d88ea15e6ac8485a66ae3
* | Merge "Add assert to delete multiple test"Jenkins2013-12-033-2/+2
|\ \
| * | Add assert to delete multiple testAvishay Traeger2013-11-283-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Make sure calls are made to delete both volumes. Also removed redundant fields in v1 fake volume. Closes-Bug: #1253142 Change-Id: I51078041c6d765f28edb155d44928286ee2b0dd6
* | | Merge "Fix string representation of VolumeBackupsRestore"Jenkins2013-11-302-2/+2
|\ \ \ | |/ / |/| |
| * | Fix string representation of VolumeBackupsRestoreStephen Mulcahy2013-11-272-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Change string representation of VolumeBackupsRestore to use volume_id rather than non-existent VolumeBackupsRestore id. Change-Id: I7b585d0de9b3b0511e7fd1a70b214090a17dfa93 Closes-Bug: #1255488
* | | Fix typo in cinderclienthuangtianhua2013-11-283-4/+4
|/ / | | | | | | | | | | | | | | | | sematics --> semantics hypen-separated --> hyphen-separated typicaly --> typically Change-Id: I5df277ef7036082d0e4b079c23d41da809e5270f Closes-Bug: #1254587
* | Fix inappropriate comment for set_metadatawanghao2013-11-262-2/+2
|/ | | | | | | Fix inappropriate comment for the param of set_metadata Change-Id: I01495f59b11ed88389b0e0f2f5fb34289f64dbac Closes-Bug: 1254951
* change assertEquals to assertEqualChristian Berendt2013-11-152-17/+17
| | | | | | | | | | | | | According to http://docs.python.org/2/library/unittest.html assertEquals is a deprecated alias of assertEqual. https://review.openstack.org/#/c/56652/ updates to a new version of hacking that enforces H602. Turning that on requires this change to land first. Re-do of https://review.openstack.org/#/c/53510 since that is abandoned. Change-Id: Ie621edc8770e2015df5712f77d4cf9679b02bd0f
* Merge "Enable del of other tenants resources by name"Jenkins2013-10-304-2/+15
|\
| * Enable del of other tenants resources by nameJay S. Bryant2013-10-294-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, due to the way that resources are being retrieved by the findall() function, an administrator can do a list, snapshot-list, etc. with the --all_tenants option and see other tenants' resources. If the admin then tries to delete the another tenants' resource by name, it fails with a 'No <resource> with a name or ID of <name> exists.' error. The solution to this is to change the call to the list() function in findall() to set the all_tenants search option to 1. This causes the admin to get a list of all the resources that they have access to back when the search is done instead of just a list of their resources. The delete by name is then possible. The server takes care of ensuring that only resources that the user has access to are returned. This will enable delete by name for all resources that use the find_resource function. Closes-bug: 1241682 Change-Id: I4e9957b66c11b7e1081f066d189cedc5a3cb2a6c
* | Fix py33 due to readonly and metadata patchesAvishay Traeger2013-10-304-10/+10
|/ | | | | | A couple recent merges broke py33 - this fixes it. Change-Id: Iaab8a08aeb42c954f3ecff7da70003d1c4eead0d
* Merge "Adding volume readonly-mode-update interface to Cinder client"Jenkins2013-10-2910-2/+89
|\
| * Adding volume readonly-mode-update interface to Cinder clientZhi Yan Liu2013-10-2410-2/+89
| | | | | | | | | | | | | | | | | | | | | | Adding volume readonly-mode-update interface to shell and client object (v1 and v2) to allow end user update volume read-only access mode flag. blueprint read-only-volumes Related-Id: I4c84614d6541d5f7c358abadb957da7b8c3d9c48 Change-Id: I2d587f301a21c0d42b8940465862da25a70ed5a8 Signed-off-by: Zhi Yan Liu <zhiyanl@cn.ibm.com>
* | Adding Read-Only volume attaching support to Cinder clientZhi Yan Liu2013-10-246-10/+18
|/ | | | | | | | | | Adding mode argument to volume manager attach function. Allow Nova using read-write or read-only mode to attach a volume to an instance. blueprint read-only-volumes Change-Id: Ib20c7802304d83f4aafef522171ebd47f672d883 Signed-off-by: Zhi Yan Liu <zhiyanl@cn.ibm.com>
* Merge "Addition of volume/snapshot_metadata CLI"Jenkins2013-10-2712-2/+360
|\
| * Addition of volume/snapshot_metadata CLIsaurabh2013-10-1512-2/+360
| | | | | | | | | | | | | | | | | | | | | | | | | | Added below CLIs:- 1. Set or Delete metadata of a snapshot 2. Show metadata of a snapshot 3. Show metadata of a volume 4. Update all metadata of volume 5. Update all metadata of snapshot Implements blueprint: add-metadata-cli Change-Id: Ic2b5f3fce6104d1756879718f666a42549458ad3
* | Enable "cinder delete" can delete multiple volumes in one requestJay Lau2013-10-256-24/+78
| | | | | | | | | | | | | | | | | | | | "nova delete" can delete multiple servers in one request but "cinder delete" can only delete one volume in one request, it is better to enhance cinder client to support remove multiple volumes in one request. Change-Id: I6a63aa3d7e4c152ae5e45bf2b36d862bdcd6df33 Closes-Bug: #1241941
* | Merge "python3: sort dict for post_volumes_1234_action test"Jenkins2013-10-232-2/+4
|\ \
| * | python3: sort dict for post_volumes_1234_action testChuck Short2013-10-132-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Python 3, hash randomization is enabled by default. It causes the iteration order of dicts and sets to be unpredictable and differ across Python runs. Sort the list and update the expecting result. Change-Id: Iead7ca8f45b035abd78f0b25865b5f7e1b9f3294 Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | Merge "python3: align the order of parameters for urlencode()"Jenkins2013-10-232-2/+14
|\ \ \
| * | | python3: align the order of parameters for urlencode()Chuck Short2013-10-132-2/+14
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Python 3.3, hash randomization is enabled by default. It causes the iteration order of dicts and sets to be unpredictable and differ across Python runs. In the test case, the fixed expecting string will not match the test result, it is relying on the dict order. This change transforms the input dict to a sequence of two-element list, with fixed order, and update the related expecitng string in test case. Change-Id: I60d7bb3c4f940b76460ad5c417a1807915e0418e Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | Deprecates --volume-id arg for v2 backup-restoreEdward Hope-Morley2013-10-231-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch deprecates the --volume-id arg of backup-restore when using the v2 api and adds (preferred) --volume arg. Both args are still usable but the former is now listed as "Deprecated" in the help message. Change-Id: I2575d93041b26f971c59bc04828b356556617b62 Fixes: bug 1242816
* | | Merge "Fixes broken v1 and v2 api backup-restore"Jenkins2013-10-226-4/+28
|\ \ \
| * | | Fixes broken v1 and v2 api backup-restoreEdward Hope-Morley2013-10-156-4/+28
| | |/ | |/| | | | | | | | | | | | | | | | Also adds unit tests. Change-Id: I741889d49cffe46bf9165b266a62f2b328354e01 Fixes: bug 1240151
* | | Merge "Add quota-usage command"Jenkins2013-10-217-7/+64
|\ \ \
| * | | Add quota-usage commandSeif Lotfy2013-10-037-7/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a new command quota-usage to allow querying the usage of the quota for a given tenant Change-Id: I720e6df94f9fedbb8b6385bd1707600542aaea08