summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Rearrange existing documentation to fit the new standard layout"3.1.0Jenkins2017-07-276-2/+82
|\
| * Rearrange existing documentation to fit the new standard layoutchenxing2017-07-266-2/+82
| | | | | | | | | | | | | | | | | | | | The layout is configured to follow the design from this spec: https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html This change is necessary to continue the doc migration process. Change-Id: I5ab2e47b67baf90bdd808cb831493a010d23a070
* | Merge "import content from cli-reference in openstack-manuals"Jenkins2017-07-273-1/+4469
|\ \ | |/
| * import content from cli-reference in openstack-manualschenxing2017-07-263-1/+4469
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is part of the great doc migration from openstack-manuals to the individual repos. The changes are based on the spec here: https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html The cinder.rst file needs to be renamed index.rst so that the template that is being used by OpenStack manuals team will magically find the documentation. Change-Id: Ia3b5ee2b6aaf17781d3de7546a9477f1c928092f
* | Merge "Make --profile load from environment variables"Jenkins2017-07-273-1/+10
|\ \ | |/ |/|
| * Make --profile load from environment variablesTovin Seven2017-07-273-1/+10
| | | | | | | | | | | | | | | | --profile argument can be loaded from OS_PROFILE environment variables to avoid repeating --profile in client commands. Co-Authored-By: Hieu LE <hieulq@vn.fujitsu.com> Change-Id: Ia9b469024395327ec0ee082ddaea3234fc3ca5a6
* | Merge "Update cinder.rst and shell.rst"Jenkins2017-07-262-2/+27
|\ \
| * | Update cinder.rst and shell.rstjunboli2017-07-262-2/+27
| | | | | | | | | | | | | | | | | | | | | | | | In the file doc/source/man/cinder.rst, add cinder command examples, In the file doc/source/shell.rst, update OS_AUTH_URL and OS_VOLUME_API_VERSION Change-Id: I6ed2c5228daad8dd6dcab1d7278f8a0b049c0490
* | | Merge "cinder show with attachments is a mess"Jenkins2017-07-263-3/+19
|\ \ \
| * | | cinder show with attachments is a messJohn Griffith2017-07-033-3/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | cinderclient might not return version for V2 APIj-griffith2017-07-263-3/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The get_server_version call in cidnerclient/client.py relies on either finding v 3.x or encountering an exception to revert back to v 2.0. It's not clear that this call will always raise if a non V3 capable Cinder is called, so just to be safe make sure we return a 2.0 response if there's no V3 reported back. Change-Id: I3b5fb895cad4b85d5f4ea286fb33f7dd0929e691 Closes-Bug: #1694729
* | | | Merge "python-cinderclient doc unclear on Volume.attach"Jenkins2017-07-261-2/+23
|\ \ \ \
| * | | | python-cinderclient doc unclear on Volume.attachnidhimittalhada2017-07-261-2/+23
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In 'attach' method, documentation vaguely says "Set attachment metadata." Actually, this method should not be called directly by API users when attempting to attach a Cinder volume to a Nova instance, else the Nova and Cinder databases will become inconsistent. Instead attach function exists solely for use by consumers of Cinder services such as Nova, so that they can inform Cinder that they're now using one of Cinder's volumes. Hence correcting the doc text for attach function. Change-Id: I34af39f857b6b918de1129f5d210326b3e84d8e1 Closes-Bug: #1611613
* | | | Merge "Fix support for Unicode value filters"Jenkins2017-07-261-1/+1
|\ \ \ \
| * | | | Fix support for Unicode value filtersjunboli2017-06-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cinder supports Unicode value filters for API get-pools, therefore, Here needs to support Unicode value filters as well. Change-Id: I51df61f9465d84c408fc057443a37834c453de41 Closes-Bug: #1701427
* | | | | Merge "Handle AttributeError in _get_server_version_range"Jenkins2017-07-262-1/+17
|\ \ \ \ \
| * | | | | Handle AttributeError in _get_server_version_rangeMatt Riedemann2017-05-312-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | _get_server_version_range only supports a v3 client object because ServiceManager.server_api_version() is only implemented for the v3 client code. If you're trying to call this with a v1 or v2 client object, it will raise an AttributeError which is unhelpful for the end python API client. This change handles the AttributeError and returns a more useful error with an explanation of how the client code is wrong. We could alternatively set versions = None and let it return two empty APIVersion objects, but that seems confusing. This change opts to be explicit about the failure. Change-Id: I2266999df6332b8c5fb0ec808db69bcc847effb0 Closes-Bug: #1694739
* | | | | | Merge "Update URLs in documentation"Jenkins2017-07-267-16/+16
|\ \ \ \ \ \
| * | | | | | Update URLs in documentationHangdong Zhang2017-07-207-16/+16
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update URLs according to OpenStack document migration BTW: Do some optimization as well (http -> https) Change-Id: I9239e2012442f3e459a21f50afd380c8f58a6a98
* | | | | | Merge "Support volume summary command"Jenkins2017-07-266-0/+54
|\ \ \ \ \ \
| * | | | | | Support volume summary commandwangxiyuan2017-07-126-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch added volume summary command support. Change-Id: I7cf6e149b9ce9cae21818c60146e6db9cc2904bd
* | | | | | | Merge "Updated from global requirements"Jenkins2017-07-251-1/+1
|\ \ \ \ \ \ \
| * | | | | | | Updated from global requirementsOpenStack Proposal Bot2017-07-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib14f036bbc10a36a8e79fe6749f6b98b660caf7b
* | | | | | | | Add cinder create --pollChaynika Saikia2017-07-248-1/+148
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usage: It adds an optional argument --poll to the cinder create command which waits while the creation of the volume is completed and the volume goes to available state. In case there is an error in volume creation, it throws an error message and exits with a non zero status. The error message printed here is the async error message in case it generates one. Depends-On: Ic3ab32b95abd29e995bc071adc11b1e481b32516 Change-Id: I1a4d361d48a44a0daa830491f415be64f2e356e3
* | | | | | | Add release note for get_highest_client_server_version return type change3.0.0Matt Riedemann2017-07-201-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change Ica4d718b3de31c31da047f07c5154b242e122596 changed the return type on the cinderclient.client.get_highest_client_server_version method from a float to a str. Since it's a public API we should document the change with a release note. Change-Id: I197c80ef657156261ecbf51cee6300268438b639 Related-Bug: #1705093
* | | | | | | Fix highest version supported by client and serverGorka Eguileor2017-07-192-10/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Current code mistakenly thinks that 3.40 is older than 3.27, because it's treating 3.40 as 3.4, thus returning the wrong maximum supported version by both server and client. It is also returning a 3.40 version as 3.4 because it's returning it as a float. This patch fixes both issues by not using float conversion but using the APIVersion object to do the comparison and by changing returned type to a string so it can be used to instantiate a client. Change-Id: Ica4d718b3de31c31da047f07c5154b242e122596 Closes-Bug: #1705093
* | | | | | | Fix reset state v3 unit tests failuresGorka Eguileor2017-07-191-8/+38
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Depending on the order of execution our unit tests could fail because we are caching the reset methods in the RESET_STATE_RESOURCES dictionary in cinderclient.v3.shell and mocking the methods before importing this file will leave future tests with the mocked value. This patch fixes this by mocking find_resource instead of individual find methods, and in one of the tests with a method that will call original method whenever it's not the call we actually wanted to mock. Change-Id: Ic3ab32b95abd29e995bc071adc11b1e481b32516 Closes-Bug: #1705249
* | | | | | Merge "Updated from global requirements"Jenkins2017-07-181-1/+1
|\ \ \ \ \ \
| * | | | | | Updated from global requirementsOpenStack Proposal Bot2017-07-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Icfe847fd8583be08e5467d4f2cb31413c9ba36f1
* | | | | | | Merge "Switch from oslosphinx to openstackdocstheme"Jenkins2017-07-183-5/+21
|\ \ \ \ \ \ \
| * | | | | | | Switch from oslosphinx to openstackdocsthemeVan Hung Pham2017-07-143-5/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As part of the docs migration work[0] for Pike we need to switch to use the openstackdocstheme. [0]https://review.openstack.org/#/c/472275/ Change-Id: Ie5930c73c748bb308940606fced8f84eecdecdc8
* | | | | | | | Merge "Cinder attachment-* does not support names"Jenkins2017-07-182-2/+25
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | Cinder attachment-* does not support namesnidhimittalhada2017-07-132-2/+25
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added capability to recognize volume name too in command. It can work with both name and id now. Change-Id: If0fc63042d76beb354e6961e1f2d2936021c3d8e Closes-Bug: #1661043
* | | | | | | Remove consistencygroup quotawangxiyuan2017-07-176-18/+4
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Updated from global requirementsOpenStack Proposal Bot2017-07-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ic8567364a423091a8ce7bcbba2db6290ca1d6791
* | | | | | Merge "Add pagination for snapshots, backups"Jenkins2017-07-123-7/+39
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add pagination for snapshots, backupsGerhard Muntingh2017-06-223-7/+39
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow cinderclient to retrieve more than osapi_max_limit (default 1000) snapshots, backups, etc. Cinder pagination support has been added to the API quite some time ago. Client support was only implemented for volumes. This commit allows other resources to be paginated as well. Change-Id: I9a6f446b680dadedccd14ba49efdef7f5ef0a58a Closes-Bug: #1691229 Co-Authored-By: Gorka Eguileor <geguileo@redhat.com>
* | | | | Dynamic log level supportGorka Eguileor2017-07-106-0/+204
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds support for microversion 3.32 allowing setting and querying Cinder services' log levels. Two new commands are available: cinder service-get-log [--binary {,*,cinder-api,cinder-volume,cinder-scheduler, cinder-backup}] [--server SERVER] [--prefix PREFIX] cinder service-set-log [--binary {,*,cinder-api,cinder-volume,cinder-scheduler, cinder-backup}] [--server SERVER] [--prefix PREFIX] <log-level> Implements: blueprint dynamic-log-levels Depends-On: Ia5ef81135044733f1dd3970a116f97457b0371de Change-Id: I50b5ea93464b15751e45afa0a05475651eeb53a8
* | | | | Merge "UnboundLocalError on message-list"Jenkins2017-07-071-1/+3
|\ \ \ \ \
| * | | | | UnboundLocalError on message-listChaynika Saikia2017-06-211-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This bug for 'cinder message-list' has been fixed such that now if the user gives wrong filters, then the command properly prints WARNING message saying it is ignoring that filter. Previously, for wrong filters which were not passed in the correct usage format, the 'cinder message-list --filters' command was giving UnboundLocalError exception. The extract_filters() method was previously assuming that all the message filters passed were in the format "<filter_name>=<value>", because of which if the user passed something like: 'cinder message-list --filters event_id:VOLUME_000002' an UnboundLocalError was thrown. The changes are made to the method such that if the user passes a filter in invalid format, the cinder CLI ignores that filter while providing the output. Closes-Bug: #1696556 Change-Id: I545b3f28e545b380842ea003dc38ad70cb413aee
* | | | | | Tiramisu: replication group supportxing-yang2017-05-056-1/+236
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds CLI support for replication group. It is built upon the generic volume groups. Server side patch is here: https://review.openstack.org/#/c/352228/ Depends-On: I4d488252bd670b3ebabbcc9f5e29e0e4e913765a Change-Id: I462c3ab8c9c3a6a1b434748f81d208359ffd2431 Implements: blueprint replication-cg
* | | | | | Merge "Fix output of update command"Jenkins2017-07-062-0/+7
|\ \ \ \ \ \
| * | | | | | Fix output of update commandpoojajadhav2017-07-052-0/+7
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | User have no information at all, regarding the update of APIs (Consistency Group, Snapshot and Backup). This patch fixes output issue and displays user friendly message based on operation of update command. Closes-Bug: #1664894 Change-Id: I58f0a3729990e9f90f01b7d84e65a13af799fd97
* | | | | | Support revert to snapshot in clientTommyLike2017-05-047-1/+73
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch added revert to snapshot support in cinder client, also fix two pylint errors. Change-Id: I20d8df8d7bcf763f6651f44901a98f6d853b27ce Partial-Implements: blueprint revert-volume-to-snapshot Depends-On: cca9e1ac54d123da8859ff918b2355606bfa474e
* | | | | Unicode value support for "--filters"Eric Harney2017-06-222-7/+12
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These need to support Unicode values, like our API does. Otherwise the shell client does not work for non-ascii names, etc. Closes-Bug: #1695927 Change-Id: Ib661bb6f8df62084bdf80e7666de5708d13674b7
* | | | Merge "Enabled like filter support in client"Jenkins2017-06-213-7/+57
|\ \ \ \ | |/ / / |/| | |
| * | | Enabled like filter support in clientTommyLike2017-06-213-7/+57
| | |/ | |/| | | | | | | | | | | | | | | | | | | Updated the help message to advertise we support filter resource by inexact match since 3.34. Depends-On: 93b1c1134931749f77c5b3c7bb0f92936b3124e4 Change-Id: I8ea0eb3f0c693c6a2adce33e4d74b2327f124d40
* | | Merge "Fix cmd options for updating a quota class"Jenkins2017-06-214-9/+34
|\ \ \
| * | | Fix cmd options for updating a quota classjeremy.zhang2017-06-204-9/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Actions on updating volume quota class: [1] Add cmd option 'backups, backup-gigabytes, per-volume-gigabytes' [2] Add test cases for cmd quota-class-show and quota-class-update [3] Fix according test cases Now, cmd option '--consistencygroups' is not yet supported for updating quota class. Change-Id: I482ae501f15a103b9e07f4f17d182c853035dca9
* | | | Merge "Remove explicit global_request_id from keystoneauth subclass"Jenkins2017-06-201-5/+0
|\ \ \ \ | |/ / / |/| | |