summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Don't enable lazy translation when loading client1.8.0Victor Stinner2016-05-301-4/+0
| | | | | | | | | | | | | | The change I9c8db9487b554b637a41620c858a7e7abf802879 introduced a regression in nova and trove. Importing cinderclient.client now calls _i18n.enable_lazy() which calls oslo_i18n.enable_lazy(). It's wrong to modify a global variable (oslo_i18n._lazy.USE_LAZY) when a module is imported. This change removes the call to _i18n.enable_lazy() from client.py. Closes-Bug: #1587071 Related-Bug: 1586976 Change-Id: I1512b86815e7248fa226c6969124ddc654145562
* Merge "Remove deprecated tempest_lib and use tempest.lib"Jenkins2016-05-252-4/+4
|\
| * Remove deprecated tempest_lib and use tempest.libwanghao2016-05-202-4/+4
| | | | | | | | | | | | | | | | | | | | Since 10.0.0 release tempest-ib is deprecated now. Cinderclient should use tempest.lib in test code and add tempest>=11.0.0 in requirements following global-requirements. Closes-Bug: #1583583 Change-Id: I9d766f46eb6fff29ae9c0267a60ff2ef4bf2343e
* | Merge "Support for cinder backup force delete"Jenkins2016-05-233-5/+40
|\ \
| * | Support for cinder backup force deleteSheel Rana2016-05-213-5/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cinder backup force delete is supported from cinder server side but cinderclient support is not present. This patchset adds support for cinder backup force delete from cinderclient side. Closes-Bug:#1567243 Partially-Implements: blueprint snapshot-backup-force-delete Change-Id: I4ec38376d57ef6f74fefdd3bd2fea7a10bc51672
* | | Fix upload_to_image method1.7.1Ivan Kolodyazhny2016-05-206-27/+104
| |/ |/| | | | | | | | | | | | | | | | | | | Commit Ie639179c5bbbaca4de62b42b368830afcfd8f7ac introduced 'visibility' and 'protected' params. These params should be used only with v3.1 microversion. Also these changes break current v2 users. This patch fixes these issues. Closes-Bug: #1584056 Change-Id: I0574631791c475bbefdb6e7d1647a20d0759df64
* | Merge "Updated from global requirements"1.7.0Jenkins2016-05-181-2/+2
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2016-05-181-2/+2
| | | | | | | | | | | | Change-Id: I299ff1546179c71664b43e8e821b9e45b9a2fe73
* | | Don't reset volume status when resetting migration statusMitsuhiro Tanino2016-05-186-8/+23
|/ / | | | | | | | | | | | | | | | | | | | | | | In case of failed volume migration, status of the volume is still in-use and the migration status is set to error. Current reset-migration-status command resets not only migration status but also volume status. However the volume status should not reset because the volume is still attached. Closes-Bug #1552058 Change-Id: I9a8a5ed6a00bdcffecbf98862fe60aee373f5e9b
* | Merge "Support --os-key option"Jenkins2016-05-183-0/+31
|\ \
| * | Support --os-key optionCedric Brandily2016-05-173-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we can specify a client certificate key using --os-key option inherited from keystoneclient cli options but it has no effect because keystoneclient Session expects as cert argument, one of the followings: * None (no client certificate), * a path to client certificate, * a tuple with client certificate/key paths. The change updates cinderclient code to support the last case (ie: os_cert and os_key are non-empty) in order to take into --os-key option and OS_KEY environment variable. Closes-Bug: #1564646 Change-Id: I258fd554ad2d6a5413ffe778acefa3a0b83e591f
* | | Merge "Add options when uploading images to Glance"Jenkins2016-05-175-5/+74
|\ \ \
| * | | Add options when uploading images to GlanceNathaniel Potter2016-04-215-5/+74
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added --visibility and --protected options when uploading volumes to the image service. DocImpact Change-Id: Ie639179c5bbbaca4de62b42b368830afcfd8f7ac Closes-Bug: #1288131 Depends-On: I6e6b2276af22b7809ea88289427c6873211b3faf Signed-off-by: Nathaniel Potter <nathaniel.potter@intel.com>
* | | Merge "Change api_version to self.api_version"Jenkins2016-05-112-2/+2
|\ \ \
| * | | Change api_version to self.api_versionscottda2016-05-102-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Call to _construct_http_client uses: api_version=api_version but should be: api_version=self.api_version Change-Id: If2686461bff1ef9afd318e999c9e517e15b1677f Closes-Bug: 1580319
* | | | Merge "Remove Python 2.5 compat shim"Jenkins2016-05-111-5/+0
|\ \ \ \ | |/ / / |/| | |
| * | | Remove Python 2.5 compat shimEric Harney2016-04-211-5/+0
| |/ / | | | | | | | | | | | | | | | We don't support or test with Python 2.5. Change-Id: Id29f0cee786205cc751d2d5bc031b3c105ae6aaa
* | | Merge "Updated from global requirements"Jenkins2016-05-091-1/+1
|\ \ \
| * | | Updated from global requirementsOpenStack Proposal Bot2016-05-061-1/+1
| |/ / | | | | | | | | | Change-Id: I0c2d5ae4869fc1f00cf6e041ad840cef0da90a65
* | | Only print volume ID in migration messagesEric Harney2016-04-211-2/+2
|/ / | | | | | | | | | | | | This displays the volume id rather than <Volume: id>. Change-Id: Iff4096a86d0fd403ee1b56d76ec78e879eee331b Closes-Bug: #1573094
* | Merge "Use six.moves.urllib.parse urlencode"Jenkins2016-04-216-25/+10
|\ \
| * | Use six.moves.urllib.parse urlencodeEric Harney2016-04-206-25/+10
| | | | | | | | | | | | | | | | | | | | | Rather than having to do try/except for imports, just use six.moves for loading this module. Change-Id: If750253d7e02608fd183675bd97e3956fbfb0853
* | | Merge "Add docs for running tests"Jenkins2016-04-212-0/+213
|\ \ \
| * | | Add docs for running testsscott2016-04-202-0/+213
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add documents for running unit and functional tests. Change-Id: I4616193dade844c5a774dd74aa1805452fd01a9e Closes-Bug: #1560615
* | | | Updated from global requirementsOpenStack Proposal Bot2016-04-202-4/+4
| |/ / |/| | | | | | | | Change-Id: Ic543aab4ffef00c706481d62f0df4488547bad82
* | | Support api-microversionsscottda2016-04-1916-68/+645
| | | | | | | | | | | | | | | | | | | | | | | | | | | Changes to cinderclient to use microversions. Implements: blueprint api-microversion-support-for-cinderclient api-microversion-support-for-cinderclient Change-Id: I840a1162b88e8ff36fa3fc4e1d6b9317104df3e0
* | | Merge "Add /v3 endpoint support for cinderclient"Jenkins2016-04-1949-4523/+5255
|\ \ \
| * | | Add /v3 endpoint support for cinderclientscottda2016-04-1849-4523/+5255
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add support for Cinder API /v3 endpoint. A couple of unit tests for /v3 endpoint were added to v3/test_shell.py to ensure that the v3 shell works, and to also test that modules work with: from cinderclient.v2.availability_zones import * syntax. Change-Id: I6ae0ada221bebb4ab1850d9c99b10fcbb585201f Implements: https://blueprints.launchpad.net/python-cinderclient/+spec/add-v3-endpoint-support
* | | | Merge "Change extension module naming to a shorter one"Jenkins2016-04-191-1/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Change extension module naming to a shorter oneIvan Kolodyazhny2016-01-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch changes extension suffix from 'python_cinderclient_ext' to 'cinderclient_ext' to get more shorter module names. Change-Id: Id78e05646d2bc4fda758710eb630dca5eefa457f
* | | | Merge "Add pylint tox env"Jenkins2016-04-124-0/+312
|\ \ \ \
| * | | | Add pylint tox envEric Harney2016-04-074-0/+312
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Run pylint with $ tox -e pylint (Copied from Cinder with minor changes.) Change-Id: I9d7f03c44c6da7515b433d6c87f3a1645184d491
* | | | | Merge "Graduate to oslo.i18n and cleanup incubator usage"Jenkins2016-04-129-847/+69
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Graduate to oslo.i18n and cleanup incubator usageRonald Bradford2016-03-249-847/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Graduate from Oslo Incubator to oslo.i18n library. Cleanup of unused Oslo Incubator utils. Added optional enable_lazy() usage. Implements: blueprint graduate-oslo-i18n [1] https://blueprints.launchpad.net/oslo-incubator/+spec/graduate-oslo-i18n Change-Id: I9c8db9487b554b637a41620c858a7e7abf802879
* | | | | Merge "Add tests for delete type by name"Jenkins2016-04-071-0/+5
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Add tests for delete type by nameGorka Eguileor2016-03-291-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On the patch that added multiple volume type deletions and deleting volume types by name we didn't have a test for the volume type deletion by name. This patch adds this test. Change-Id: Id3bd86eb5af95007ed5c0d6e0a936af3babcc5fb
* | | | | Merge "Fix api v2 so that you can delete more than one volume_type at a time."Jenkins2016-03-313-5/+32
|\ \ \ \ \ | |/ / / /
| * | | | Fix api v2 so that you can delete more than one volume_type at a time.zwei2016-03-293-5/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This path is also allowing us to delete them by name or ID instead of only by ID. eg: cinder --os-volume-api-version 2 type-delete test01 test02 Closes-bug: #1554794 Change-Id: I54faad2c5b60ab69f4b406310eb8059cf1e8cf76
* | | | | Merge "Fix wrong request url when retrieving multiple request"Jenkins2016-03-291-2/+3
|\ \ \ \ \
| * | | | | Fix wrong request url when retrieving multiple requestzwei2016-03-251-2/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the length of resource list is larger than osapi_max_limit. Cinder will return a "maker url" which marks index of resources already returned to client. _cs_request() function fo HTTPClient will link the endpoint url of cinder and this "marker url" together then use this wrong linked url to request the server side, which cause a NotFound exception. Closes-bug: #1560862 Change-Id: I416b63758373dc8ad29818d6952d1e9dd58d05f2
* | | | | Merge "Removed Extra code"Jenkins2016-03-244-19/+0
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Removed Extra coderoot2016-03-144-19/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some extra code is present like unused variables, unreachable code after return statement etc. Same is removed. Change-Id: Ifca88a19625c56ed520321ecbdd91739a304be8e
* | | | | Merge "Add docstrings for chessecake methods"Jenkins2016-03-221-0/+3
|\ \ \ \ \
| * | | | | Add docstrings for chessecake methodsJohn Griffith2016-03-081-0/+3
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We missed docstrings on freeze, thaw and failover replication v2.1 methods. This adds them. Change-Id: If68a674c66b4f18e80432d1cef269b02813fcc37
* | | | | Merge "Revert "Cleanup for Replication v2: remove 'replication-promote'""Jenkins2016-03-215-0/+33
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Revert "Cleanup for Replication v2: remove 'replication-promote'"Sean McGinnis2016-03-165-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Forgot this was a replication v1 command. We need to keep v1 support around in the client for users with a new client and old service. This reverts commit 9685009da2311d871bd90c7705d842dac7e7ed53. Change-Id: Ifad5ddb021d98b0fc973bc9ea478baccc60f912f
* | | | | Merge "Keep py3.X compatibility for urllib"Jenkins2016-03-173-12/+3
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Keep py3.X compatibility for urllibLiuNanke2016-01-143-12/+3
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use six.moves.urllib.parse instead of urllib. For urllib2 compatibility, it need to sync policy module from oslo first. Then six.moves.urllib.request can replace urllib2. Change-Id: Ia351cd3088dd64fd8c8ce4ca8b9a5889c574888e Partial-Bug: #1280105
* | | | Merge "Revert "Cleanup for Replication v2: remove 'replication-reenable'.""Jenkins2016-03-165-0/+32
|\ \ \ \
| * | | | Revert "Cleanup for Replication v2: remove 'replication-reenable'."Sean McGinnis2016-03-165-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Forgot this was a replication v1 command. We need to keep v1 support around in the client for users with a new client and old service. This reverts commit eda73c3b27159de97df132b53264cd8a620b4b95. Change-Id: Id46abbf668e6cce600ca3c6ab37133e267ba8346