summaryrefslogtreecommitdiff
path: root/cinderclient
Commit message (Collapse)AuthorAgeFilesLines
* Incorrect version for hacking package on stable/kiloYuriy Nesenenko2015-07-219-46/+50
| | | | | | | | | | | | python-cinderclient should follow global requirements because we have hacking>=0.10.0,<0.11 in global requirements and hacking>=0.8.0,<0.9 in python-cinderclient in stable/kilo. It causes issues with packaging and conflicts in stable/kilo env. It updates from global-requirements change: I3e0e288974d6497481d6ea2d1b1a74b0cebe3ec7 Closes-Bug: #1475654 Change-Id: Ia7e5cae5e32f68c8552e16755ec139488d893c27
* Enables debug mode for keystone session objectgit-harry2014-09-181-0/+3
| | | | | | | This fixes the lack of debug messages introduced by commit 84af49c. Change-Id: I7bf3e98c21f09bd4f3c2d5d3bc372f60d717c9d5 Closes-bug: #1370152
* Merge "Make <volume-types> required option for create cg"Jenkins2014-09-123-10/+8
|\
| * Make <volume-types> required option for create cgXing Yang2014-09-103-10/+8
| | | | | | | | Change-Id: I101ec89ab95cedd8b55247301c1c3b9cba2645c5
* | Merge "Use adapter from keystoneclient"Jenkins2014-09-093-143/+110
|\ \
| * | Use adapter from keystoneclientJamie Lennox2014-09-023-143/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | The keystoneclient provides an adapter which maintains the client state around the session. We should re-use this rather than copy it and it also means we will get new adapter functions and parameters as they become available. Change-Id: I05c0d650dcdd69f7e77a06563d735efe521a41ae
* | | Merge "Cinder Client for Consistency Groups"Jenkins2014-09-099-3/+641
|\ \ \ | | |/ | |/|
| * | Cinder Client for Consistency GroupsXing Yang2014-09-089-3/+641
| | | | | | | | | | | | | | | | | | | | | | | | This patch implements CLI commands for the Consistency Groups feature. Only snapshots for CGs will be implemented in phase 1. Change-Id: I447555fd8a92bceecf6f40be59030d65461e4cbb Implements: blueprint consistency-groups
* | | Merge "Quotaset update does not return result"Jenkins2014-09-084-6/+8
|\ \ \ | |/ / |/| |
| * | Quotaset update does not return resultJuan Manuel Olle2014-07-294-6/+8
| | | | | | | | | | | | | | | | | | | | | This patch adds the updated quota values to the output shell Closes-Bug: #1316474 Change-Id: I95bbefa5fdee316a3a3ba77a5b3067345da3344f
* | | Merge "Add client support in Cinder for volume replication"Jenkins2014-09-055-4/+74
|\ \ \ | |_|/ |/| |
| * | Add client support in Cinder for volume replicationRonen Kat2014-08-275-4/+74
| | | | | | | | | | | | | | | | | | | | | | | | Provide Cinder client support to manage volume replication. Cinder-specs available at https://review.openstack.org/#/c/98308/ Change-Id: Id60bacbcc113d42730e822b29b2fa78be94d3276 Implements: blueprint volume-replication
* | | Merge "convert availability zone tests to requests-mock"Jenkins2014-08-283-16/+98
|\ \ \
| * | | convert availability zone tests to requests-mockJamie Lennox2014-08-253-16/+98
| | | | | | | | | | | | | | | | | | | | Blueprint: http-mock-testing Change-Id: Ib02db9b9e24808d2f5eac42cf6f5b3c5ca123e9b
* | | | Merge "Fix order of arguments in assertEqual"Jenkins2014-08-2815-92/+92
|\ \ \ \ | |/ / / |/| | |
| * | | Fix order of arguments in assertEqualKIYOHIRO ADACHI2014-08-1815-92/+92
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some tests used incorrect order assertEqual(observed, expected). The correct order expected by testtools is... def assertEqual(self, expected, observed, message=''): """Assert that 'expected' is equal to 'observed'. :param expected: The expected value. :param observed: The observed value. :param message: An optional message to include in the error. """ The string length of the sum of the results of repr(expected) and repr(observed) is greater than 70, then, MismatchError message is changed, as below. Ex.: raise mismatch_error MismatchError: !=: reference = '_123456789_123456789_bar' actual = '_123456789_123456789_123456789_123456789_123456789' Change-Id: Id02ddfb2ca03df7f432cff67a7bed182cccc4924 Closes-Bug: #1259292
* | | | Convert snapshot tests to requests-mockJamie Lennox2014-08-256-24/+223
| |/ / |/| | | | | | | | | | | Blueprint: http-mock-testing Change-Id: I06eae25cda6de9c266d34947471675fd95359739
* | | Replace httpretty with requests-mockJamie Lennox2014-08-254-250/+36
| | | | | | | | | | | | | | | Blueprint: http-mock-testing Change-Id: I0cc96b54be6f32436e3c3b6a2edd0fc50e98d8e3
* | | Merge "Avoid extra lookups in extra-specs-list"Jenkins2014-08-232-28/+4
|\ \ \
| * | | Avoid extra lookups in extra-specs-listJordanP2014-08-052-28/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | extra-specs-list used to use the extra-specs cinder extension to get information on volume type extra specs. This information is publicly available through the volume-type API [1] thus we can avoid the extra-lookups of querying the extra-specs extension. As the extra-spec info is publicly available, remove the part of the docstring of cinder extra-specs-list that says 'admin only'. [1] http://docs.openstack.org/api/openstack-block-storage/2.0/content/GET_getVolumeTypes__v2__tenant_id__types_Volume_Types.html Change-Id: I9a8256a044511ac5233d8170fc57118ccaa1e217 Closes-Bug: 1343876
* | | | Merge "Support pagination for volume list"Jenkins2014-08-214-3/+99
|\ \ \ \ | |_|/ / |/| | |
| * | | Support pagination for volume listliyingjun2014-08-024-3/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since v2 api supports pagination, we should add this support in cinderclient too. Change-Id: I9d4cb9e192523c14cf7df043deabe156dbb8e216 Closes-bug: 1325773
* | | | Merge "Reuse Resource from oslo"Jenkins2014-08-161-85/+4
|\ \ \ \
| * | | | Reuse Resource from osloAndrey Kurilin2014-04-041-85/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Class Resource from oslo is equal to code from cinderclient. In the process of unification of the clients code we should reuse common functionality from Oslo. Related to blueprint common-client-library-2 Change-Id: If39e49ff739d659d145725966192acf81975c90c
* | | | | Merge "Use suitable assert"Jenkins2014-08-064-26/+26
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Use suitable assertliuqing2014-07-094-26/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace assertTrue(a in b) with assertIn(a, b) Replace assertTrue(a in [b, c]) with assertIn(a, [b, c]) Replace assertTrue(a is None) with assertIsNone(a) Replace assertTrue(a == b) with assertEqual(a, b) Replace assertFalse(a == b) with assertNotEqual(a, b) Change-Id: I10c3abce14953b97568a5ce9d45f63af22d71ac3
* | | | | Merge "Change "Connection refused" to "Connection error""Jenkins2014-08-031-2/+1
|\ \ \ \ \
| * | | | | Change "Connection refused" to "Connection error"Eric Harney2014-07-241-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The python-requests documentation indicates that the ConnectionError exception can be raised for reasons other than "connection refused". Let's just say "error" rather than implying the wrong type of failure. Change-Id: I2205e27c7c68164db430cdfdc71b57b3002b24be
* | | | | | Merge "Fix the return code of the command reset-state"Jenkins2014-08-034-18/+42
|\ \ \ \ \ \
| * | | | | | Fix the return code of the command reset-stateZhengguang2014-07-284-18/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the command reset-state return code is zero if it failed for any of volumes, it should be nonzero. Change-Id: Ic6da238861fd003375003399bc6def6bf860a015 Closes-Bug: #1339643
* | | | | | | Merge "Mask passwords in client debug output"Jenkins2014-08-022-1/+36
|\ \ \ \ \ \ \
| * | | | | | | Mask passwords in client debug outputJay S. Bryant2014-07-302-1/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change looks for the use of 'password' in the data that is sent and uses mask_password() to remove the actual password text. This change will prevent debug output that is being saved from saving passwords. A test case is added to verify that password output is being removed. Change-Id: I93bde838ea21101df08c0e824d9f9457ed2ad077 Closes-Bug: 1341735
* | | | | | | | Merge "sync latest strutils to python-cinderclient"Jenkins2014-07-303-136/+265
|\ \ \ \ \ \ \ \ | |/ / / / / / /
| * | | | | | | sync latest strutils to python-cinderclientJay S. Bryant2014-07-163-136/+265
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit syncs the latest strutils code into python-cinderclient. The motivation for this sync is to get access to the mask_password function that was added with commit cb5a804b . The sync touches both the strutils.py and gettextutils.py files. The detailed breakdown of the sync is as follows: Current HEAD in OSLO: ----------------------------------- commit 5fa2dae429a9e37dfd1a527eb3957cea57a3e8c4 Merge: 5fb12c2 0506d17 Author: Jenkins <jenkins@review.openstack.org> Date: Tue Jul 15 10:05:12 2014 +0000 Merge "cfgfilter has graduated, remove it" ----------------------------------- Changes to strutils.py (newest to oldest): ----------------------------------- cb5a804b Move `mask_password` to strutils 8a0f5678 Remove str() from LOG.* and exceptions fd18c288 Fix safe_encode(): return bytes on Python 3 302c7c80 strutils: Allow safe_{encode,decode} to take bytes as input bec3a5eb Implements SI/IEC unit system conversion to bytes Changest to gettextutils.py (newest to oldest): ----------------------------------- 3d90045d Backport code for i18n to check lazy at runtime de4adbc4 pep8: fixed multiple violations 9912e5df Add API for creating translation functions 6cc96d05 Fix test_gettextutils on Python 3 fd33d1ea Fix gettextutil.Message handling of deep copy failures 047b2e4e Change lazy translation to retain complete dict 6d55e26a Add support for translating log levels separately Change-Id: Icc38bc97e47c8236ccb82283c246bf266bc62929 Related-Bug: 1341735
* | | | | | | | Merge "Add tenant uuid when running cinder list --all-tenants"Jenkins2014-07-303-6/+28
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | |
| * | | | | | | Add tenant uuid when running cinder list --all-tenantsliyingjun2014-06-273-6/+28
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the tenant uuid to the output when running cinder list --all-tenants, since this is an admin command any way, it would help to list the tenant UUID so we do not have to run cinder show to see the tenant uuid when we have further queries to run. Change-Id: I661789e957fa00947c4d5595f7e0515c27963735 Closes-bug: 1333257
* | | | | | | Move debug logging to shellJamie Lennox2014-07-292-6/+9
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding channels to a logger is not the responsibility of a library. This sort of thing should be handled by an application so move the logging over to the shell. Change-Id: Ie11571d428913eba1aae5aa42a6e925228ba6808
* | | | | | Merge "Retry when connection to cinder is refused"Jenkins2014-07-252-2/+23
|\ \ \ \ \ \ | | |_|/ / / | |/| | | |
| * | | | | Retry when connection to cinder is refusedXu Chen2014-07-242-2/+23
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, cinder client does not retry when connections to cinder service is refused. There are many legitimate scenarios under which retry should be attempted: 1) cinder service being restarted; 2) cinder service is running on multiple API nodes behind a LB, which might be temporarily overwhelmed or being maintained. In any scenario, retry with a backoff timer does not seem to hurt. Change-Id: I3c290c59fa67262c4a3473815b4380ee39e24332 Closes-Bug: 1347843
* | | | | Merge "Add commands for managing and unmanaging volumes"Jenkins2014-07-245-0/+243
|\ \ \ \ \
| * | | | | Add commands for managing and unmanaging volumesGeraint North2014-07-245-0/+243
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add manage and unmanage commands. Cinder code: https://review.openstack.org/#/c/72501 See also adding support for "bootable" flag for volume manage, and changing the LVM backend to use the source-volume-name key as part of the existing-ref structure, to make the CLI easier to use: https://review.openstack.org/#/c/108488/ Implements: blueprint add-export-import-volumes Change-Id: I27d0d3396d80063a51b0fe56d2d3c92931fa9c6c
* | | | | Merge "Optional size parameter for volume creation"Jenkins2014-07-244-9/+95
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Optional size parameter for volume creationJuan Manuel Olle2014-07-084-9/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch makes optional the parameter size for creating a volume from a snapshot or other volume. The parameter is still required for other kind of volume creation. Change-Id: I75dcf0e647f6e82a2407bcf23ef359f5f87323e8 Closes-Bug: #1304463
* | | | | Merge "Add set-bootable command"Jenkins2014-07-168-0/+50
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add set-bootable commandHiroyukiEguchi2014-04-028-0/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bootable Status is set to "True" automatically when user create a volume from a image. But user have to set bootable status manually when creating a bootable volume manually. blueprint add-bootable-option this commit is related to https://review.openstack.org/#/c/84057/ Change-Id: Ib4a44334001b7b9e3b896c2e44607c414e75e952
* | | | | Merge "Remove dependent module py3kcompat"Jenkins2014-07-114-72/+5
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Remove dependent module py3kcompatllg82122014-02-144-72/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Module py3kcompat was removed from oslo-incubator, we can use six directly. * Sync commit of removing py3kcompat from oslo * use six replace usage of py3kcompat Change-Id: Ifabce6866cf82f81a33924ea586987388afa589b Closes-Bug: #1280033
* | | | | Fix version discovery and auth_pluginsHaneef Ali2014-07-014-36/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | V3Client support added version discovery and session supports. Most of the external auth system doesn't support this. This fix bypasses version discovery if the idenity service doesn't support that. Session is used only if no external auth plugin is used Change-Id: Ia84a2ad45940d35c5ef740727f5f7477c72ea9d4 Closes-Bug: #1333961
* | | | | Merge "Remove deprecated command-line args"Jenkins2014-07-011-58/+14
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Remove deprecated command-line argsMatt Fischer2014-06-261-58/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some command-line args were deprecated in Diablo and slated for removal in Folsom. They're being removed in Juno instead. Change-Id: I02339ba08c997fecca5a6639382360962e668bfd