summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "support global_request_id in constructor"2.1.0Jenkins2017-05-252-5/+41
|\
| * support global_request_id in constructorSean Dague2017-05-252-5/+41
| | | | | | | | | | | | | | | | | | | | This allows a global_request_id to be created in the constructor which will be passed on all subsequent calls to support cross service request tracking. oslo spec I65de8261746b25d45e105394f4eeb95b9cb3bd42 Change-Id: Ib271ac01c2a4a36e611ae9181ae93305f2318680
* | Merge "Eliminate function redefined pylint error"Jenkins2017-05-251-10/+17
|\ \
| * | Eliminate function redefined pylint errorTommyLike2017-05-251-10/+17
| |/ | | | | | | | | | | | | Disable function redefined pylint error when redefining is desired in v3/shell.py, also fix some typos. Change-Id: Ic86b14ab7286a8a76f0db551ac6330211a509ff3
* | Updated from global requirementsOpenStack Proposal Bot2017-05-241-1/+1
| | | | | | | | Change-Id: Ic7a897da81773355c863ef323ae0c4155bc23294
* | Merge "Handle dashes in encryption-type-create arguments"Jenkins2017-05-242-9/+27
|\ \ | |/ |/|
| * Handle dashes in encryption-type-create argumentsEric Harney2017-05-242-9/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | encryption-type-create currently takes args "--key_size" and "--control_location". Our standard is to use dashes rather than underscores for arguments like this, because it's easier to type them. Additionally, encryption-type-update already uses "--key-size" and "--control-location" for the same args. This adds the dashed versions to the CLI and makes them the default shown in the help. The underscore versions are retained for compatibility. Also removes redundant "(Optional)" text for arguments listed under "Optional Arguments". This matches other commands. Change-Id: I1bd2b7657ec577b9775eacd163cfdf6eb6b6eab2
* | Merge "Cinder client reset-state improvements"Jenkins2017-05-238-2/+224
|\ \
| * | Cinder client reset-state improvementsTommyLike2017-05-198-2/+224
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now we can use 'cinder reset-state' command for 'volume', 'snapshot', 'backup', 'group' and 'group-snapshot'. Also change volume's default status from 'available' to 'None' when not any status is specified. Co-Authored-By: Eric Harney <eharney@redhat.com> Change-Id: I0aefeaf5ece74f1f2bc4b72d5705c8c088921e20 Partial-implements: blueprint client-reset-state-improvements
* | | Merge "Add doc for noauth usage"Jenkins2017-05-221-0/+32
|\ \ \
| * | | Add doc for noauth usagejohn-griffith2017-05-221-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | Sometimes docs are useful, let's add one for using noauth. Change-Id: I7d450031929ca5f6ea1b285e49f693776a63a23d
* | | | Updated from global requirementsOpenStack Proposal Bot2017-05-192-4/+4
| | | | | | | | | | | | | | | | Change-Id: I83fb98c2902cb1c24a266249e3a9935330e21703
* | | | Fix the wrong help message of markerwangxiyuan2017-05-191-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The marker for list manageable resources should be the resources' "reference" property, not "volume id" Change-Id: Icfe10b9a041daa6a9333f71fae15eba86d330b7b Closes-bug: #1657992
* | | | Pretty print 'extra_specs' and 'group_specs'TommyLike2017-05-122-1/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now we have 'extra_specs', 'group_specs' pretty printed. +-------------+---------------+ | other_key | value | | extra_specs | key1 : value1 | | | key2 : value2 | +-------------+---------------+ Change-Id: I3500f148f29bad89aacc89ad12a993edf4f7d460
* | | | Merge "Fix client `retries` default value."Jenkins2017-05-033-3/+3
|\ \ \ \
| * | | | Fix client `retries` default value.Georgy Dyuldin2017-05-023-3/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In case of constructing Client with session without specifying `retries` value, `cinderclient.client.Client.retries` value become to None, instead of 0. This raises TypeError on python 3 during comparison `attempts > self.retries` on `cinderclient.client.SessionClient._cs_request` Closes-Bug: #1684787 Change-Id: If1ac36acb0d1c92e9779e3e8995606c9b34f2cde
* | | | Merge "Replace http with https"Jenkins2017-05-022-3/+3
|\ \ \ \
| * | | | Replace http with httpsyfzhao2017-04-262-3/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Use https instead of http to ensure the safety without containing our account/password information Change-Id: I40d748c6044cc10eadd24eedc2d1753aa0323caf
* | | | Merge "Remove direct dependency on requests"Jenkins2017-05-021-1/+0
|\ \ \ \
| * | | | Remove direct dependency on requestsMonty Taylor2017-04-211-1/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | keystoneauth is the library that pulls in requests. If we have client libraries also depend directly on requests, we can get into situations for end users in installations where, because of sequencing of client library releases there are conflicting versions of requests needed and pkg_resources/entrypoints break. Remove the direct depend and let keystoneauth pull it in for us. Change-Id: If82ee8674b2166ec66c2a978827a203786fabec7
* | | | Merge "Update tox to delete py34"Jenkins2017-05-022-2/+1
|\ \ \ \ | |_|/ / |/| | |
| * | | Update tox to delete py34yanjun.fu2017-02-172-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to the change from Ubuntu Trusty to Xenial, where only python3.5 is available. There is no need to continue to keep these settings Change-Id: I04e036521c902d846b6341ce1fd7fc2662bff8df
* | | | Merge "Tests: Add info to assert_called failure message"Jenkins2017-05-011-1/+4
|\ \ \ \ | |_|/ / |/| | / | | |/ | |/|
| * | Tests: Add info to assert_called failure messageEric Harney2017-05-011-1/+4
| | | | | | | | | | | | | | | | | | | | | This provides enough info to understand why this assertion is failing without having to attach a debugger. Change-Id: Ib042039d7589e7c86502ac15773fce995ff64bb9
* | | Updated from global requirementsOpenStack Proposal Bot2017-04-111-1/+1
| | | | | | | | | | | | Change-Id: Ib43b540698cf5a1125058dd3405e8614a92dd511
* | | Merge "Remove log translations"Jenkins2017-04-112-13/+2
|\ \ \
| * | | Remove log translationswanghao2017-03-212-13/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Log messages are no longer being translated. This removes all use of the _LE, _LI, and _LW translation markers to simplify logging and to avoid confusion with new contributions. See: http://lists.openstack.org/pipermail/openstack-i18n/2016-November/002574.html http://lists.openstack.org/pipermail/openstack-dev/2017-March/113365.html Change-Id: I92f0729448255cec09263eccea4724c0cf9e8b61 Closes-Bug: #1674550
* | | Merge "[BugFix] Add 'all_tenants', 'project_id' in attachment-list"Jenkins2017-04-063-3/+15
|\ \ \
| * | | [BugFix] Add 'all_tenants', 'project_id' in attachment-listTommyLike2017-03-283-3/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some issues around new attach/detach API/CLI, fix them step by step. This patch adds 'all_tenants' and 'tenant' support in cinder-client. Change-Id: I6e360987a8d8bd6dfbeb34ea88f4964813f620b2 Depends-On: 8eac3b071cb7cca8f8b11952c95476fa57daffc0 Depends-On: bae8bd017e30425295e03d13b39202a81e923c3e Closes-Bug: #1675974
* | | | [BugFix] Make 'instance_id' required in attachment-create CLI.TommyLike2017-04-013-8/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some issues around new attach/detach APIs/CLIs, fix them step by step. Based our current API design in cinder [1] and irc conversation [2] the 'instance_uuid' is required, also change the name from 'instance_uuid' to 'server_id'. [1] https://github.com/openstack/cinder/blob/master/cinder/api/v3/attachments.py#L152 [2] http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2017-03-24.log.html#t2017-03-24T13:32:03 Change-Id: Ia819d04800a0f9cbd46e844895729126edc77ed9 Depends-On: 273c724382ae846fe075aa6d93b30b3d0b07bf97 Closes-Bug: #1675975
* | | | Merge "Remove duplicate do_upload_to_image() method def"Jenkins2017-03-291-41/+0
|\ \ \ \ | |/ / / |/| | |
| * | | Remove duplicate do_upload_to_image() method deflihaijing2017-03-201-41/+0
| |/ / | | | | | | | | | | | | | | | | | | | | | In cinderclient/v2/shell.py line348 and line1107, you can find the same method defination: def do_upload_to_image(). It doesn't make sense. So remove one of them. Change-Id: I0b5f2bd0c935d75d873720c9bd38f8cefe31bb30
* | | Merge "Fix noauth support"Jenkins2017-03-2711-611/+183
|\ \ \
| * | | Fix noauth supportIvan Kolodyazhny2017-03-2411-611/+183
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch includes several improvements: 1. Use keystoneauth1 plugin mechanism for auth plugins. 2. Implements CinderNoAuthPlugin. 3. Deletes non-working cinderclient.auth_plugin module. 4. Deprecates --bypass-url in flavor of --os-endpoint to be consistent with keystoneauth1 plugins. 5. Deprecates in --os-auth-system in falvor of --os-auth-type to be consistent with keystoneauth1 plugins. Both bypass_url and os_auth_system params are not changed for client objects to not break backward compatibility for Python API. How to use noauth with cinderclient CLI: OS_USER_ID=userid OS_PROJECT_ID=projectis cinder --os-auth-type noauth --os-endpoint=http://localhost:8776/v2 list Change-Id: I3be59a5a39235acbc3334e0a0b797081507a5c88 Closes-Bug: #1657156
* | | Merge "[BugFix] 'Mountpoint' is missing in attachment CLIs."Jenkins2017-03-274-4/+128
|\ \ \
| * | | [BugFix] 'Mountpoint' is missing in attachment CLIs.TommyLike2017-03-274-4/+128
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some issues around new attach/detach APIs/CLIs, fix them step by step. This patch also adds related testcases and fix some errors in help message. Closes-Bug: #1675973 Change-Id: I769ea6267403919220c515d471e7bbb8d2d95463
* | | Merge "Update README.rst to remain consistent with python-cinderclient."Jenkins2017-03-221-64/+247
|\ \ \ | |/ / |/| |
| * | Update README.rst to remain consistent with python-cinderclient.lihaijing2017-03-161-64/+247
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The latest python-cinderclient version is 2.0.1, but the content of project README file is out-of-date. So update README.rst to remain consistent with python-cinderclient. The changes are as follows: 1. Update OS_AUTH_URL to use Keystone url. 2. Modify OS_VOLUME_API_VERSION to 3(Since API V2 is offical deprecated). 3. Update "cinder help" output. 4. Update python api usage. Change-Id: Id81eb1c6b0723667bb725b6dabbe7a886a6c7f03
* | | Merge "Add cinder tests for cinder snapshot create commands with parameters"Jenkins2017-03-161-0/+51
|\ \ \
| * | | Add cinder tests for cinder snapshot create commands with parametersSofiiaAndriichenko2017-03-141-0/+51
| | | | | | | | | | | | | | | | Change-Id: Icec43c572e43eccc0408667877329bedf0f2fc1a
* | | | Merge "Fix pep8 errors"Jenkins2017-03-165-32/+28
|\ \ \ \
| * | | | Fix pep8 errorsGorka Eguileor2017-03-155-32/+28
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some PEP8 errors that are not caught by our gate because of the pep8 package version installed by hacking. This patch fixes E402, W503, and F999 erors from our codebase. Change-Id: I0b86730b1493f161645a87c3daa91ec3d774d3b1
* | | | 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