summaryrefslogtreecommitdiff
path: root/cinderclient
Commit message (Collapse)AuthorAgeFilesLines
* Fix typo: dow -> downLuigi Toscano2020-08-051-1/+1
| | | | | | | Closes-Bug: #1883674 Change-Id: I4f4468b23d04ecf74fb6347bfb518377127b563d (cherry picked from commit 718474a092192ea0034a8107b6947a10636a4a0a) (cherry picked from commit f591104f5e707985f0297ee2edd7fdc1f72e341f)
* Fix: --poll inconsistency5.0.1whoami-rajat2020-02-212-1/+2
| | | | | | | | | | | | | | | | When we use `--poll` parameter with cinder create command, it waits for the volume to become available but doesn't update the data displayed to the user. Due to this, there are inconsistency between several parameters in the output after 'poll' and 'cinder show' command. Eg: cinder create 1 --image <image-id> --poll shows 'bootable' flag as false whereas, cinder show <vol-id> shows 'bootable' as true Change-Id: I1502e88f1cd84d225b75c07313e4eb252cc2d645 Closes-Bug: #1855224 (cherry picked from commit ceddb3cfd033661f63d1245a45e0e197561be19c)
* Merge "Drop support for --sort_key and --sort_dir"5.0.0Zuul2019-09-119-162/+19
|\
| * Drop support for --sort_key and --sort_dirSean McGinnis2019-09-119-162/+19
| | | | | | | | | | | | | | | | | | These arguments were deprecated in the kilo release in favor of a combined --sort argument. This drops support for the deprecated arguments. Change-Id: If8f8ac44cc81f553009a15ca67257e86cb925b6f Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Merge "Drop support for --allow-multiattach"Zuul2019-09-1111-70/+15
|\ \ | |/
| * Drop support for --allow-multiattachSean McGinnis2019-09-1111-70/+15
| | | | | | | | | | | | | | | | | | | | The ability to enable multiattach on the command line was deprecated in Queens with the full implementation of multiattach enabling it through volume type extra specs. This removes the command line arg and handling for specifying it with volume creation. Change-Id: Ifc0c874657f959266050cd1a7a40e6ecccc8c114 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Merge "Optional filters parameters should be passed only once"Zuul2019-09-114-21/+136
|\ \ | |/ |/|
| * Optional filters parameters should be passed only oncewhoami-rajat2019-09-054-21/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch does the following : 1) Errors out if similar type of filters args are passed Eg : cinder list --name abc --name xyz cinder list --name abc --filters name=xyz 2) Allows multiple filter parameters of different type cinder list --filters name=abc --filters size=1 Change-Id: I2f8662555f830b0821147324849d04e7a29d0580
* | Drop support for OS_TENANT_NAME and OS_TENANT_IDSean McGinnis2019-09-102-22/+6
| | | | | | | | | | | | | | | | These have been deprecated for a very long time now. Switch to only support the current OS_PROJECT_NAME and OS_PROJECT_ID. Change-Id: I2fb78768b4308a3aaf12764edadb58d4a307adaa Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Merge "Drop 'endpoints' and 'credentials' commands"Zuul2019-09-082-27/+0
|\ \
| * | Drop 'endpoints' and 'credentials' commandsSean McGinnis2019-04-182-27/+0
| | | | | | | | | | | | | | | | | | | | | | | | We are past the deprecation period for these commands and they should now be removed. Change-Id: I37f0dc539da5d43f629ea726bb603fa995c1fe6f Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | Merge "Drop support for Cinder v1 API"Zuul2019-09-0844-5819/+19
|\ \ \ | |/ /
| * | Drop support for Cinder v1 APISean McGinnis2019-04-1844-5819/+19
| | | | | | | | | | | | | | | | | | | | | This drops compatibility with Cinder services with the V1 API. Change-Id: Ic7c6bd05a3991c3afce3dec80b29501932c5aac9 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | Merge "Add custom CA support for get_server_version"Zuul2019-09-052-4/+50
|\ \ \ | |_|/ |/| |
| * | Add custom CA support for get_server_versionIvan Kolodyazhny2019-08-282-4/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_server_version fails when self-signed CA cert is used. This patch adds: * insecure option to ignore SSL certificate validation * cacert to add ability to provide a custom SSL certificate Change-Id: Ib1d34a5a6b595c53473ddd3acb182ab5a39cbba5 Related-Bug: 1744670
* | | Merge "Autonegotiate API version for shell"Zuul2019-09-043-12/+112
|\ \ \
| * | | Autonegotiate API version for shellEric Harney2019-08-283-12/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If OS_VOLUME_API_VERSION is not set, use the highest supported by both the client and the server. If OS_VOLUME_API_VERSION exceeds that supported by the server, use the highest supported by both the client and the server. A warning message is printed for the user indicating that this happened. (This is similar to the behavior of the manila CLI, and is mostly code from manilaclient tweaked to work in cinderclient.) Change-Id: Ie1403eca2a191f62169e60c0cde1622575327387
* | | | Merge "Migrate the functional job to Zuul v3"Zuul2019-09-031-53/+0
|\ \ \ \
| * | | | Migrate the functional job to Zuul v3Luigi Toscano2019-08-081-53/+0
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - rename it so that it follows the documented pattern (<project>-<specifier>); - remove the old job, including the hooks, hoping that no 3rd-party CI uses them. Change-Id: Ib690c4fa96354dc422576d76c1296b1d5ba3494d
* | | | Merge "Fix: Quota update successfully executes with no params"Zuul2019-09-011-0/+3
|\ \ \ \
| * | | | Fix: Quota update successfully executes with no paramsRajat Dhasmana2019-06-241-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since all params of quota update command are optional (except project_id), when no quota field is supplied the command shouldn't execute successfully. This patch shows an error in this case. Change-Id: I22e5ef7900631d1394e0ab5b57c4e4444f0d5a50 Closes-Bug: #1778975
* | | | | Merge "Add transfer-list --sort argument"Zuul2019-08-304-13/+82
|\ \ \ \ \
| * | | | | Add transfer-list --sort argumentSean McGinnis2019-05-044-13/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change Idb04f783b2287b2b45d626131648b0005a232fbe to the cinder service introduced pagination and the ability to sort results for listing volume transfers. This adds the sort ability to the cinder client. The service side uses the sort_key and sort_dir that we've deprecated long ago, but was unfortunately missed when merging this support. Since we have been giving deprecation warnings for using those for other operations, this adds support for --sort that internally will convert to the API's expected sort_key and sort_dir. Change-Id: I137436c76852cbb974eee87e49712c698cbf081b Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | | | Flag safe usage of sha1 w/ #nosecEric Harney2019-08-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This tells bandit that this usage of sha1 is not interesting for security scans. Change-Id: Ibc0f75d9eef04c919410cafb1a1713d6470f8142
* | | | | | Merge "Add 'is_public' support in '--filters' option"Zuul2019-08-214-5/+48
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Add 'is_public' support in '--filters' optionwhoami-rajat2019-04-154-5/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '--is-public' is a valid argument for cinder type-list command[1] and cinder group-type-list command but is missing in cinderclient. This patch adds the support for it. [1] https://developer.openstack.org/api-ref/block-storage/v3/?expanded=list-all-volume-types-detail#list-all-volume-types Change-Id: I8af9bb06a28f3cc384c4925b8b52bdeaed52cb15
* | | | | | Merge "Remove promote/reenable replication"Zuul2019-07-105-71/+0
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Remove promote/reenable replicationMinmin Ren2019-06-195-71/+0
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The replication 1.0 API os-promote-replica and os-reenable-replica were removed[1], then remove useless code from cinderclient. [1]: https://review.openstack.org/#/c/275797/ Change-Id: Icf98f294c089942e11607786e932abc959f77b3b
* | | | | Add missed 'Server ID' output in attachment-listMinmin Ren2019-06-043-9/+31
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'server_id' is not Attachment attribute, should be set by 'instance' attribute. v3/attachments respond body: {"attachments": [{"status": "attached", "instance": INSTANCE_UUID, "id": ATTACHMENT_UUID, "volume_id": VOLUME_UUID, }, ... ] } Change-Id: Ica5d278cb7455befe1db4be0ab65114fd606ea0a
* | | | Merge "Add bash completion for groups"Zuul2019-06-031-0/+25
|\ \ \ \
| * | | | Add bash completion for groupsEric Harney2019-03-071-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Completion for group create/list. Change-Id: I0b7027a72a3bc8ad7ca7323c6dbe0a501616cdd5
* | | | | Merge "Fix shell upload-to-image with no volume type"Zuul2019-05-235-18/+51
|\ \ \ \ \
| * | | | | Fix shell upload-to-image with no volume typeEric Harney2019-04-035-18/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upload-to-image would error after launching the operation if the volume type is None. Closes-Bug: #1821818 Change-Id: I015e0ddfa98d62f25334e2df5effaee72a3988ab
* | | | | | Merge "Correct discover_version response"Zuul2019-05-042-9/+27
|\ \ \ \ \ \
| * | | | | | Correct discover_version responseBrian Rosmaita2019-04-252-9/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The discover_version function was ignoring the max microversion supported by the client. This patch corrects its behavior to return the most recent API version, if any, supported by both the client and the server it is communicating with. Closes-bug: #1826286 Change-Id: If22b72452065080b24cb1b899c5a5a88b809e986
* | | | | | | Merge "Remove bash-completion calls from base.py"Zuul2019-04-261-9/+3
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Remove bash-completion calls from base.pyEric Harney2019-04-021-9/+3
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should not be here since this is client library code, but it isn't needed anyhow since this completion is handled from the shell code. Closes-Bug: #1817782 Change-Id: I3e7ddbe4a50a66db8961a71d71592ce708320b0d
* | | | | | Merge "Handle auth redirects"Zuul2019-04-222-9/+31
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | / | | |_|_|/ | |/| | |
| * | | | Handle auth redirectsSean McGinnis2019-04-182-9/+31
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Our checking for keystone v2 and v3 was causing v2 tests to go down the v1 code path. Fixing this surfaced a couple of long standing issues with our v2/v3 handling of redirects and getting auth info. This fixes our version identification, fixes extracting redirect location for response headers, and returning the auth_url to calling code. Change-Id: I939ff027bf43e513e338bff1e99ca41fa52637b6 Closes-bug: #1825372 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | Raise API max version for Stein updatesSean McGinnis2019-04-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Functionality was added during stein, but the MAX_VERSION of the client was not updated for these versions. This raises the version to support the added functionality. 3.57 - Improve volume transfer records - no client changes 3.58 - Add project_id to group - no client changes 3.59 is not included here as that adds sort and pagination that will need to be slightly reworked before we expose it. Change-Id: If8a3b88b9a2811ae6681e52dbca62b906d3542eb Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | Add support for backup user IDSean McGinnis2019-04-172-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Backup user_id was added with microversion 3.56 but never added to the client. Added in Rocky with I3ffb544ef3ee65276cee8b1e870d524fd0e57085. Change-Id: Ib9a4159db0f8af599dcf8315334de4f859be077e Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | Raise API max version for Rocky updatesSean McGinnis2019-04-171-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 Change-Id: I8c8fb8f552e529c8474a3e3b771ba8eb7aed193a Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | Merge "Tests: Don't write bash-completion cache files"Zuul2019-04-026-0/+33
|\ \ \ | |/ / | | / | |/ |/|
| * Tests: Don't write bash-completion cache filesEric Harney2019-02-266-0/+33
| | | | | | | | | | | | | | | | | | Mock out writing of bash-completion cache files during unit tests. Related-Bug: #1817782 Change-Id: I944862c30fb4684dd034eba6953e9302d2d22439
* | Merge "Fix: cinder group-list not working with non-admin user"Zuul2019-02-252-13/+11
|\ \
| * | Fix: cinder group-list not working with non-admin userwhoami-rajat2019-02-182-13/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The all_tenants filter is passed to API when it's value is 0 (when not specified) which shouldn't be the case. This patch only allows adding of all_tenants when specified manually by user. Change-Id: Ic7810c4d6e9f4be7c28c7a8778d57bb5ccb996a0 Closes-Bug: #1808621 Closes-Bug: #1808622
* | | Merge "Change bash completion dir permissions to 0750"Zuul2019-02-151-1/+1
|\ \ \ | |_|/ |/| |
| * | Change bash completion dir permissions to 0750Eric Harney2018-12-051-1/+1
| | | | | | | | | | | | | | | | | | This is no reason for this dir to be world-readable. Change-Id: I50e85b5bb6116c64535ecbf09718141086c703c5
* | | Fix max version handling for help outputSean McGinnis2019-01-302-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit fefe331f218d73ba6d1d7acf81b5eb02609c953e incorrectly set the default API version to the max the client knew about in order to get the full help output, including all microversioned commands. The original intent was to have help print out information for all versions, but still require the user to specify a version if they wanted to use any microversioned version of an API. The code accidentally made it so all commands would request the max version the client knew about. This meant an unspecified request would get the newer functionality rather than the default v3 functionality, and also meant the client could request a microversion higher than what the server knew about, resulting in an unexpected error being returned. To keep the originally intended functionality, but keep all help output, this only uses the max API version for the help command unless the user specifies otherwise. Closes-bug: #1813967 Change-Id: I20f6c5471ffefe5524a4d48c967e2e8db53233f1 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | Merge "Remove i18n.enable_lazy() translation"Zuul2019-01-301-5/+0
|\ \ \