<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/openstack/python-cinderclient.git/cinderclient/v2/shell.py, branch queens-eol</title>
<subtitle>opendev.org: openstack/python-cinderclient.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/'/>
<entry>
<title>Removes unicode 'u' response for "cinder get-capabilities"</title>
<updated>2018-01-23T13:02:11+00:00</updated>
<author>
<name>Abijitha Nadagouda</name>
<email>abijitha.nadagouda@tatacommunications.com</email>
</author>
<published>2017-04-06T12:30:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=c3a22519bcd91db227146a982bfb281092428f6c'/>
<id>c3a22519bcd91db227146a982bfb281092428f6c</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>Support create volume from backup in client</title>
<updated>2017-12-07T03:31:39+00:00</updated>
<author>
<name>TommyLike</name>
<email>tommylikehu@gmail.com</email>
</author>
<published>2017-10-30T02:42:16+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=7aedf4c898dff8e69d73da45668ec3d909f63a44'/>
<id>7aedf4c898dff8e69d73da45668ec3d909f63a44</id>
<content type='text'>
This patch adds create volume from backup support
in cinderclient.

Change-Id: I01dbcf6b113d88732c174b848be2127ee7242b3c
Implements: blueprint support-create-volume-from-backup
Depends-On: 58d0fb327f9fc980e0c8b84dcd9f64c093285d13
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch adds create volume from backup support
in cinderclient.

Change-Id: I01dbcf6b113d88732c174b848be2127ee7242b3c
Implements: blueprint support-create-volume-from-backup
Depends-On: 58d0fb327f9fc980e0c8b84dcd9f64c093285d13
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix attachment_id returned by create and show volume</title>
<updated>2017-08-28T12:36:12+00:00</updated>
<author>
<name>Steve Noyes</name>
<email>steve.noyes@oracle.com</email>
</author>
<published>2017-08-25T18:23:18+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=aab7a587b6a10a964ca79d7e81a4563a143fcf12'/>
<id>aab7a587b6a10a964ca79d7e81a4563a143fcf12</id>
<content type='text'>
The attachment_ids in the volume info returned by show volume
were incorrect. It was showing the volume_id, not the actual
attachment_id. This fix changes the attachment_ids returned by show
volume to correctly reflect the attachment_id.

Also, added a unit test for this, and added an attachment_id
to the fake volume so that other tests wouldn't now fail.

Closes-Bug: #1713082
Change-Id: I9ec36af5dd460d03d786aeeb3cc36a869c19ff62
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The attachment_ids in the volume info returned by show volume
were incorrect. It was showing the volume_id, not the actual
attachment_id. This fix changes the attachment_ids returned by show
volume to correctly reflect the attachment_id.

Also, added a unit test for this, and added an attachment_id
to the fake volume so that other tests wouldn't now fail.

Closes-Bug: #1713082
Change-Id: I9ec36af5dd460d03d786aeeb3cc36a869c19ff62
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "cinder show with attachments is a mess"</title>
<updated>2017-07-26T22:16:39+00:00</updated>
<author>
<name>Jenkins</name>
<email>jenkins@review.openstack.org</email>
</author>
<published>2017-07-26T22:16:39+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=63ac82a55489d55246da939b5ae60b8a65fb9ec7'/>
<id>63ac82a55489d55246da939b5ae60b8a65fb9ec7</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove consistencygroup quota</title>
<updated>2017-07-17T02:35:09+00:00</updated>
<author>
<name>wangxiyuan</name>
<email>wangxiyuan@huawei.com</email>
</author>
<published>2017-05-27T01:07:52+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=f3a1e6e7087899118028a270004bfa519fc8c16e'/>
<id>f3a1e6e7087899118028a270004bfa519fc8c16e</id>
<content type='text'>
Cinder doesn't support to update consistencygroup quota. And
since cg will be deprecated in Queue, there is no value to support it
at server or client side. So this patch removed it.

Change-Id: I5fbfb30611a60d575fedb676119bb0a1564700df
Closes-bug: #1693584
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Cinder doesn't support to update consistencygroup quota. And
since cg will be deprecated in Queue, there is no value to support it
at server or client side. So this patch removed it.

Change-Id: I5fbfb30611a60d575fedb676119bb0a1564700df
Closes-bug: #1693584
</pre>
</div>
</content>
</entry>
<entry>
<title>cinder show with attachments is a mess</title>
<updated>2017-07-03T18:06:38+00:00</updated>
<author>
<name>John Griffith</name>
<email>john.griffith8@gmail.com</email>
</author>
<published>2017-06-22T20:35:24+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=0ea7b2e0cf1dfd31ee8f12ceae5193c4a68c60b3'/>
<id>0ea7b2e0cf1dfd31ee8f12ceae5193c4a68c60b3</id>
<content type='text'>
The show command currently just dumps the entire attachments
dictionary into the output which is a real mess and completely
screws up the displayed output (shell command).

