summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Change "--sort" description in help message"2.0.1Jenkins2017-03-152-2/+3
|\
| * Change "--sort" description in help messagelihaijing2017-03-152-2/+3
| | | | | | | | | | | | | | | | | | | | In Cinder server project, class ManageResource just supports 'size' and 'refrence' sort key. So change "--sort" description in manageable-list and snapshot-manageable-list commands help message. Change-Id: Ie07bb73a9b9f2ef1a2a1e90834fcbe1f53cb562d Closes-Bug: #1657988
* | Fix service-list command for API v.3.0-3.6Ivan Kolodyazhny2017-03-142-3/+19
|/ | | | | | | | Commit I90a2b713556e91db69270a03ef6b798e08f93f90 introduced an issue with service-list command. It was broken for version from 3.0 to 3.6 Change-Id: I0febcb5debdbe157febfd1d1796e716897bde33d Closes-Bug: #1672705
* Fix all_tenants doesn't work for group listwangxiyuan2017-03-142-2/+9
| | | | | | | | | all_tenants is forgotten to pass by group list. This patch fixed it. Change-Id: I419430e929038c35747c59600be83f2e2d084802 Closes-bug: #1671293
* Merge "Disable functional tests with multiattach"Jenkins2017-03-131-0/+3
|\
| * Disable functional tests with multiattachIvan Kolodyazhny2017-03-101-0/+3
| | | | | | | | | | | | | | LVM driver doesn't support mulltiattach now. That's why this test fails on gates. We have to disable it until multiattach will be supported. Change-Id: I690982dfa81a566d1632ebe81c54f5596316b02f
* | Merge "Remove unused and duplicated fake_client module"Jenkins2017-03-121-175/+0
|\ \
| * | Remove unused and duplicated fake_client moduleIvan Kolodyazhny2017-03-101-175/+0
| | | | | | | | | | | | Change-Id: I3d3f335832b8cc44681235c79855a0cdd579894b
* | | Merge "Make V3 the default and fixup version reporting"2.0.0Jenkins2017-03-115-3/+24
|\ \ \
| * | | Make V3 the default and fixup version reportingJohn Griffith2017-03-105-3/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds in a version-list command that reports the min/max versions of the Cinder API supported by this version of the client, and also queries the Cinder API V3 server to obtain min/max micro-versions and report those as well. In addition, we bump the default version of the client to 3.0, of course if you specify V2 in your rc file or on the cmd line that works fine too. I did run into one problem where I broke: cinder.tests.unit.test_shell:test_cinder_service_name Seems to be some hidden trickery with a fake, fixture or mock that I can't figure out. For now I added a skip to that test, but maybe somebody can point out the problem during review. Change-Id: I44e667c511d89de28af758a3c9ea1f812e682f18
* | | | Merge "Add get_highest_version method"Jenkins2017-03-102-0/+18
|\ \ \ \
| * | | | Add get_highest_version methodscottda2017-03-102-0/+18
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | Add method to query a server that supports /v3 endpoint and get the highest supported microversion. Change-Id: If179760787526440c852803eafaf9617bcd2d36e
* | | | Remove duplicate get_highest_client_server_versionJohn Griffith2017-03-101-7/+0
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | The client method get_highest_client_server_version was so nice, we added it twice. Turns out it's not so nice and one is more than enough. This patch removes the duplicate. Change-Id: Ic5ad022e7cd0322c319bfb412e3113c121848fe0
* | | Merge "Remove cinder credentials command"Jenkins2017-03-102-7/+8
|\ \ \ | |_|/ |/| |
| * | Remove cinder credentials commandJon Bernard2017-02-232-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | This function has been broken for some time, this patch replaces it with a helpful message. This function can be removed in the next release. Change-Id: Ic0a4177e7818c0a1493c15d9e496dc14c63ae7a3 Close-Bug: #1615695
* | | Merge "Fix discover_version"Jenkins2017-03-105-42/+160
|\ \ \ | | |/ | |/|
| * | Fix discover_versionscottda2017-03-095-43/+161
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | discover_version needs to find the proper module for server_api_version method, and it needs to properly parse a list of versions. Do some refactor to clean things up. Add unit tests. Co-Authored-By: waj334 <justin.wilson@intel.com> Change-Id: I742bc33074cc55fe5f9682b8b97a82573c51183f Closes-Bug: #1632872
* | | Merge "Add print_function import"Jenkins2017-03-081-0/+2
|\ \ \
| * | | Add print_function importMichael Dovgal2017-01-271-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As in PY2 and PY3 we have different result after calling print, add print_function import to have the same behaviour when using empty print. Change-Id: I59c644d196805bb8a2592fc84a839c7a75d78f1a
* | | | Updated from global requirementsOpenStack Proposal Bot2017-03-022-3/+3
| | | | | | | | | | | | | | | | Change-Id: Ib145e6b0f15cf5bcacc1c3bb858e5c64c3839ee2
* | | | Group show command should be in V3wangxiyuan2017-02-202-13/+14
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | Group feature was introduced in Cinder V3, So the command should be there as well. But the "group show" command is in V2 which is wrong. Change-Id: If3ba44b2b188607542bdadfeb58f8e4b363837b7 Closes-bug: #1663496
* | | Merge "Remove duplicate columns from list output"Jenkins2017-02-164-2/+28
|\ \ \
| * | | Remove duplicate columns from list outputDinesh Bhor2017-02-144-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you specify duplicate fields in --fields, then it prints duplicate columns on the console. By default 'ID' column is added to the output so if you specified it in --fields, then it shouldn't be displayed twice. A user can pass 'NaMe', ' Name ' or 'naMe' in --fields option and it displays same name values three times under the user supplied column names. If a user doesn't pass --fields option, then it shows "Name" column in the list. To maintain consistency between user supplied column and default column names, converted it into title case and removed leading and trailing whitespaces. Kept ID field as capital only('ID') for consistency. Closes-Bug: #1659742 Change-Id: I98999e4c5934b56cd2e5a3fac1fe4d2a73a0d5a1
* | | | Merge "Add cinder tests for cinder volume create commands with parameters"Jenkins2017-02-141-0/+36
|\ \ \ \ | |/ / / |/| | |
| * | | Add cinder tests for cinder volume create commands with parametersSofiiaAndriichenko2016-09-021-0/+36
| | | | | | | | | | | | | | | | Change-Id: Icae58a69d0accc0308d581951a26970ac3872a49
* | | | Updated from global requirementsOpenStack Proposal Bot2017-02-112-3/+3
| | | | | | | | | | | | | | | | Change-Id: I5e2c2da4aec930ca080195d4d55c8c31e1fafdc4
* | | | Merge "Add start_version check for do_list() image_metadata option"Jenkins2017-02-102-2/+8
|\ \ \ \
| * | | | Add start_version check for do_list() image_metadata optionMitsuhiro Tanino2017-02-072-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The image_metadata options requires Cinder API v3.4, but user can specify this option before API v3.4 then the option will be ignored without any warning message. We should check this option using start_version to avoid confusion. Change-Id: I6983494cb6653a9eaf35f4eedba87aeb70b55260
* | | | | Merge "Add --metadata option to API v2 cinder list command again"Jenkins2017-02-103-14/+7
|\ \ \ \ \ | |/ / / /
| * | | | Add --metadata option to API v2 cinder list command againMitsuhiro Tanino2017-02-033-14/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the commit I90a2b713556e91db69270a03ef6b798e08f93f90, --metadata option of do_list() in v2/shell.py was unexpectedly removed and --image_metadata option was added instead of --metadata option. This is wrong fix because --image_metadata option requires API version >= "3.4" and is not supported at API v2. On the other hands, --metadata option of do_list() is supported from API v1. We should remove --image_metadata option and then add --metadata option to do_list() again. Also comment on API v3 cinder list --metadata should be fixed because this doesn't require API >=3.4. Co-Authored-By: Masaki Kimura <masaki.kimura.kz@hitachi.com> Change-Id: Ic7d5cfa2acb47fbf73776e034d958ad8fb9119a8 Closes-Bug: #1661045
* | | | | Handle log message interpolation by the loggerGábor Antal2017-02-074-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to OpenStack Guideline[1], logged string message should be interpolated by the logger. [1]: http://docs.openstack.org/developer/oslo.i18n/guidelines.html#adding-variables-to-log-messages Change-Id: Ib5f86d1f0846e292457813c893b73c6999e554a5 Closes-Bug: #1596829
* | | | | static method to get_highest_client_server_versionscottda2017-01-312-0/+26
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | This method takes a url for the cinder server endpoint and queries the server for version info. It then returns the min of the server's highest supported version and the cinderclients MAX_VERSION. Change-Id: Ifb3478f1dba660a5d75d243dc2aaf6b421940752
* | | | Bump MAX_VERSION to 3.27scottda2017-01-311-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | We added cinderclient support for new cinder attach APIs, but we didn't not bump the cindercliient MAX_VERSION to 3.27 to correspond with the microversion of the new APIs. Change-Id: I8ee93e28e8425cd46784a6f7280227ab28135177 Closes-Bug: 1660673
* | | Update reno for stable/ocataOpenStack Release Bot2017-01-282-0/+7
| |/ |/| | | | | Change-Id: I60fbb9160d3a7b7cf68dd1ea5e611e7f154fec5d
* | Merge "Attach/Detach V2"ocata-em1.11.0Jenkins2017-01-273-2/+273
|\ \
| * | Attach/Detach V2John Griffith2017-01-263-2/+273
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add an attachments API This includes a new attachment controller and shell commands. To use you'll need to set your api version `export OS_VOLUME_API_VERSION=3.27` Now you can do things like attach a volume (cinders part at least): `cinder attachment-create --connect True ......` List/show/delete existing attachments: `cinder attachment-list` `cinder attachment-show <attachment-id>` `cinder attachment-delete <attachemnt-id>` Change-Id: I2c463f0910b6c9e37502869b7ec33073f12939f1
* | | Merge "Missing client version 3.0 support for "delete_metadata" method"Jenkins2017-01-271-1/+16
|\ \ \
| * | | Missing client version 3.0 support for "delete_metadata" methodJustin A Wilson2016-09-291-1/+16
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Attempting to call the "delete_metadata" method with a client at version 3.0 gives versioning error. closes bug: 1659670 Change-Id: I839d405f02bcb6cb6bf6ed91dba972809b9916f7
* | | Merge "Update param docstring to ducument search_opts"Jenkins2017-01-262-2/+4
|\ \ \
| * | | Update param docstring to ducument search_optsmaxinjian2017-01-232-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Param that needs docstring is not volume_types, but search_opts, update the param docstring to document the search_opts. Change-Id: Idef956d6e616988d9dacee227781a85bb60d6ded
* | | | Merge "static method to get_server_version"Jenkins2017-01-263-0/+93
|\ \ \ \ | |_|/ / |/| | |
| * | | static method to get_server_versionscottda2017-01-263-0/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a static method that takes a url for the cinder endpoint and returns the server's min and max api version as APIVersion objects. Change-Id: I33fa9d0883ad7377c480c9e230412dfa487ccbc9
* | | | Merge "Fix getting metadata attr error in snapshot-list command"Jenkins2017-01-261-3/+9
|\ \ \ \
| * | | | Fix getting metadata attr error in snapshot-list commandMykhailo Dovgal2017-01-261-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because of 'start_version' parameter in decorator here [0] we won't have metadata attr in args using api versions from 3.0 to 3.21. This patch changes getting metadata from args logic for compatibility with the api versions 3.0 - 3.21. [0] - https://github.com/openstack/python-cinderclient/blob/b73b3932404c4645e05aaefae5502ab2304a5334/cinderclient/v3/shell.py#L1237 Change-Id: I4aa099556c57c49e9ad74fe80e5591d738cf3aa0 Closes-Bug: #1659561
* | | | | Merge "Support filter volumes by group_id"Jenkins2017-01-262-0/+18
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Support filter volumes by group_idwangxiyuan2017-01-252-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After v3.10, cinder support filter volumes by group_id, this patch support this feature for client side. Change-Id: Ie7df4d8b81789fd36ca6f91d96a477c88e8d5a52 Partial-Implements: blueprint improvement-to-query-consistency-group-detail
* | | | | Merge "Fix test_auth_with_keystone_v3 test"Jenkins2017-01-261-1/+7
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix test_auth_with_keystone_v3 testGorka Eguileor2017-01-251-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test is subject to spurious errors due to an incorrect data check. The check is assuming that a call to json.dumps with different dicts will always generate the same string, which is incorrect. This patch tests the JSON data that is sent in the request on its own based on converting the passed JSON string to a dict and comparing expected and actual dicts instead of strings. TrivialFix Closes-Bug: #1658704 Change-Id: I386cfee2e2c1dc2971d8a760b485505a90f6f120
* | | | | Merge "Remove assertTableStruct from ClientTestBase"Jenkins2017-01-251-14/+0
|\ \ \ \ \
| * | | | | Remove assertTableStruct from ClientTestBaseIvan Kolodyazhny2016-09-131-14/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | assertTableStruct is the same like in ClientTestBase, so we don't need to duplicate the code. Change-Id: I96c3bc450223d5f3f6f84c6313a0205cfb247514
View Lorry Controller Status