delta/openstack/python-cinderclient.git - opendev.org: openstack/python-cinderclient.git

	Commit message (Collapse)	Author	Age	Files	Lines
*	is_public=N/A in output of cinder type-updateliberty-eol stable/liberty	Sheel Rana	2016-03-18	1	-1/+2
\| \| \| \| \| \| \| \| \| \| \|	In output of cinder type-update execution, is_public field always displays N/A. This is due to wrong usage of @property for is_public field in code. Change-Id: I1d76c3d5c937f5c16c668bd114e2df7dfc62f2f8 Closes-Bug: #1538412 (cherry picked from commit 7d1e4d9baf76a2e42424acdaeb37265f8bdcfff1)
*	Update .gitreview for stable/liberty	Doug Hellmann	2015-09-21	1	-0/+1
\| \| \| \|	Change-Id: Idb7eda2da65d78a45f0633852cc51cbec166fc8b
*	Merge "Implement reset-state for attach_status and migration_status"1.4.0	Jenkins	2015-09-09	4	-7/+66
\|\
\| *	Implement reset-state for attach_status and migration_status	Tomoki Sekiyama	2015-07-16	4	-7/+66
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Currently the reset-state command in the client only handles the main volume status column. This adds options to set attach_status and to reset migration_status. DocImpact: Add new options to cinder reset-state command Change-Id: Ifaa04c2f8a7ff21100f7c9c51df881313e8babea Closes-Bug: #1309086
* \|	Updated from global requirements	OpenStack Proposal Bot	2015-09-03	1	-1/+1
\| \| \| \| \| \| \| \|	Change-Id: I36dc1bf00ac761486284d045cb1c1ae8ee37fd96
* \|	Update path to subunit2html in post_test_hook	Matt Riedemann	2015-09-02	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Per: http://lists.openstack.org/pipermail/openstack-dev/2015-August/072982.html The location of subunit2html changed on the images in the gate so update the path used in the post_test_hook. Long-term we should just use what's in devstack-gate. Change-Id: I5e50e7d7ad845aba26403df1df412c0a139a6dc7 Closes-Bug: #1491646
* \|	Merge "Volume status management for volume migration"	Jenkins	2015-09-02	5	-19/+101
\|\ \
\| * \|	Volume status management for volume migration	Vincent Hou	2015-09-01	5	-19/+101
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This patch proposes the following changes to the 'available' volume: * Add the --lock-volume flag to cinder migrate The default value is False, which means the migration is aborted if the owner of the volume issues commands like attach or detach during the volume migration. The volume will be in 'available' during migration. If it is set to True, the migration of this volume will not be aborted by other commands. The volume will be in 'maintenance' during migration. * List migration status for all the volumes The attribute migration_status will be listed, if the request is issued by an admin. Otherwise, the migration_status will not be listed. The option migration_status is added for the admin to filter the volumes returned via 'cinder list' command. DocImpact APIImpact Partial-implements: blueprint migration-improvement Change-Id: I5a1a717d1d08f550b45836d958a51f1f3fba5ced Depends-On: Ia86421f2d6fce61dcfeb073f8e7b9c9dde517373
* \| \|	Adds command to fetch specified backend capabilities	Mitsuhiro Tanino	2015-09-02	6	-0/+120
\|/ / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This change adds a new admin-api to allow admin to fetch specified backend capabilities which includes volume stats and vendor unique properties. With this command, admin can obtain what the current deployed backend in Cinder is able to do from the endpoint. DocImpact Implements: blueprint get-volume-type-extra-specs Change-Id: I355a5b57994d1407a088c86530d7ef362ecd31ed
* \|	Merge "CLI: Non-disruptive backup"	Jenkins	2015-08-29	4	-3/+24
\|\ \
\| * \|	CLI: Non-disruptive backup	Xing Yang	2015-08-11	4	-3/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This is the CLI change required to support non-disruptive backup for volumes in 'in-use' status. A force flag is added to the create backup CLI. The force flag needs to be True when backing up an 'in-use' volume. By default it is False and it is not needed when backing up an 'available' volume. The Cinder server side change is merged: https://review.openstack.org/#/c/193937/ Partial-implements blueprint non-disruptive-backup Change-Id: I53aff3973cc6365a5b1d40c21b0885c1d8166df5
* \| \|	Merge "Fix help message for reset-state commands"	Jenkins	2015-08-28	2	-8/+8
\|\ \ \
\| * \| \|	Fix help message for reset-state commands	Rajesh Tailor	2015-08-26	2	-8/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixed help messages for reset-state and snapshot-reset-state commands for v1 as well as v2 api. Change-Id: I06ca6e33f4e4b486ed29f258325550b89f6dc930 Closes-Bug: 1489285
* \| \| \|	Fixed test_password_prompted	Derrick J. Wippler	2015-08-27	1	-4/+8
\|/ / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	test_password_prompted no longer makes DNS lookups on the network. Change-Id: If310f52e829b09b01c76d02deae4aec74f00870d Closes-Bug: 1486647
* \| \|	Merge "Add functional tests for python-cinderclient"	Jenkins	2015-08-26	2	-4/+76
\|\ \ \
\| * \| \|	Add functional tests for python-cinderclient	Yuriy Nesenenko	2015-08-25	2	-4/+76
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add functional tests for cinder volume snapshot creation and deletion. Change-Id: I60f3a880102461f107a7f2d06c0b97f0bbe0c2e4 Partial-Implements: blueprint cinderclient-functional-tests
* \| \| \|	Merge "Show backup and volume info in backup_restore"	Jenkins	2015-08-24	3	-2/+48
\|\ \ \ \
\| * \| \| \|	Show backup and volume info in backup_restore	lisali	2015-07-20	3	-2/+48
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	When no volume is specified in backup_restore command, a new volume is created with specified name. If running the same command several times, several volumes with same name are created, it needs to check logs to find out which volume is latest. This update is to show the returned info from api. Change-Id: I035fc85aa636b716b16c8a543c843ebf4659a96e Closes-Bug: #1472493 Depends-On: I9f5d8ded993fcf1d351d4c65b6806929693b15d6
* \| \| \| \|	Merge "Set default OS_VOLUME_API_VERSION to '2'"	Jenkins	2015-08-23	2	-1/+12
\|\ \ \ \ \ \| \|_\|/ / / \|/\| \| \| \|
\| * \| \| \|	Set default OS_VOLUME_API_VERSION to '2'	Ivan Kolodyazhny	2015-07-30	2	-1/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Cinder API v1.1 is deprecated and will be removed. Switching to API v2 in cinderclient makes it works well with new API after API v1 will be removed. Change-Id: I639f4b54dc7e220e8e351113a4de1dff104b50a7
* \| \| \| \|	Merge "CLI: Clone CG"	Jenkins	2015-08-20	4	-14/+73
\|\ \ \ \ \ \| \|_\|_\|/ / \|/\| \| \| \|
\| * \| \| \|	CLI: Clone CG	Xing Yang	2015-08-08	4	-14/+73
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This patch adds support for clone CG in cinderclient. It uses the existing create CG from source command and adds an existing CG as a source, in addition to a CG snapshot. The server side patch is merged: https://review.openstack.org/#/c/203473/ Change-Id: I2a923de42348c22c6ae5ff5880cdbe6d719ea0ae Partial-implements: blueprint clone-cg
* \| \| \| \|	Merge "Add support '--all-tenants' for cinder backup-list"	Jenkins	2015-08-19	3	-6/+79
\|\ \ \ \ \
\| * \| \| \| \|	Add support '--all-tenants' for cinder backup-list	Yuriy Nesenenko	2015-08-19	3	-6/+79
\| \| \|/ / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Also added support '--name', '--status', '--volume-id' arguments for cinder backup-list. DocImpact Closes-Bug: #1422046 Depends On: I73f6377c7d6fd92d0464d13f9c8dd6682fef78e3 Change-Id: I5f2ab6370a8333a9ee498c6158037b0433f36a23
* \| \| \| \|	Merge "Add tests for python-cinderclient"	Jenkins	2015-08-19	2	-3/+144
\|\ \ \ \ \
\| * \| \| \| \|	Add tests for python-cinderclient	Yuriy Nesenenko	2015-08-11	2	-3/+144
\| \| \|/ / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add tests for cinder volume creation and deletion by ID and by Name. Add test for cinder show volume output. Add method to wait for given volume status. Add method to check that volume deleted. Add method to check that volume is not deleted (for negative cases). Add method for volume deletion. Add method to get Volume properties from CLI output. Change-Id: I1e1f9bea2afc63e408a88e64fc1b6b6ee9f4e79b Implements: partial blueprint cinderclient-functional-tests Co-Authored-by: Kyrylo Romanenko <kromanenko@mirantis.com>
* \| \| \| \|	Merge "Fixes table when there are multiline in result data"	Jenkins	2015-08-16	2	-1/+41
\|\ \ \ \ \
\| * \| \| \| \|	Fixes table when there are multiline in result data	liyingjun	2015-08-03	2	-1/+41
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The table doesn't display right when there are multiple line in result data. Fixes this by replace "\r" with " ". Change-Id: I0b994466f3c65ea973e80d9f03ca9a248147d49d Closes-bug: #1476462
* \| \| \| \| \|	Merge "Replace assertEqual(None, *) with assertIsNone in tests"	Jenkins	2015-08-16	3	-4/+4
\|\ \ \ \ \ \
\| * \| \| \| \| \|	Replace assertEqual(None, *) with assertIsNone in tests	Yusuke Hayashi	2015-08-09	3	-4/+4
\| \| \|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Replace assertEqual(None, *) with assertIsNone in tests to have more clear messages in case of failure. Change-Id: I7ea0306009e8fdde468c1394615972de4d296f45 Closes-Bug:#1280522
* \| \| \| \| \|	Merge "Remove redundant statement and refactor"	Jenkins	2015-08-14	1	-5/+3
\|\ \ \ \ \ \ \| \|/ / / / / \|/\| \| \| \| \|
\| * \| \| \| \|	Remove redundant statement and refactor	Lin Yang	2015-07-08	1	-5/+3
\| \| \|_\|/ / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	1. Remove redundant pass-statement in except-block. 2. Define "body = None" before if-block to get rid of body-statement both in except-block and else-case. 3. Acknowledge error occurred instead of silently squashing it. Change-Id: I31e17827b40e7a15d88ecacf1fdb2abbbe2da5c0 Signed-off-by: Lin Yang <lin.a.yang@intel.com>
* \| \| \| \|	Fix ClientException init when there is no message on py34	Matt Riedemann	2015-08-04	2	-3/+37
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	BaseException.message was removed in python 3 per PEP 0352 so if no message is passed to the ClientException __init__ it will blow up: AttributeError: type object 'ClientException' has no attribute 'message' So this change does two things: 1. Default to 'n/a' for message and details when body['keys'] doesn't have a message or details in it (which should be fine since from_response defaults to n/a if 'keys' is not in body). 2. Use getattr for self.__class__.message and default to None if that attribute is not set. Arguably we could just remove this and make the message kwarg default to 'n/a' in ClientException.__init__ but I figured that was more invasive. Closes-Bug: #1481478 Change-Id: I738cb9c8d4f015048c45a1df16bf18e29190e392
* \| \| \| \|	Merge "Add response message when volume delete"	Jenkins	2015-07-30	2	-0/+2
\|\ \ \ \ \ \| \|_\|_\|/ / \|/\| \| \| \|
\| * \| \| \|	Add response message when volume delete	Tom Jose Kalapura	2015-07-16	2	-0/+2
\| \| \|_\|/ \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	When a request is send for deleting a volume it will not print any success message if it deleted successfully.It will print only error message if the volume deletion fails. This fix will print response message if the volume deletion is success. Change-Id: Icef2045107e1dcc3b7f2abfeb50b3f180ded1a27 Closes-Bug: #1475430
* \| \| \|	Add commands for modifying image metadata	Dave Chen	2015-07-27	4	-0/+78
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	These are Cinder Client specific changes for modifying volume and snapshot's image metadata. - cinder image-metadata volume_id set key=value This command can be used to create a new image metadata associated with the volume when the key not exists, and update the value for the existed key. - cinder image-metadata volume_id unset key This command can be used to delete image metadata associated with the volume. Depends-On: I22792ef7bd49c763d7c130aa8cac9abc3fff2d4c Partially implements: bp support-modify-volume-image-metadata Change-Id: I03787563cb3ea4ef96ed145a37fc665f80565a12
* \| \| \|	Updated from global requirements	OpenStack Proposal Bot	2015-07-24	3	-4/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: If8c4873d1ef925f6454358843cbeb3b2d0077bc5
* \| \| \|	Merge "Remove H302"	Jenkins	2015-07-23	1	-1/+1
\|\ \ \ \ \| \|_\|_\|/ \|/\| \| \|
\| * \| \|	Remove H302	Zhenguo Niu	2015-07-21	1	-1/+1
\| \|/ / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	H302 rule was removed from hacking. [H302] Do not import objects, only modules. Change-Id: I86d45d5bc86e42e009b214cff71821ff171d69af
* \| \|	Merge "Remove unnecessary check for tenant information"	Jenkins	2015-07-20	1	-30/+36
\|\ \ \
\| * \| \|	Remove unnecessary check for tenant information	Ian Cordasco	2015-07-08	1	-30/+36
\| \| \|/ \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Keystone v3 has changed the name of tenants to projects which means that OS_TENANT_NAME is meaningless in a cloud where v3 is the default authentication version. As such, if a user does not have OS_TENANT_NAME enabled, cinderclient will not work even though it has checks for v3 immediately below this check (and the same check replicated further below that). Removing this allows cinderclient to work with Keystone v3 without setting OS_TENANT_NAME unnecessarily. Further, the check for project information was incorrectly referring to project_domain_name and project_domain_id instead of os_project_domain_name and os_project_domain_id. Closes-bug: 1472636 Change-Id: Ifabb4a2aec070c00349e794364a71394feea99f1
* \| \|	Merge "Use shared shell arguments provided by Session"	Jenkins	2015-07-16	1	-25/+4
\|\ \ \
\| * \| \|	Use shared shell arguments provided by Session	Michal Dulko	2015-06-25	1	-25/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The cinderclient implemented it's own version of the CLI arguments that are now provided by keystoneclient's session object. This changeset converts cinder over to utilizing the shared argument registration. Closes-Bug: 1434105 Change-Id: I72b16aab909d72a47097434436093e616667634a
* \| \| \|	Add more details for replication	Kuo-tung Kao	2015-07-15	1	-2/+5
\| \|_\|/ \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Show more information when user uses help function to get more details about replication-promote and replication_reenable. Change-Id: Icd41bd746c32cdfabbd27a80a818344136e7b71a Closes-Bug: #1401877
* \| \|	New mock release(1.1.0) broke unit/function tests	rico.lin	2015-07-14	4	-8/+9
\| \|/ \|/\| \| \| \| \| \| \|	Change-Id: Ia84c667b65fb69ba877bdb0ecb45827281539e03 Closes-Bug: #1473454
* \|	Merge "Fix outdated URLs and some minor fixes"1.3.1	Jenkins	2015-07-08	4	-13/+10
\|\ \
\| * \|	Fix outdated URLs and some minor fixes	Kallebe Monteiro	2015-03-02	4	-13/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A minor commit. Credit to Jeremy Stanley for findind out this in other projects. The developer documentation changed. It is now at infra manual, so now the CONTRIBUTING.md was updated accordingly. Fixed some other things, like a line break which was not needed (80 char limit), the help text, and updated URL for jacobian Change-Id: I3b7cce90f72349a986a5b055fa59e53c089ba308
* \| \|	Merge "Updated from global requirements"	Jenkins	2015-07-08	1	-2/+2
\|\ \ \
\| * \| \|	Updated from global requirements	OpenStack Proposal Bot	2015-07-07	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: I6803896e5f09a09da691d217c6d03e3957ab10db
* \| \| \|	Prep for 1.3.0	Mike Perez	2015-07-07	1	-0/+17
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: I5c6665665427d0916a5e29396ab4fa0eb3e059c6