summaryrefslogtreecommitdiff
path: root/cinderclient
Commit message (Collapse)AuthorAgeFilesLines
* Deprecate multiattach flag on volume createSean McGinnis2017-12-212-2/+20
| | | | | | | | | | | | | | This form of multiattach was never fully supported and is now being replaced by the new mutliattach workflow with Nova. As part of this, volume type extra specs will be used to indicate multiattach ability and will allow retyping between volume types to change the flag after creation. That work is still in progress and has some potential to change. But we know we are not going to support this old style, so we should get that deprecated now so we can remove it and not cause any confusion. Change-Id: Icbb9c0ca89b25620cedff6cac7a4723e7126eca6
* Merge "Fix the way to get backup metadata"Zuul2017-12-202-5/+16
|\
| * Fix the way to get backup metadatalihaijing2017-12-132-5/+16
| | | | | | | | | | | | | | | | | | | | In v3/shell.py do_backup_update(), the metadata through 'args.metadata' is a list, like this: 'metadata': [u'k1=v1']. But we need a metadata dict like this: 'metadata': {"k1": "v1"}. So call the right method shell_utils.extract_metadata() to get metadata from args. Change-Id: I82cb96b1f89b4009d4b3f4a0e64db076a7a04977
* | Merge "Backup create is not available from 3.0 to 3.42"Zuul2017-12-153-16/+69
|\ \
| * | Backup create is not available from 3.0 to 3.42TommyLike2017-12-143-16/+69
| |/ | | | | | | | | | | | | | | Backup create command is shielded by patch [1]. [1]: 2255fc99da9752737dcaa96ae4507b646074afb2 Change-Id: I100b8734ee2df4d81e16e2bfdafd81227c20d25e
* | Removed unnecessary parameters from group and group_snapshots create APIspooja jadhav2017-12-145-23/+5
|/ | | | | | | | | | | As per code, the 'status', 'user_id' and 'project_id' parameter is not required to be passed in the request body in case of group and group_snapshot create APIs. Even if you pass these parameter, it is silently ignored in the code. This patch removes those parameters passed in the request body. Change-Id: I29e7d4c8a3eee52f4ea7278d2edf2c7deec40628
* Merge "Remove 'end_version' parameter in backup update"Zuul2017-12-112-5/+2
|\
| * Remove 'end_version' parameter in backup updateTommyLike2017-12-042-5/+2
| | | | | | | | | | | | | | | | | | Remove unnecessary 'end_version' which was added in patch [1]. [1]: 2255fc99da9752737dcaa96ae4507b646074afb2 Change-Id: If014ba9c12bf662ef23ebf17decfa59273c63944
* | Bump Max API version to 3.483.3.0John Griffith2017-12-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | In order for Nova to be able to make use of the code for shared_targets that has gone into Cinder we need to bump to API Version to 3.48. This bumps to 3.48 to match the latest add to the Cinder side: https://review.openstack.org/#/c/524697/1 Change-Id: Icaa17654bd2810098d02f8074fedb35c5221cd55
* | Support create volume from backup in clientTommyLike2017-12-076-7/+49
| | | | | | | | | | | | | | | | | | This patch adds create volume from backup support in cinderclient. Change-Id: I01dbcf6b113d88732c174b848be2127ee7242b3c Implements: blueprint support-create-volume-from-backup Depends-On: 58d0fb327f9fc980e0c8b84dcd9f64c093285d13
* | Bump Max version to 3.46TommyLike2017-12-071-1/+1
| | | | | | | | | | | | | | | | | | | | Bump max version in client directly as the client changes are not needed. 1. 3.46: support create bootable volume from nova 0 size image. Change-Id: Ib0cf2d88e55006c10b873327ccc85dca2dd09769
* | Support list with 'with_count' in clientTommyLike2017-12-055-17/+114
|/ | | | | | | | Add 'with_count' parameter in list volume, snapshot and backup APIs. Change-Id: I2b7b41b3579c24703a7a67ab5dc6f960a3ccbdc2 Partial-Implements: bp add-amount-info-in-list-api
* Merge "Add cluster support in migration and manage"Zuul2017-12-018-48/+447
|\
| * Add cluster support in migration and manageGorka Eguileor2017-10-038-48/+447
| | | | | | | | | | | | | | | | | | | | | | | | This patch adds support for API microversion 3.16, which allows us to pass --cluster optional argument to migration and manage operations. For this, a new type of CLI argument is added, the mutually exclusive arguments that can be used similarly to the utils.arg decorator, but with utils.exclusive_arg decorator. Implements: blueprint cinder-volume-active-active-support Change-Id: If004715b9887d2a0f9fc630b44d6e11a4a8b778d
* | Merge "Revert "Bump MAX version of client to 3.45""Zuul2017-11-281-1/+1
|\ \
| * | Revert "Bump MAX version of client to 3.45"Sean McGinnis2017-10-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | 3.45 has landed, but it's not the expected change. This reverts commit 664299b235be69ae35670d96fb70e62836b9b5b0. Change-Id: I39e93c6078871f8652889cf8a46c1af6962288e4
* | | Merge "Remove unused attribute when updating quota_class"Zuul2017-11-023-8/+6
|\ \ \
| * | | Remove unused attribute when updating quota_classTommyLike2017-08-183-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When updating quota class, attribute 'class_name' is neither used in the request nor the response [1]. [1]: https://github.com/openstack/cinder/blob/master/cinder/api/contrib/quota_classes.py#L56 Change-Id: Ic1a743ce36a087f369703f10313d51b79b5cab9c
* | | | Merge "Bump MAX version of client to 3.45"Zuul2017-10-291-1/+1
|\ \ \ \ | | |/ / | |/| |
| * | | Bump MAX version of client to 3.45j-griffith2017-09-261-1/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding a shared_targets field to Volumes in Cinder. This doesn't require any changes in the client, however we do need to bump the MAX MV support value. Show volumes now returns two new fields that will be displayed in the show command: shared_targets: <True|False> backend_name: <backend-name-of-storage-device> Change-Id: Ie1291f90756c3752de562f022126af2f6b3e0637
* | | Merge "Fix OS_AUTH_TYPE env var usage"Zuul2017-10-192-2/+24
|\ \ \
| * | | Fix OS_AUTH_TYPE env var usageGorka Eguileor2017-08-042-2/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now only deprecated OS_AUTH_SYSTEM environmental variable works to set the authentication to noauth, the reason is that we have multiple arguments writing on the same destination. This patch fixes this by making both arguments have the same default value, which is to use environmental variable OS_AUTH_TYPE if it has a value or deprecated OS_AUTH_SYSTEM if not. Closes-Bug: #1708687 Change-Id: I478fb0e628d4bebd4a1dc78c2559202916aa051f
* | | | Use generic user for both zuul v2 and v3Nam Nguyen Hoai2017-10-171-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Zuul v2 uses 'jenkins' as user, but Zuul v3 uses 'zuul'. Using $USER solves it for both cases. Change-Id: I7bd4c3d5a0acc2c73557a65dcc274a97f16bfd07
* | | | Merge "Correct sphinx source code syntax"Jenkins2017-10-101-0/+1
|\ \ \ \
| * | | | Correct sphinx source code syntaxliuyamin2017-08-091-0/+1
| | | | | | | | | | | | | | | | | | | | Change-Id: Ia6763a019dbb89f0944d702ce78be963a7ef8271
* | | | | Merge "Add api_version_wraps to attachment-complete"Jenkins2017-10-084-6/+95
|\ \ \ \ \
| * | | | | Add api_version_wraps to attachment-completej-griffith2017-09-094-6/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The attachment-complete method didn't include the version checking. This patch adds that and adds a couple of tests. Change-Id: I63dba6ee6d56bb63a79f3ad2ef3412ebd9c1ce07 Closes-Bug: #1715732
* | | | | | Let keystoneauth set the microversion headerEric Fried2017-09-281-4/+5
| |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Constructing the Adapter with the default_microversion kwarg causes the Adapter to handle setting the appropriate Openstack-API-Version header. With this change set, SessionClient now uses the api_version kwarg to set default_microversion instead of setting the header explicitly. Change-Id: I5de721fa373f5d3adb42ddb4b5598f19aabfff3b
* | | | | Implement UserID in snapshot list responsej-griffith2017-09-212-3/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have a gap in the microversions implemented in the cinderclient and the MAX version being reported by the client. One of those is the addition of "user_id" in the snapshot list response. This patch adds that with the appropriate version checking. Note that the Max version has already been updated, but there are a few implementations missing. Change-Id: Ia8bdb46b46091a89996e404825466430b8906d18 Partial-Bug: #1715759
* | | | | Implement metadata for backup create/updatej-griffith2017-09-093-2/+115
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have a gap in the micro versions implemented in the cinderclient and the MAX version being reported by the client. This patch addresses the missing changes for micro version 3.43 which adds support for metadata for backup create and update. Change-Id: I14c7205784c53ec3ab0985a4460a88f59de171fb Partial-Bug: #1715759
* | | | Merge "Unsupported 'message' Exception attribute in PY3"3.2.0Jenkins2017-08-291-3/+3
|\ \ \ \
| * | | | Unsupported 'message' Exception attribute in PY3liuyamin2017-08-281-3/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'message' attribute has been deprecated and removed from Python3. Use six.text_type(e) instead of e.message. For more details, please check [1]: [1] https://www.python.org/dev/peps/pep-0352/ Change-Id: Ibd4e7f5fefa6b1dfb6258c5eacfa53cd3485d22c
* | | | Merge "Fix attachment_id returned by create and show volume"Jenkins2017-08-283-3/+21
|\ \ \ \
| * | | | Fix attachment_id returned by create and show volumeSteve Noyes2017-08-283-3/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | Enable F811, F821Eric Harney2017-08-234-6/+6
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | F811 is a little noisy with api_versions.wraps() decorated methods, but opting to disable it in these cases seems better than missing problems in the rest of the code. Change-Id: I9c46938ab2a442e55f96b5ab6c119c4475afaecd
* | | | Fix method/module redefinition errorsEric Harney2017-08-233-13/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are caught by F811, but that test is currently disabled because it fails on @api_versions.wraps() decorated methods. Just clean up these errors for now. Change-Id: I42aed024352f798ebafddeb6f12880d8726360fe
* | | | Merge "Enable H306"Jenkins2017-08-2242-63/+86
|\ \ \ \ | |_|_|/ |/| | |
| * | | Enable H306Eric Harney2017-08-0142-63/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enforce ordering of imports with H306. For tests, this is mostly done by grouping test imports after other cinderclient imports. Change-Id: Ie40fda014d1aedb057e5b4ea1f27f999c84e6373
* | | | Add an attachment_complete API callJohn Griffith2017-08-143-1/+28
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new attachment_update method in Cinder's API creates an attachment object and populates it with the provided connector info. In addition, we set the volumes status to in-use and update the attachment object status to "attached". This isn't really accurate though, because we don't know if the volume is actually attached (connected) by the consumer or not. Also a big side effect here is that currently all of our tests and automation use volume-status to determine if a volume is fully connected/ready for use and that everything went well. It's used as an ack in most cases. This change goes back to using multiple states to signify where a an attachment is in it's life-cycle: 1. attachment_create We've created an empty attachment record but haven't done anything with it yet. 2. attachment_update We provided a connector and set up the TGT so that everything is ready for a consumer to connect/use it. 3. attachment_complete An ACK back from the consumer letting us know that they connected it successfully and are doing their thing. It would be cool to just key all these checks of the attachment object itself, and just use attachment_update to do so, maybe in the future? Change-Id: I3b28d2cb89a8d81c7cdec425f355c78bfdfe3450 Related-Bug: #1710295
* | | Merge "Support skip-validation for quota update"Jenkins2017-08-084-1/+55
|\ \ \
| * | | Support skip-validation for quota updatewangxiyuan2017-07-184-1/+55
| | | | | | | | | | | | | | | | | | | | | | | | Support skip-validation parameter for quota update. Change-Id: Iec4f1598987c632f45ad6eee751f78bedbf3ec26
* | | | Merge "Fix get_highest_client_server_version with Cinder API + uWSGI"Jenkins2017-08-042-12/+30
|\ \ \ \ | |_|_|/ |/| | |
| * | | Fix get_highest_client_server_version with Cinder API + uWSGIIvan Kolodyazhny2017-08-042-12/+30
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_highest_client_server_version should work with any endpoint type: cinder-api with eventlet, uWSGI, etc. This patch is based on python-novaclient patch [1]. In a future, we can deprecate get_highest_client_server_version in flavor of keystoneauth descovery feature. [1] Icba858b496855e2ffd71b35168e8057b28236119 Closes-Bug: #1708188 Change-Id: I9acf6dc84c32b25bfe3254eb0f97248736498d32
* | | Merge "Added missing column 'Allocated'"Jenkins2017-08-031-1/+1
|\ \ \ | |/ / |/| |
| * | Added missing column 'Allocated'TommyLike2017-07-311-1/+1
| | | | | | | | | | | | | | | | | | | | | Added missing column 'Allocated' for quota usage command. Change-Id: Ic2656e2f849e834c7a576b0462d6e8a399a95006
* | | Merge "Clean the redundant code in shell.py"Jenkins2017-08-011-12/+7
|\ \ \ | |/ / |/| |
| * | Clean the redundant code in shell.pyjunboli2017-07-271-12/+7
| | | | | | | | | | | | | | | | | | | | | In the file cinderclient/shell.py, packages requests and six are double imported. This patch is to clean the redundant code. Change-Id: I4bade57753a50245df1b4b82c4a8708d3b0acdba
* | | Merge "[Optimize] Adds interval and increase waiting time"Jenkins2017-07-281-1/+2
|\ \ \
| * | | [Optimize] Adds interval and increase waiting timeTommyLike2017-06-161-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Gives more time to wait resource's status to change in functional testcases and adds default interval to reduce query amount. Change-Id: I2a29a2d04836fd1261d45e404341bb1aa657417b
* | | | Merge "Make --profile load from environment variables"Jenkins2017-07-271-1/+3
|\ \ \ \
View Lorry Controller Status