There's really no reason to do this, we can just give the
attachment ID's and then when you're on the newer versions
you can do an attachment-show for all the crazy details if
you want.  Keep in mind that the list command already shows
the server-id we're attached too, but that might also be
nice from the show command rather than jumping through
multiple commands.  To try and accomodate various use cases
we'll also add an "attached_servers" field to the show command,
but you'll have to coorelate manually from there.

Change-Id: I45ac49d8d9a185c52727c5bc24a6a1323be83689
Closes-Bug: #1494941
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The show command currently just dumps the entire attachments
dictionary into the output which is a real mess and completely
screws up the displayed output (shell command).

There's really no reason to do this, we can just give the
attachment ID's and then when you're on the newer versions
you can do an attachment-show for all the crazy details if
you want.  Keep in mind that the list command already shows
the server-id we're attached too, but that might also be
nice from the show command rather than jumping through
multiple commands.  To try and accomodate various use cases
we'll also add an "attached_servers" field to the show command,
but you'll have to coorelate manually from there.

Change-Id: I45ac49d8d9a185c52727c5bc24a6a1323be83689
Closes-Bug: #1494941
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "Cleared type restrictions for metadata option"</title>
<updated>2017-05-25T18:43:51+00:00</updated>
<author>
<name>Jenkins</name>
<email>jenkins@review.openstack.org</email>
</author>
<published>2017-05-25T18:43:51+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=07565abf3e8901ba499d418b5484c16a077dfb2a'/>
<id>07565abf3e8901ba499d418b5484c16a077dfb2a</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "Handle dashes in encryption-type-create arguments"</title>
<updated>2017-05-24T15:46:56+00:00</updated>
<author>
<name>Jenkins</name>
<email>jenkins@review.openstack.org</email>
</author>
<published>2017-05-24T15:46:56+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=58c60e787f7b5a0fcc856f1114e5a2fe03beb364'/>
<id>58c60e787f7b5a0fcc856f1114e5a2fe03beb364</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Handle dashes in encryption-type-create arguments</title>
<updated>2017-05-24T13:04:50+00:00</updated>
<author>
<name>Eric Harney</name>
<email>eharney@redhat.com</email>
</author>
<published>2017-03-24T17:39:56+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=69f0090b65be88c93691f89dc1cd3e9f8e2a5ee0'/>
<id>69f0090b65be88c93691f89dc1cd3e9f8e2a5ee0</id>
<content type='text'>
encryption-type-create currently takes args "--key_size"
and "--control_location".

Our standard is to use dashes rather than underscores for
arguments like this, because it's easier to type them.

Additionally, encryption-type-update already uses
"--key-size" and "--control-location" for the same args.

This adds the dashed versions to the CLI and makes them the
default shown in the help.  The underscore versions are
retained for compatibility.

Also removes redundant "(Optional)" text for arguments listed
under "Optional Arguments".  This matches other commands.

Change-Id: I1bd2b7657ec577b9775eacd163cfdf6eb6b6eab2
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
encryption-type-create currently takes args "--key_size"
and "--control_location".

Our standard is to use dashes rather than underscores for
arguments like this, because it's easier to type them.

Additionally, encryption-type-update already uses
"--key-size" and "--control-location" for the same args.

This adds the dashed versions to the CLI and makes them the
default shown in the help.  The underscore versions are
retained for compatibility.

Also removes redundant "(Optional)" text for arguments listed
under "Optional Arguments".  This matches other commands.

Change-Id: I1bd2b7657ec577b9775eacd163cfdf6eb6b6eab2
</pre>
</div>
</content>
</entry>
<entry>
<title>Cleared type restrictions for metadata option</title>
<updated>2017-05-24T09:37:41+00:00</updated>
<author>
<name>Alexey Stupnikov</name>
<email>astupnikov@mirantis.com</email>
</author>
<published>2017-05-24T09:33:57+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-cinderclient.git/commit/?id=760cc59c5a34133f6b34118fd4b1e3b902c7e297'/>
<id>760cc59c5a34133f6b34118fd4b1e3b902c7e297</id>
<content type='text'>
It turns out that it is impossible to send unicode metadata
values with some cinder commands. The reason is a type restriction
that demands all values to be a 'str' type. As a result, it is
impossible to use unicode.

Change-Id: I1213b0d3b8177b670cd3e5d587fee9dabd971923
Closes-bug: #1693151
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
It turns out that it is impossible to send unicode metadata
values with some cinder commands. The reason is a type restriction
that demands all values to be a 'str' type. As a result, it is
impossible to use unicode.

Change-Id: I1213b0d3b8177b670cd3e5d587fee9dabd971923
Closes-bug: #1693151
</pre>
</div>
</content>
</entry>
</feed>
