delta/openstack/python-cinderclient.git/cinderclient/tests/unit, branch 4.2.1

Fix shell upload-to-image with no volume type

2019-06-03T14:54:13+00:00

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)

Merge "Fix max version handling for help output" into stable/stein

2019-03-26T14:42:11+00:00

Merge "Replace assertRaisesRegexp with assertRaisesRegex" into stable/stein

2019-03-26T14:42:09+00:00

Fix max version handling for help output

2019-03-25T12:05:55+00:00

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 
(cherry picked from commit bccbd510a77843824b8187e15c4d8df9c2dfa55e)

Fix: cinder group-list not working with non-admin user

2019-03-21T21:14:09+00:00

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)

Replace assertRaisesRegexp with assertRaisesRegex

2019-03-21T17:47:13+00:00

This replaces the deprecated (in python 3.2) unittest.TestCase method
assertRaisesRegexp() with assertRaisesRegex().

Change-Id: I85025ad141f8436913ba192716435ce63e1e2d05
(cherry picked from commit 3362a65085b562632ce15c25ad29f0fda2a67784)

Merge "Fix encoding of query parameters"

2018-10-03T17:54:19+00:00

Merge "Fix backwards compat for volume transfer < 3.55"

2018-10-02T17:33:26+00:00

Merge "Fix endpoint identification for api-version query"

2018-10-02T16:16:14+00:00

Fix encoding of query parameters

2018-09-14T00:51:17+00:00

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