summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Drop lower-constraints jobstein-eolstable/steinBrian Rosmaita2021-04-091-1/+0
| | | | | | | | | | Running a lower-constraints job is optional in master, and seems pointless in the stable branches, so drop it. Additional information here: http://lists.openstack.org/pipermail/openstack-discuss/2021-March/021204.html Change-Id: I5faa237047c186e4f8bc85925040d906a1003aaa
* Add missed 'Server ID' output in attachment-liststein-em4.2.2Minmin Ren2020-01-213-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, }, ... ] } Closes-Bug: #1860393 Change-Id: Ica5d278cb7455befe1db4be0ab65114fd606ea0a (cherry picked from commit 03f228c11e0d88dcc396b30b7544b5cfde894750)
* Remove bash-completion calls from base.pyEric Harney2019-08-291-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 (cherry picked from commit 0707191d43396026a6ddd18213b06e1b26d94e9e)
* Tests: Don't write bash-completion cache filesEric Harney2019-08-226-0/+33
| | | | | | | | | | Mock out writing of bash-completion cache files during unit tests. Related-Bug: #1817782 Change-Id: I944862c30fb4684dd034eba6953e9302d2d22439 (cherry picked from commit a39805c6f45baf94c0d14da3ae775ee204a267ee)
* Fix shell upload-to-image with no volume type4.2.1Eric Harney2019-06-035-18/+51
| | | | | | | | | Upload-to-image would error after launching the operation if the volume type is None. Closes-Bug: #1821818 Change-Id: I015e0ddfa98d62f25334e2df5effaee72a3988ab (cherry picked from commit 274fa111696783e846561d12f670967ed01ebcbc)
* Raise API max version for Stein updates4.2.0Sean McGinnis2019-05-091-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> (cherry picked from commit eae7d55a07c7d1237614e65a6c32a8c6608318ff)
* Add support for backup user IDSean McGinnis2019-05-093-1/+9
| | | | | | | | | 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> (cherry picked from commit 44061cfab2dc74dca054546af955d12e13d6e6d2)
* Raise API max version for Rocky updatesSean McGinnis2019-04-221-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> (cherry picked from commit b843a168efc9261a12ebe322fe701ef355474874)
* OpenDev Migration PatchOpenDev Sysadmins2019-04-194-9/+9
| | | | | | | | | | | | | | | This commit was bulk generated and pushed by the OpenDev sysadmins as a part of the Git hosting and code review systems migration detailed in these mailing list posts: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html Attempts have been made to correct repository namespaces and hostnames based on simple pattern matching, but it's possible some were updated incorrectly or missed entirely. Please reach out to us via the contact information listed at https://opendev.org/ with any questions you may have.
* Merge "Fix doc build error" into stable/steinZuul2019-03-271-0/+2
|\
| * Fix doc build errorEric Harney2019-03-211-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Doc build keeps failing w/ Recursion error: maximum recursion depth exceeded while pickling an object Sphinx recommends bumping up the recursion limit to get around this, and it appears to work. Change-Id: If0a78f9af2e4ddd5fd88c44757642355b3a5bf2e (cherry picked from commit 2505e2a9031f13fbcb1f639ee5f21a2a30f77715)
* | Merge "More shell completion cache additions" into stable/steinZuul2019-03-262-1/+44
|\ \
| * | More shell completion cache additionsEric Harney2019-03-212-1/+44
| | | | | | | | | | | | | | | | | | Closes-Bug: #1712835 Change-Id: I9326d5d92ff2e93dd0398d9a115210b376059064 (cherry picked from commit e40166740ea78d4a1eb2a7040afe3a502374af51)
* | | Merge "Re-enable shell UUID completion cache" into stable/steinZuul2019-03-261-0/+24
|\ \ \ | |/ /
| * | Re-enable shell UUID completion cacheEric Harney2019-03-211-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This enables writing ids to a local completion cache when using the cinderclient shell, which allows tools/cinder.bash_completion to complete commands such as cinder delete a<tab> Volume ids are recorded on "cinder list" and "cinder create" operations, similar for backup ids. This functionality was unintentionally removed in changes some time ago. Labeled as Partial-Bug because I haven't added name caching yet, which also used to exist. Partial-Bug: #1712835 Change-Id: Id56aa43b061758a00a2a8c9c92a5a33ab9f7ab84 (cherry picked from commit c4b37c2830fdd90b57383a93e09a8bd40ca41221) (cherry picked from commit 7319a40767641aa83d75cc99a807fd231f52f1ac)
* | | Merge "Change bash completion dir permissions to 0750" into stable/steinZuul2019-03-261-1/+1
|\ \ \ | |/ /
| * | Change bash completion dir permissions to 0750Eric Harney2019-03-211-1/+1
| |/ | | | | | | | | | | | | This is no reason for this dir to be world-readable. Change-Id: I50e85b5bb6116c64535ecbf09718141086c703c5 (cherry picked from commit 5d2116e7476fb294d70e763a6022abd7322407dd)
* | Merge "Fix max version handling for help output" into stable/steinZuul2019-03-262-2/+17
|\ \
| * | Fix max version handling for help outputSean McGinnis2019-03-252-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> (cherry picked from commit bccbd510a77843824b8187e15c4d8df9c2dfa55e)
* | | Merge "Remove i18n.enable_lazy() translation" into stable/steinZuul2019-03-261-5/+0
|\ \ \ | |/ /
| * | Remove i18n.enable_lazy() translationSean McGinnis2019-03-211-5/+0
| |/ | | | | | | | | | | | | | | | | | | Per [1], don't. [1] http://lists.openstack.org/pipermail/openstack-dev/2018-November/136289.html Change-Id: I4a1e8cf84a88e5bd301d1baba41b064cde336f38 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com> (cherry picked from commit 4b0f5876d6a10e599246f7b6d8c1b3c49aa4afb0)
* | Merge "Replace assertRaisesRegexp with assertRaisesRegex" into stable/steinZuul2019-03-262-10/+10
|\ \
| * | Replace assertRaisesRegexp with assertRaisesRegexzhangyangyang2019-03-212-10/+10
| |/ | | | | | | | | | | | | | | This replaces the deprecated (in python 3.2) unittest.TestCase method assertRaisesRegexp() with assertRaisesRegex(). Change-Id: I85025ad141f8436913ba192716435ce63e1e2d05 (cherry picked from commit 3362a65085b562632ce15c25ad29f0fda2a67784)
* | Merge "Don't quote {posargs} in tox.ini" into stable/steinZuul2019-03-261-2/+2
|\ \
| * | Don't quote {posargs} in tox.iniVieri2019-03-221-2/+2
| |/ | | | | | | | | | | | | | | | | Quotes around {posargs} cause the entire string to be combined into one arg that gets passed to stestr. This prevents passing multiple args (e.g. '--concurrency=16 some-regex') Change-Id: Ieeae90ecbd4f89cbff0513dcd949a0c1a8e8d8c1 (cherry picked from commit 3b954791e4ebbaa1ed0721a441698f9acb56c7d2)
* | Merge "Fix incorrect punctuation" into stable/steinZuul2019-03-261-3/+3
|\ \
| * | Fix incorrect punctuationyenai2019-03-211-3/+3
| |/ | | | | | | | | Change-Id: I70e1ad2d0152e74ab0672a3ac2e7b47f85ee92d7 (cherry picked from commit a8cd90a760c7067b2f4d17051e114a287bf3725a)
* | Merge "Change openstack-dev to openstack-discuss" into stable/steinZuul2019-03-261-1/+1
|\ \
| * | Change openstack-dev to openstack-discusssunjia2019-03-211-1/+1
| |/ | | | | | | | | | | | | Mailinglists have been updated. Openstack-discuss replaces openstack-dev. Change-Id: Iee7d2087d7015f49e96345d96217a150bb1c23e2 (cherry picked from commit c6fdd2a67da8df7dd0d5494e7e6031be50fa1c2f)
* | Merge "Fix: cinder group-list not working with non-admin user" into stable/steinZuul2019-03-262-13/+11
|\ \
| * | Fix: cinder group-list not working with non-admin userwhoami-rajat2019-03-212-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 (cherry picked from commit 532aef0c738edc937dbfa1b54efd8d7773af2204)
* | Merge "Replace openstack.org git:// URLs with https://" into stable/steinZuul2019-03-252-2/+2
|\ \
| * | Replace openstack.org git:// URLs with https://Ian Wienand2019-03-242-2/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a mechanically generated change to replace openstack.org git:// URLs with https:// equivalents. This is in aid of a planned future move of the git hosting infrastructure to a self-hosted instance of gitea (https://gitea.io), which does not support the git wire protocol at this stage. This update should result in no functional change. For more information see the thread at http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html Change-Id: I7b9b66536d102aeae64cfed6a62c69303943c068
* | Merge "Add Python 3.6 classifier to setup.cfg" into stable/steinZuul2019-03-251-0/+1
|\ \
| * | Add Python 3.6 classifier to setup.cfgqingszhao2019-03-211-0/+1
| |/ | | | | | | | | Change-Id: I8eb981871397e1d9ae84e775d4e68915c06ecef7 (cherry picked from commit 75bc7e50f475f40b3023ed6dd21830d53071cf69)
* | Merge "Cleanup the home page" into stable/steinZuul2019-03-255-91/+11
|\ \
| * | Cleanup the home pageMatt Riedemann2019-03-215-91/+11
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Good lord this thing needed some love. This change does a few things: * Remove the duplicate man page. * Rename some titles to drop the CINDERCLIENT yelling - this is docs about cinderclient, I get it, no need to repeat in all caps. * Fixed the sub-section title formatting in the home page. * Dropped the duplicate usage guide wording and simply linked to the user docs from the home page. Change-Id: I259787a40151e8c875ad87021f655a141e06b15e (cherry picked from commit 85a3f0e30d7ac8532b86f359803eba341b9590d9)
* | Add dependency on requests libEric Harney2019-03-211-0/+1
|/ | | | | | | cinderclient uses the requests library. Change-Id: I2944ce54a304cb91f456c6c53daf2f083017c929 (cherry picked from commit a952c550b3d2b89cb4107e71842133a9a5574aeb)
* Update UPPER_CONSTRAINTS_FILE for stable/steinOpenStack Release Bot2019-03-181-4/+4
| | | | | | | | | | | | Update the URL to the upper-constraints file to point to the redirect rule on releases.openstack.org so that anyone working on this branch will switch to the correct upper-constraints list automatically when the requirements repository branches. Until the requirements repository has as stable/stein branch, tests will continue to use the upper-constraints list on master. Change-Id: If0abae9bbfabbd0cd2fb385b65e0eaeec8255df4
* Update .gitreview for stable/steinOpenStack Release Bot2019-03-181-0/+1
| | | | Change-Id: I9f27e8710f390a3917dae064a1c84a31f81e68fa
* Merge "Fix encoding of query parameters"4.1.0Zuul2018-10-0312-60/+65
|\
| * Fix encoding of query parametersGoutham Pacha Ravi2018-09-1312-60/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | IETF RFC 3986 classifies "~" as a reserved character [1], however until python3.7 [2], python's url parsing used to encode this character. urllib has seen a lot of churn in various python releases, and hence we were using a six wrapper to shield ourselves, however, this backwards-incompatible change in encoding norms forces us to deal with the problem at our end. Cinder's API accepts "~" in both, its encoded or un-encoded forms. So, let's stop encoding it within cinderclient, regardless of the version of python running it. Also fix an inconsitency around the use of the generic helper method in utils added in I3a3ae90cc6011d1aa0cc39db4329d9bc08801904 (cinderclient/utils.py - build_query_param) to allow for False as a value in the query. [1] https://tools.ietf.org/html/rfc3986.html [2] https://docs.python.org/3/library/urllib.parse.html#url-quoting Change-Id: I89809694ac3e4081ce83fd4f788f9355d6772f59 Closes-Bug: #1784728
* | Merge "refactor the getid method base.py"Zuul2018-10-032-13/+5
|\ \
| * | refactor the getid method base.pyzhubx0072018-08-082-13/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refer to a merged commit. https://review.openstack.org/#/c/588983/ Refactor the getid method both in cinderclient/base.py and in cinderclient/apiclient/base.py TrivialFix Change-Id: I4d1fb81f6876ab072ded3f14004ad064dcc949d3
* | | Merge "Fix backwards compat for volume transfer < 3.55"Zuul2018-10-023-31/+128
|\ \ \
| * | | Fix backwards compat for volume transfer < 3.55Sean McGinnis2018-08-043-31/+128
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | All volume transfer v3 calls were being sent to the endpoint introduced with microversion 3.55. This fixes backwards compatibility by routing calls less than 3.55 to the original API extension endpoint. Change-Id: I7205033ddd5be126b8614372a9fc82a2bc555f48 Closes-bug: #1785330 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | Merge "Fix functional error check for invalid volume create size"Zuul2018-10-021-2/+2
|\ \ \
| * | | Fix functional error check for invalid volume create sizeSean McGinnis2018-09-111-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the api-schema enforcement on the Cinder API side, the error message returned when specifying a volume creation of 0 has changed. This results in our functional tests failing. Change-Id: I1a9a13f683134faa01ad50f7f073db8b1845a901
* | | | Merge "Fix endpoint identification for api-version query"Zuul2018-10-022-4/+21
|\ \ \ \
| * | | | Fix endpoint identification for api-version querySean McGinnis2018-08-062-4/+21
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The api-version query is a GET against the root endpoint of cinder. Determining this root endpoint had a flaw with the newer wsgi deployment URLs that would cause it to find the root endpoint of the web server and not cinder. This updates the logic to work with legacy, wsgi, and custom URLs for the Cinder endpoint. Change-Id: Iaeba1f8d50ee8cc9410cc9f638770a56796871fb Closes-bug: #1785594 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>