summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Update version to 1.1.1 in index1.1.1John Griffith2014-09-231-0/+4
| | | | | | Prep for new tag Change-Id: I49f8f7fcc2647dc09e38fee1fa5068df714a8d83
* 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 "Update index file for release of 1.1.0"1.1.0Jenkins2014-09-121-0/+23
|\
| * Update index file for release of 1.1.0John Griffith2014-09-101-0/+23
| | | | | | | | Change-Id: I75bbf1cf88fb516409f0ed30b962af7e5ac6236e
* | 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 "Updated from global requirements"Jenkins2014-08-292-5/+5
|\ \ \
| * | | Updated from global requirementsOpenStack Proposal Bot2014-08-282-5/+5
| | | | | | | | | | | | | | | | Change-Id: I8d7b21b4ac55e9f02f067e31ad09a99b95180c4e
* | | | 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-255-251/+37
| | | | | | | | | | | | | | | 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 "Update theme for docs"Jenkins2014-08-042-2/+3
|\ \ \ \ \
| * | | | | Update theme for docsSteve Martinelli2014-07-242-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current developer docs theme used is out of sync with the other openstack projects. This patch will update the docs to provide a more consistent look and feel when using developer docs Change-Id: I9fd11b61d4404ba2627569074dc86ee0ddf0f96a
* | | | | | 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 "Add a tox job for generating docs"Jenkins2014-08-032-0/+5
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | |
| * | | | | | Add a tox job for generating docsSteve Martinelli2014-07-242-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a tox job to make it easier for developers to generate their own docs before submitting a patch. Change-Id: Icbfd5ee5556a9c344ebe9ca7f4b579a87ad2d01b
* | | | | | | 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
|\ \ \ \ \ | |/ / / / |/| | | |