summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* OpenDev Migration Patchocata-eolstable/ocataOpenDev Sysadmins2019-04-191-1/+1
| | | | | | | | | | | | | | | This commit was bulk generated and pushed by the OpenDev sysadmins as a part of the Git hosting and code review systems migration detailed in these mailing list posts: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html Attempts have been made to correct repository namespaces and hostnames based on simple pattern matching, but it's possible some were updated incorrectly or missed entirely. Please reach out to us via the contact information listed at https://opendev.org/ with any questions you may have.
* import zuul job settings from project-configDoug Hellmann2018-09-092-0/+20
| | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to complete step 1 of moving the zuul job settings out of project-config and into each project repository. Because there will be a separate patch on each branch, the branch specifiers for branch-specific jobs have been removed. Because this patch is generated by a script, there may be some cosmetic changes to the layout of the YAML file(s) as the contents are normalized. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I01acc93d7d3ea4c55b1662c59708a07f6157ad9a Story: #2002586 Task: #24288
* Updated from global requirementsOpenStack Proposal Bot2017-05-151-1/+1
| | | | Change-Id: Iaf5edbcfb3ea8a4334050a6684ccdd1988c6e2ac
* Merge "Update UPPER_CONSTRAINTS_FILE for stable/ocata" into stable/ocataJenkins2017-03-091-1/+1
|\
| * Update UPPER_CONSTRAINTS_FILE for stable/ocataOpenStack Release Bot2017-01-281-1/+1
| | | | | | | | Change-Id: Ibe80016752ca17a6eb3066c4da36ef42361a599b
* | Merge "Update .gitreview for stable/ocata" into stable/ocataJenkins2017-03-091-0/+1
|\ \ | |/
| * Update .gitreview for stable/ocataOpenStack Release Bot2017-01-281-0/+1
| | | | | | | | Change-Id: If930e479545002c97a80d7a5f5565d8d32f0f847
* | Updated from global requirementsOpenStack Proposal Bot2017-03-091-1/+1
|/ | | | Change-Id: I0d53f5dec6852ea00dfd18c8604c28c03205f7d5
* 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
* | | | | Merge "Removed unnecessary 'service_type' decorator"Jenkins2017-01-257-247/+0
|\ \ \ \ \
| * | | | | Removed unnecessary 'service_type' decoratorIvan Kolodyazhny2017-01-237-247/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | @utils.service_type was introduced with 'Initial split from python-novaclient' commit and used for CLI. Now both Python and Command-line APIs for each version work well without this decorator. Unused 'get_service_type' function is removed too. 'utils.retype_method' removed as well because everything works well without it. Change-Id: Ic2470d8ca9d581b7c47da8d7e6a414c3c78ad27a Partial-Bug: #1643584 Related-Bug: #1621126
* | | | | | Merge "x-openstack-request-id logged twice in logs"Jenkins2017-01-242-16/+3
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | x-openstack-request-id logged twice in logsAbhishek Kekane2017-01-232-16/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the recent release of keystoneauth1 2.18.0 provision is made to log x-openstack-request-id for session client. Once this new library is synced in openstack projects, the x-openstack-request-id will be logged twice on the console if session client is used. For example, $ cinder --debug list DEBUG:keystoneauth:GET call to volumev2 for http://10.232.48.204:8776/v2/61da9e4b59cf4920acc5e78438f93223/volumes/detail used request id req-dcc22730-021e-468a-8b12-da7d58b573a7 DEBUG:cinderclient.client:GET call to volumev2 for http://10.232.48.204:8776/v2/61da9e4b59cf4920acc5e78438f93223/volumes/detail used request id req-dcc22730-021e-468a-8b12-da7d58b573a7 Above log will be logged twice on the console. Removed logging of x-openstack-request-id in case of SessionClient as it is already logged in keystoneauth1. x-openstack-request-id will only be logged once on console if HTTPClient is used. Depends-On: Id0693a9958d26162b7a2a40173ca28de2d3e4f62 Closes-Bug: #1657351 Change-Id: I0861212a38466d0e65cf3389c7d2757cff86ea0d
* | | | | | Merge "Updated from global requirements"Jenkins2017-01-241-1/+1
|\ \ \ \ \ \
| * | | | | | Updated from global requirementsOpenStack Proposal Bot2017-01-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id0693a9958d26162b7a2a40173ca28de2d3e4f62
* | | | | | | Merge "Python3 common patterns"Jenkins2017-01-244-8/+8
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Python3 common patternsxianming mao2017-01-104-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Modify some codes in order to lead them meet the python3 common pattern. Look through the following patterns of Cinder, map() and filter() if a list is needed on Python 3: Replace map(func, data) with [func(item) for item in data] Replace filter(lambda obj: test(obj), data) with [obj for obj in data if test(obj)] Replace exceptions.OSError with OSError and remove "import exceptions" Replace iterator.next() with next(iterator) Replace basestring with six.string_types Replace unicode with six.text_type Replace (str, unicode) with six.string_types Replace "for key in dict.iterkeys()" with "for key in dict" Replace dict.iteritems() with dict.items() Replace dict.itervalues() with dict.values() I found that only "filter(lambda obj: test(obj), data)" and "map(func, data)"need to modify. The other items are not be founded in Cinder. Reference:https://wiki.openstack.org/wiki/Python3#Common_patterns Change-Id: If33ec39eb176c14086132d3099c6ec577f956ded
* | | | | | | Merge "(Trival)Modify the version_header with self.version_header"Jenkins2017-01-231-1/+1
|\ \ \ \ \ \ \
| * | | | | | | (Trival)Modify the version_header with self.version_headerxianming mao2016-12-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because compiler raise a "undefined name" warning, so we should use self.version_header instead of version_header. Change-Id: If290fbf044fa86d9a8510a6e3cf65f6fc41c0ef7
* | | | | | | | Merge "_human_id_cache or _uuid_cache error about completion_cache"Jenkins2017-01-231-1/+6
|\ \ \ \ \ \ \ \
| * | | | | | | | _human_id_cache or _uuid_cache error about completion_cachedrngsl2017-01-181-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cinderclient raises _human_id_cache or _uuid_cache exception in method completion_cache. Error happend when launching many api request with creating and list volumes on the client host. Change-Id: I5c7de6fbb0a2d5106fca180ae6f940d6b738de93 Closes-Bug: #1634112
* | | | | | | | | Merge "Use 'ostestr {posargs}' to run functional tests"Jenkins2017-01-231-0/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Use 'ostestr {posargs}' to run functional testsIvan Kolodyazhny2016-11-011-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch allows us to pass such params like concurrency, test name, etc for ostestr to run functional tests. Change-Id: Icb17309a148dfebf6b679107bab1514ab3dec729 Related-Bug: #1638302
* | | | | | | | | | Merge "Fix adding non-ascii attrs to Resource objects error"Jenkins2017-01-232-1/+12
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / |/| | | | | | | | |
| * | | | | | | | | Fix adding non-ascii attrs to Resource objects errorMichael Dovgal2017-01-202-1/+12
| | |_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to these lines of code [0] we don't have an opportunity to add attributes with non-ascii symbols to Resource objects, but information about it will be in _info dict. Example of side effect - quota_show command. Because we don't have such an attr, here [1] it will be added None value instead of real value [2]. This patch fixes this problem. [0] - https://github.com/openstack/python-cinderclient/blob/f8c93ed03b388612ca28b8055debf915ce631cec/cinderclient/apiclient/base.py#L498-L499 [1] - https://github.com/openstack/python-cinderclient/blob/f8c93ed03b388612ca28b8055debf915ce631cec/cinderclient/shell_utils.py#L179 [2] - http://paste.openstack.org/show/593358/ Change-Id: I0493845dafc5dad836e899b9c22d563023c1dab0 Closes-Bug: #1652605
* | | | | | | | | Merge "Fix v3 volume list based on image_metadata"Jenkins2017-01-233-1/+15
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | Fix v3 volume list based on image_metadataVivek Agrawal2017-01-193-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | volume list filtered on image_metadata does not work as expected. The filtering based on image_metadata depended on non existent utility for extracting image_metadata. Fixed the issue by calling appropriate utility module for extracting image_metadata. Looks like a typo. Change-Id: I13e78277fc1afd22d044bb49b6a5ccc00904228c Closes-bug: #1657650
* | | | | | | | | Merge "Fix spelling of consistency groups"Jenkins2017-01-204-14/+14
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix spelling of consistency groupsPetr Kovar2017-01-184-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Trivial fixes making the consistency group spelling more consistent. Change-Id: I3716606a5415f2cbf966749de17b512c347b1790
* | | | | | | | | | Merge "Add convertation of query parameters to string"Jenkins2017-01-203-4/+23
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | |
| * | | | | | | | | Add convertation of query parameters to stringMykhailo Dovgal2016-12-303-4/+23
| | |_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some problems with non-ascii chars and special symbols during using cinderclient. This patch closes bug connected with parse.urlencode py27 unicode encode bug by adding convertation of query parameters before creating query string in manager._build_list_url method. Also it fix the problems with encoding in quota commands. Change-Id: I96269cca7ad203eaad02d87b30c16d970b26b25f Closes-Bug: #1636621 Closes-Bug: #1518141
* | | | | | | | | Updated from global requirementsOpenStack Proposal Bot2017-01-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I2f17771caf16880f9488932d306808a59f8bbed5
* | | | | | | | | Merge "Metadata based snapshop filtering"Jenkins2017-01-183-0/+109
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | |
| * | | | | | | | Metadata based snapshop filteringVivek Agrawal2017-01-183-0/+109
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The snpapshot-list API for cinder gives a list of snapshots based on certain criteria to the user. From microversion 3.22 onwards the snapshot-list API has been enhanced to support snapshot list filtering based on metadata of snapshots. The metadata is stored as key-value pair for every snapshot. With this commit cinder will be queried based on metadata key and value specified in the API snaphot-list. All the snapshots which match the key, value provided by the user along with any other filter criteria will be returned. Added the test cases for the CLI and web requests. DocImpact: "Filters results by a metadata key and value pair. Default=None." on cinder snapshot-list APIImpact Closes-bug: #1569554 Change-Id: Idec0d0d02e7956843f202508e32c023c3cafbb0f
* | | | | | | | Merge "Enable coverage report in console output"Jenkins2017-01-181-1/+3
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |