delta/openstack/python-cinderclient.git/cinderclient/utils.py, branch ocata-em

Removed unnecessary 'service_type' decorator

2017-01-23T11:58:33+00:00

@utils.service_type was introduced with 'Initial split from
python-novaclient' commit and used for CLI. Now both Python and
Command-line APIs for each version  work well without this decorator.

Unused 'get_service_type' function is removed too.

'utils.retype_method' removed as well because everything works well
without it.

Change-Id: Ic2470d8ca9d581b7c47da8d7e6a414c3c78ad27a
Partial-Bug: #1643584
Related-Bug: #1621126

Remove extra 'u' from cli output

2016-12-16T14:11:31+00:00

The unicode_key_value_to_string() function is designed to remove
extra 'u' in cinderclient cli output.
However this patch[1] bring the extra 'u' back.
Let's remove the extra 'u' again.

Closes-bug: #1615921
Closes-bug: #1606904

[1] https://review.openstack.org/#/c/342734/
Change-Id: I26f0ad7149f57e935953c2398ba90b7b3585e201

Replace six.iteritems(iter) with iter.items()

2016-12-01T12:24:29+00:00

As mentioned in [1], we should avoid using six.iteritems(iter) to
achieve iterators. We can use iter.items() instead, as it will
return iterators in PY3 as well.

[1] http://lists.openstack.org/pipermail/openstack-dev/2015-June/066391.html

Change-Id: Ic41abf2ca6ec3ecb651b980091b52d0a185c9089

Optimize: add build_query_param method to clean code

2016-11-07T01:26:52+00:00

Currently the client build query params in respective method
this patch intends to use 'utils.build_query_param' to make
the code clean.

Change-Id: I3a3ae90cc6011d1aa0cc39db4329d9bc08801904

Fix Unicode error printing extra-specs

2016-07-15T10:46:53+00:00

If type-key is set to unicode the command cinder extra-specs-list
fails with the ERROR: 'ascii' codec can't encode characters in position.
This patch fixes it.

Change-Id: Id82bfbe8870351605f53c7ca029b9aa7e6089f6a
Closes-Bug: #1568937

Fix the incorrect alignment

2016-05-20T03:25:36+00:00

The "aligns" attribute no longer exists in the Prettytable,
"align" instead. So we should use "align" attribute to force
the output with left alignment by default.

Change-Id: I97b30216000b6e31c1bef614cf0b0d68ab8cfb08
Closes-Bug: #1583880

Support api-microversions

2016-04-19T17:19:45+00:00

Changes to cinderclient to use microversions.

Implements: blueprint api-microversion-support-for-cinderclient
api-microversion-support-for-cinderclient

Change-Id: I840a1162b88e8ff36fa3fc4e1d6b9317104df3e0

Add /v3 endpoint support for cinderclient

2016-04-18T16:49:51+00:00

Add support for Cinder API /v3 endpoint.
A couple of unit tests for /v3 endpoint were added to v3/test_shell.py
to ensure that the v3 shell works, and to also test that modules work
with:
from cinderclient.v2.availability_zones import *
syntax.

Change-Id: I6ae0ada221bebb4ab1850d9c99b10fcbb585201f
Implements: https://blueprints.launchpad.net/python-cinderclient/+spec/add-v3-endpoint-support

Use instanceof instead of type

2016-02-24T19:46:39+00:00

Adjusted conditional statements to use instanceof when
comparing variables. Instanceof supports inheritance type
checking better than type.

Change-Id: I449f3df92b3646c384de717b7d53f7f85258c449

Extra 'u' in output of cinder cli commands

2016-02-19T17:26:24+00:00

In output of cinder show command, below fields contains extra 'u'
character being unicode:
1. volume_image_metadata
2. metadata

In output of "cinder credentials", below field contains extra 'u'
1. roles

In output of "cinder qos-create", below field contains extra 'u'
1. specs

In output of "cinder qos-list", below field contains extra 'u'
1. specs

In output of "cinder extra-specs-list", below field contains extra 'u'
1. extra_specs

In output of "cinder qos-show", below field contains extra 'u'
1. specs

Change-Id: I8be32f117ddc29b087ee872ff065c175dd70b372
Closes-Bug: #1538413
Closes-Bug: #1538415