summaryrefslogtreecommitdiff
path: root/cinderclient
Commit message (Collapse)AuthorAgeFilesLines
* Raise API max version for Rocky updates4.0.2Sean McGinnis2019-05-081-1/+1
| | | | | | | | | | | | | | | | | | | Functionality was added during rocky, but the MAX_VERSION of the client was not updated for these versions. This raises the version to support the added functionality. 3.53 - Schema validation - no client changes 3.54 - Add mode option to attachment-create: I22cfddd0192c4a72b8f844f23d1fa51b96c57e06 3.55 - Transfer snapshots with volumes I61a84b5abf386a4073baea57d8820c8fd762ae03 This patch adds a release note to the cherry pick indicated below. Change-Id: I8c8fb8f552e529c8474a3e3b771ba8eb7aed193a Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com> (cherry picked from commit b843a168efc9261a12ebe322fe701ef355474874) (cherry picked from commit ee37f267354a8dfe5d4b9b05c261bf86f329cfb8)
* Allow volume-transfer creation < 3.55 microversion4.0.1Sean McGinnis2018-07-313-63/+24
| | | | | | | | | | | Handling for the new `no_snapshots` option was incorrectly handling microversion evaluation that would prevent anything less than the new microversion from working. This changes the check to only handle the changed argument for 3.55 and later. Change-Id: If96889ccde6044706e6a5dcd83fde3c20fe1c1fd Closes-bug: #1784703 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* Merge "Reflect multiattach deprecation in help text"4.0.0Zuul2018-07-272-2/+2
|\
| * Reflect multiattach deprecation in help textSean McGinnis2018-07-262-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Change Icbb9c0ca89b25620cedff6cac7a4723e7126eca6 notified that the ``multiattach`` argument was deprecated, but nothing was added to the args help text to indicate this, relying on a user to have read the release notes. In preparation of removing this option, this updates the help text so there is at least some indication that it is going away. Change-Id: I9e767a3f1411fbfc0bf0e433b45560e451d547d5 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Remove replication v1 supportSean McGinnis2018-07-264-15/+3
| | | | | | | | | | | | | | | | | | | | | | | | The replication v1 implementation in Cinder was deprecated in the Mitaka release in favor of the v2 "Cheesecake" version. Support was kept in the client for backwards compatibility, but it has now been several releases and these options should be removed. Closes-bug: #1705470 Change-Id: I978a39a552fffc9ac7ba6e4726d1df2072fa45ba Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Remove unnecessary parameters from volume create APIsNeha Alhat2018-07-266-70/+9
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | As per Cinder code, following parameters are not required to be passed in the request body of create volume API. * status * user_id * attach_status * project_id * source_replica If you pass these parameters, previously it was ignored but in the schema validation changes[1] we don't allow additionalProperties to be passed in the request body. If user passes additional parameters which are not as per API specs[2], then it will be rejected with 400 error. On patch[3], tempest tests: test_volume_snapshot_create_get_list_delete, test_volume_create_get_delete" are failing because of these unnecessary parameters. This patch removes these unnecessary parameters passed to the create Volume API. [1]https://blueprints.launchpad.net/cinder/+spec/json-schema-validation [2]https://review.openstack.org/#/c/507386/ [3]https://review.openstack.org/#/c/573093/ Change-Id: I37744bfd0b0bc59682c3e680c1200f608ad3991b
* Merge "Remove deprecated CLI options"Zuul2018-07-241-28/+0
|\
| * Remove deprecated CLI optionsSean McGinnis2018-07-231-28/+0
| | | | | | | | | | | | | | | | | | | | | | | | The following options were deprecated in Queens or Pike and are now being removed: --endpoint-type --bypass-url --os-auth-system Change-Id: I3b951cc4eb3adff23f3d2cbe674971816261ef56 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Update pylint to work with python 3Sean McGinnis2018-07-231-1/+1
|/ | | | | | | | | | | | | | | | The pylint job was switched over to run under python 3, but the job is not voting and it was apparently missed that the conversion was causing it to fail. This updates the version of pylint to one that is actually supported by python 3 and makes tweaks to our script to for the minor changes between versions. Single character change to get rid of the more strict py3 regex string escape character format. Change-Id: I93124b62c5ee177815457b32f55f5453fc3d387e Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* Merge "Transfer snapshots with volumes"Zuul2018-07-205-2/+218
|\
| * Transfer snapshots with volumeswanghao2018-07-195-2/+218
| | | | | | | | | | | | | | | | | | This patch will support to transfer volumes with or without snapshots in new V3 api after mircoversion 3.55. Change-Id: I61a84b5abf386a4073baea57d8820c8fd762ae03 Depends-On: https://review.openstack.org/533564/ Implements: blueprint transfer-snps-with-vols
* | Merge "fix tox python3 overrides"Zuul2018-07-191-1/+1
|\ \
| * | fix tox python3 overrideshuang.zhiping2018-07-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to default to running all tox environments under python 3, so set the basepython value in each environment. We do not want to specify a minor version number, because we do not want to have to update the file every time we upgrade python. We do not want to set the override once in testenv, because that breaks the more specific versions used in default environments like py35 and py36. Change-Id: I04b87b308a82113a57ea41bde0329e92b07b642e
* | | Add mode option to attachment-createJohn Griffith2018-07-174-3/+60
| |/ |/| | | | | | | | | | | | | | | | | | | | | We introduce a mode option in attachment-create starting with microversion 3.54. The mode option is the new/preferred way of setting ro mode (as opposed to admin-metadata). This patch adds the support for the mode option to the client for endpoints that support 3.54. Depends-on: https://review.openstack.org/532702/ Change-Id: I22cfddd0192c4a72b8f844f23d1fa51b96c57e06
* | Merge "Remove initialization of logger if logger is None"Zuul2018-07-031-5/+0
|\ \
| * | Remove initialization of logger if logger is NoneNeha Alhat2018-06-261-5/+0
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If default_log_level=keystoneauth=WARN is enabled in nova.conf, then these log messages [1][2][3] are logged using cinderclient.v3.client logger name instead of keystoneauth.session. This is because cinderclient initializes logger if not passed by nova and since debug=True is set in nova.conf it’s uses root logger log level. This patch removes initialization of logger code to ensure log messages[1][2][3] are logged only when keystoneauth=DEBUG is enabled in nova.conf. This will also enable us to consume split_logger config option [4] when enabled in nova without making any additional changes in client code. [1] REQ: https://review.openstack.org/#/c/505764/8/keystoneauth1/session.py@391 [2] RESP: https://review.openstack.org/#/c/505764/8/keystoneauth1/session.py@422 [3] RESP BODY: https://review.openstack.org/#/c/505764/8/keystoneauth1/session.py@454 [4] https://review.openstack.org/#/c/568878/ Change-Id: I937c6ac2f6e254c438ee2f36eb6c291f62c0f411
* | Merge "Use api version 3 for functional test"Zuul2018-06-252-4/+11
|\ \
| * | Use api version 3 for functional testjunboli2018-06-192-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | switch to use OS_VOLUME_API_VERSION = 3 to run functional tests. Closes-bug: #1775354 Co-Authored-By: liuyamin <liuyamin@fiberhome.com> Change-Id: If6c4291aa7d8e85a4f8c1d5665a0e4a72aa1f1d1
* | | Keep is_public usage backwards compatible3.6.1Sean McGinnis2018-06-223-12/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5a1513244caf7acbd41e181419bc8b62bf4bcaba added the ability to filter by AZ, but it changed the existing behavior if is_public was not specified. This adds handling to make sure we are still consistent with the previous behavior. Co-Authored-by: Alan Bishop <abishop@redhat.com> Change-Id: I5000aab092c1b434c8dc17bbe4b2d3d632f528c3 Closes-bug: #1778055
* | | unable to create group from src in cinderclientwanghao2018-06-203-9/+20
|/ / | | | | | | | | | | | | | | | | | | | | According the api schema of cinder, create_group_from_src should only specify one of arguments from [group_snapshot_id, source_group_id]. Now cinderclient specified them both even if one of them is None. This patch fix this issue to just pass one argument. Change-Id: Idef51ab9a1452dd5eb3be4d4b6dca095a777d611 Closes-Bug: #1777555
* | Merge "Fix failing functional test cases"Zuul2018-06-181-2/+4
|\ \
| * | Fix failing functional test casesPooja Jadhav2018-06-131-2/+4
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | After applying schema validation to volume actions APIs[1], error messages are formatted as per the standards. So the tests are failing because of error messages. This patch fixes failed functional tests by correcting error messages. [1]https://review.openstack.org/#/c/559042/ Change-Id: Id05bf637d77ab401fc0aec5be09eee423bf1a223
* | Merge "Unreadable output of upload-to-image."Zuul2018-06-151-0/+4
|\ \ | |/ |/|
| * Unreadable output of upload-to-image.deepak_mourya2018-05-281-0/+4
| | | | | | | | | | | | | | | | | | The output of upload-to-image is not readable due to volume type is printed in output as a whole. Printing volume type name only cleans up the output. Change-Id: Ia588cf7ccc0873a8cf956c551afcccf4b6ddaa39 Closes-Bug: #1557486
* | Merge "Add the parameter service-id for service cleanup command"Zuul2018-05-282-3/+12
|\ \
| * | Add the parameter service-id for service cleanup commandliuyamin2018-05-252-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | There isn't service_id param in cinderclient. We have this param in cinder, so add this param in the cinderclient. Change-Id: I579741595f91802bcf117029889567f234b5b14c Closes-bug: #1771721
* | | Remove useless args in create_group_from_srcwanghao2018-05-223-13/+1
| |/ |/| | | | | | | | | | | | | | | | | | | Remove status, project_id and user_id args from create_group_from_src. Those args will cause error when using cinderclient since schema validation in Cinder side. Change-Id: Iba9a45625f9e6423e8e582425605ca184238cde9 Closes-Bug: #1772375
* | Merge "Support availability-zone in volume type"Zuul2018-05-204-5/+44
|\ \ | |/ |/|
| * Support availability-zone in volume typeTommyLike2018-05-024-5/+44
| | | | | | | | | | | | | | | | | | | | Since 3.52, new option '--filters' has been added to 'type-list' command, and it's only valid for administrator. Change-Id: I140f6d61a2747d4fcaabfbccea864dcc7eb841d1 Depends-On: I4e6aa7af707bd063e7edf2b0bf28e3071ad5c67a Partial-Implements: bp support-az-in-volumetype
* | Allow --help for specific commandsEric Harney2018-04-102-0/+17
|/ | | | | | | | | | | | This makes calls such as: $ cinder list --help work as expected, equivalent to: $ cinder help list rather than just printing the full help message. Change-Id: I038eeeea554f0bb5dd170f503eaec570441520b3
* Merge "fix a typo in documentation"Zuul2018-04-021-1/+1
|\
| * fix a typo in documentationmelissaml2018-03-231-1/+1
| | | | | | | | Change-Id: Icb6c9808829c6088f6ae2408718bcb03ddc7b53a
* | Merge "Correct errors in snapshot-manageable-list help text"Zuul2018-03-271-10/+11
|\ \
| * | Correct errors in snapshot-manageable-list help textSean McGinnis2018-03-201-10/+11
| |/ | | | | | | | | | | | | | | Fixes some copy paste issues with the help text for listing snapshots. Also minor grammar fixes. Closes-bug: #1756358 Change-Id: Idee5e8400d8e7d55e9f9232df55b0dfec3e36c3b
* | Merge "Support cross AZ backups"Zuul2018-03-273-28/+71
|\ \ | |/ |/|
| * Support cross AZ backupsGorka Eguileor2018-02-123-28/+71
| | | | | | | | | | | | | | | | | | | | | | Add support for microversion 3.51 that allows us to specify the Availability Zone of the backup service that should create the backup. New optional parameter available for "backup-create" when using microversion 3.51 is "--availability-zone". Depends-On: I595932276088d25abd464025c99dce33a2cc502b Change-Id: Ia4bab5fcb95f6a8d1adb99c5d7dc1b3f7546c6f6
* | Add api_version wraps for group snapshot list in v3/shell.pyjeremy.zhang2018-03-121-0/+1
| | | | | | | | | | | | | | | | The API 'do_group_snapshot_list' in v3/shell.py misses api_version wraps, and the cmd 'cinder group-snapshot-list' can be used even not specifing the param 'os-volume-api-version'. This patch just to fix it. Change-Id: I6a46e587b1aab692fe6b81b46fa3d95ab66c0625
* | Remove unused cinderclient/apiclient/client.py moduleMatt Riedemann2018-02-201-368/+0
| | | | | | | | | | | | | | | | | | This old module was a carry over from the oslo incubator days and is no longer used. Change-Id: I44982d2581e90b781c78f3d2421cd1dcd8e590fd Related-Bug: #1685678 Related-Bug: #1640269
* | Update help text for encryption providerBrianna Poulos2018-02-161-3/+3
|/ | | | | | | | | The volume encryption provider no longer uses class names. Instead, 'luks' and 'plain' are used. This patch updates the help text for the volume encryption provider to use the new encryption provider format constants. Change-Id: I6072e18f8c1945082f421a3bf725a874565d6f80
* Merge "Add api_version wraps for some v3 volume APIs"Zuul2018-01-314-0/+13
|\
| * Add api_version wraps for some v3 volume APIsjeremy.zhang2018-01-254-0/+13
| | | | | | | | | | | | This patch is mainly to add api_version wraps for some v3 volume APIs. Change-Id: Iea8b9b68930548156749da3f98629e3602ffca83
* | Fix for v3 volume unit testsjeremy.zhang2018-01-256-33/+47
|/ | | | | | | This patch is mainly to specify the APIversion for FakeClient used in different volume resources' unit tests. Change-Id: I18eb9b57434061e95c909907e6ba263b9091a446
* Merge "Add snapshot_id param note for backup-create"queens-em3.5.0Zuul2018-01-232-0/+9
|\
| * Add snapshot_id param note for backup-createzhengyin2018-01-112-0/+9
| | | | | | | | Change-Id: Id7c565629aeed6d915b21b8de514afcee42470cf
* | Merge "Bump API microversion to 3.50"Zuul2018-01-231-1/+1
|\ \
| * | Bump API microversion to 3.50Jay S. Bryant2018-01-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change is to bump MAX_VERSION to 3.50 in api_version. This bump is necessary to keep cinderclient in sync with cinder which moved to 3.50 with change: f1bfd9790d2a7cac9a3e66417b11dc8e3edd8109 With MV 3.50 we are able to request multiple attachments for a single volume. Change-Id: Ic51b890b816112cecaedd68df8d13ec175623c66
* | | Removes unicode 'u' response for "cinder get-capabilities"Abijitha Nadagouda2018-01-232-4/+5
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The output of "cinder get-capabilities" command returns unicoded response. But it would appear from the utils class that setting formatters will go through the capabilities dict and make sure all values are properly string formatted. Therefore added formatters to return string formatted response. Added formatters=sorted(prop.keys()) line instead of static values as suggested by the reviewer, to avoid tying server knowledge to the client and also any update on the server side would easily reflect here. Closes-bug: #1680444 Change-Id: Ie38236db364d59ddab42cb925d0435777b0ffe86
* | Merge "Support for reporting backend state in service list"Zuul2018-01-184-1/+24
|\ \
| * | Support for reporting backend state in service listwanghao2018-01-094-1/+24
| |/ | | | | | | | | | | | | | | | | | | This patch will support the feature: report backend state in service list in client side. Depends-On: I561dca3ef7c1901401621bc112389dbd178a907e Change-Id: If15e1fa50b5feecd74c7394c918f4fc9d87bcf3e Implements: blueprint report-backend-state-in-service-list
* | Merge "Add api_version wraps for generic volume groups"Zuul2018-01-181-0/+10
|\ \
View Lorry Controller Status