summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add -d short option for --debug"1.3.0Jenkins2015-02-182-2/+2
|\
| * Add -d short option for --debugKallebe Monteiro2015-02-122-2/+2
| | | | | | | | | | | | | | | | | | | | A short option for --debug is a convenience. This commit adds the -d short option as a shortcut for --debug Glance already provides -d. Change-Id: Iefae6cee566f9c7d7375e89c3f8603f149e555ef Closes-Bug: #1421210
* | Merge "Fix volume_transfers import in v2"Jenkins2015-02-151-1/+1
|\ \ | |/ |/|
| * Fix volume_transfers import in v2Kallebe Monteiro2015-02-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixing the import of volume_transfers in v2 client. It was importing volume_transfers from v1. Even though at this moment the imported files are identical, in the future volume_transfers in v2 may change, and v2 client should use the v2 volume_transfers. Since the intent is for v1 to be deprecated in the future, we must be careful with these little details. Closes-Bug: #1420238 Change-Id: I41c385de32ccf66d23002a8e369848016fa2c790
* | Updated from global requirementsOpenStack Proposal Bot2015-02-051-1/+1
|/ | | | Change-Id: Ib1c572eae4c4039c6a6f5e39b0243ed0c263feb4
* Merge "Fix up help message for reset-state call"Jenkins2015-02-052-11/+18
|\
| * Fix up help message for reset-state callJohn Griffith2015-01-282-11/+18
| | | | | | | | | | | | | | | | | | | | | | There's some confusion about what the reset-state call does. It's not clear to some that this call simply changes the state of the object in the data-base. This patch just updates the help message to indicate more clearly that reset-state ONLY changes state in the DB. Change-Id: Ia389cfd8b1a821a356706201c009cf3c8dca8419
* | Merge "reset-state should warn that it is DB only"Jenkins2015-02-041-1/+8
|\ \
| * | reset-state should warn that it is DB onlyscottda2015-02-041-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The python-cinderclient command 'reset-state' will only change the state in the Cinder database, without affecting any connection/export of a volume from the driver backend to a Nova compute host/instance. The Doc string should explicitly state that this is the case, and Warn that it might result in an unusable volume, i.e. a volume is set to 'available' in the Cinder DB, but is still attached and therefore cannot be attached to another instance, nor detached. Change-Id: Iee66570dbb93776490ae8fb663a7f835dc496bc2 Closes-Bug: 1417273
* | | Merge "Don't use sessions if third party plugin is used"Jenkins2015-02-031-1/+2
|\ \ \
| * | | Don't use sessions if third party plugin is usedRobert Myers2014-12-091-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If an auth_plugin is used do not try to load the session from the default keystone providers. Change-Id: If2a87e23cabde006833de70e5c7aa066d95dbf50 Closes-Bug: #1380729
* | | | Merge "Add support for os-volume-type-access extension"Jenkins2015-02-038-9/+279
|\ \ \ \ | |_|/ / |/| | |
| * | | Add support for os-volume-type-access extensionMathieu Gagné2014-12-198-9/+279
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds the ability to manage volume type access: - Create non-public volume type - List volume type access - Add a project access - Remove a project access This change also adds the is_public flag to volume type list. Note: The volume type access extension is only implemented in the Cinder API v2. DocImpact: Add volume type access extension support Implements: blueprint private-volume-types Change-Id: Ife966120d9250be8d8149cdec9c1a53405d37027
* | | | Merge "Expose cinder's scheduler pool API"Jenkins2015-01-304-8/+127
|\ \ \ \
| * | | | Expose cinder's scheduler pool APIGary W. Smith2015-01-304-8/+127
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the scheduler pool API available to other clients. DocImpact Change-Id: I3f0e0c2ed806b5bfda3129ed2c5b210b9210daa5 Partially-Implements: blueprint get-volume-type-extra-specs
* | | | | Merge "Add tests for consistency groups and cgsnapshots"Jenkins2015-01-233-10/+91
|\ \ \ \ \
| * | | | | Add tests for consistency groups and cgsnapshotsMichal Dulko2015-01-223-10/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds missing unit tests for consistency groups and consitency groups snapshots. Tests cover update operation and various use cases of list operations. Change-Id: I8998a5c8643559cee83c8c0c81b616b5564c0a30
* | | | | | Merge "cinder list fails with 'name' sort key"Jenkins2015-01-232-4/+22
|\ \ \ \ \ \
| * | | | | | cinder list fails with 'name' sort keySteven Kaufer2015-01-222-4/+22
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The client restricts the sort keys that the user can supply. The 'name' key is allowed but is not the correct key for sorting by name, it should be 'display_name'. If 'name' is used then the client returns with the error 500 Internal Server Error. This patch will add support for mapping client sort keys (eg, 'name') to server sort keys (eg, 'display_name'), allowing the user to supply the 'name' key to sort by name. This patch also adds UT for the sort key and direction error cases when an invalid value is supplied. Change-Id: I0bdad6d61da83a3924a6b18678afe4722b5778d6 Closes-Bug: 1404020
* | | | | | Merge "Leverage openstack.common.importutils import_class"Jenkins2015-01-222-9/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Leverage openstack.common.importutils import_classAaron Rosen2015-01-122-9/+1
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch drops the import_class method from utils and instead leverages the one in openstack.common.importutils. Change-Id: I9f740b9941d477776597c4c4ab88a65a39fb4fb8 Closes-bug: 1365273
* | | | | Merge "Remove commented code in cinderclient/v1/volumes.py"Jenkins2015-01-201-5/+0
|\ \ \ \ \
| * | | | | Remove commented code in cinderclient/v1/volumes.pyZhiteng Huang2015-01-151-5/+0
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | migrate_volume_completion() function in Volume class was commented out when it was added. It hasn't been changed since then. This change remove it. Change-Id: Iad8c88e20268265f9063e2faa2a21514439150ad
* | | | | Merge "Allow CG quota to be showed and updated"Jenkins2015-01-164-7/+27
|\ \ \ \ \
| * | | | | Allow CG quota to be showed and updatedXing Yang2014-12-034-7/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch added the ability for CG quota to be showed and updated. - Added consistencygroups as a quota resource to be displayed. - Added an option for consistencygroups quota to be updated. Change-Id: I7e88eb426eb1733bce955b7a15b8bffdb1e8ee20 Closes-Bug: #1373766
* | | | | | Merge "v2 error message grammatical error"Jenkins2015-01-161-4/+4
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | v2 error message grammatical errorgit-harry2015-01-081-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes an error that occurs in a number of places in the v2 shell; the word 'the' is missing from a number of the error message strings. Change-Id: I6d1026c965f84c9c39fcfd175b5d5cbbbad5ef5c
* | | | | | Merge "Add command to show pool information for backends"Jenkins2015-01-135-0/+56
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Add command to show pool information for backendsMitsuhiro Tanino2015-01-055-0/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An admin-api extension to show pool info was supported at commit https://review.openstack.org/#/c/119938/. This change adds a command to show pool information for backends using the admin-api. This change also closes the gap for end users. Partial-Bug: 1403902 Change-Id: I20e0828c5403b73bc44d07eebf08e2aa2deb428a
* | | | | | Merge "Client output is not sorted by --sort_key"Jenkins2015-01-074-5/+70
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Client output is not sorted by --sort_keySteven Kaufer2014-12-224-5/+70
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The cinder client supports invoking the volumes REST API with the sort_dir and sort_key parameters. However, the client output table is always sorted by ID even though the REST API is returning in sorted order based on the sort key/direction provided. For example, the command below supplies the 'size' sort key but the output table is still sorted by ID: cinder list --sort_key size --sort_dir desc This fix contains: * Updates to the print_list utility to prevent any re-ordering so that the current object order is maintained * Updates to the shell to disable re-ordering if the user supplies sort parameters Change-Id: I4cc111a8bdd2b89158dfc4bb0d16fa6dbf36cc57 Closes-Bug: 1402846
* | | | | Merge "Fix incorrect variable name"Jenkins2015-01-062-1/+14
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix incorrect variable nameJamie Lennox2014-10-272-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | valid_versions does not exist here, it is obviously supposed to mean _VALID_VERSIONS. Closes-Bug: #1386232 Change-Id: I477baa7642feba72f80d884d6183512185b02cf1
* | | | | Merge "Use immutable arg rather mutable arg"Jenkins2014-12-213-4/+8
|\ \ \ \ \
| * | | | | Use immutable arg rather mutable argliuqing2014-07-033-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Passing mutable objects as default args is a known Python pitfall. We'd better avoid this. This commit changes mutable default args with None. Change-Id: I7817c92069d32d1926e185ca0d0ab14e738b8b3d
* | | | | | Merge "Add CONTRIBUTING.rst"Jenkins2014-12-201-0/+12
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Add CONTRIBUTING.rstliuqing2014-07-011-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no CONTRIBUTING.rst file, so i add it. Change-Id: I92150e79f76559f90dd0341fdd6f5b0fa2d54e32
* | | | | | Added type description for volume type clientGloria Gu2014-12-175-4/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch added client handling and unit tests for volume type description: * Added 2 client methods for volume type. default: to get the default volume type update: to upate an existing volume type to update description * Added 2 new command-line operations. type-update (adminitrator only) type-default * type-list should display description. * type-create should have an option for entering the description. The corresponding cinder APIs change volume-type-description: https://review.openstack.org/#/c/131871/ Implements: blueprint volume-type-description Change-Id: I2735d7050d90589d19f45e21096577febdcca8bb
* | | | | | Merge "Fix comment in tearDown()"Jenkins2014-12-121-2/+2
|\ \ \ \ \ \
| * | | | | | Fix comment in tearDown()Matt Thompson2014-07-231-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This minor change fixes two typos in the tearDown() method. Change-Id: Ie99c671cbfdaed01715046ea65d49e09114b1e08
* | | | | | | Merge "Remove cinderclient/tests from coverage report"Jenkins2014-12-121-1/+1
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Remove cinderclient/tests from coverage reportgit-harry2014-11-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently coverage reports on coverage of the tests themselves. This makes the percentage coverage higher than if they are excluded and therefore distorts the reports values for the codes as a whole. This commit updates the .coveragerc files to remove cinderclient/tests/* from the coverage report. Change-Id: If7f1765017f3f0d55975219b12ff7bf149146931
* | | | | | | Merge "List all the request items when the list is over osapi_max_limit"Jenkins2014-12-074-10/+91
|\ \ \ \ \ \ \
| * | | | | | | List all the request items when the list is over osapi_max_limitVincent Hou2014-12-044-10/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Convert the function _list into a loop function, which can retrieve the items from the next link till the limit or the end of items has been reached. This works for v2 only. So far, only volume list in v2 support limit. The limit parameter work for volume list in v2 only, but other list can extend it in future work. Change-Id: I011f0ed1a4ab639f67db6cae580d978c0b44c1bb closes-bug: #1342192
* | | | | | | | Workflow documentation is now in infra-manualJeremy Stanley2014-12-051-1/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace URLs for workflow documentation to appropriate parts of the OpenStack Project Infrastructure Manual. Change-Id: I58e362f5a94e2c47a103d6901464df38110d6383
* | | | | | | Merge "Add ability to create volume from image by image name"Jenkins2014-12-041-1/+10
|\ \ \ \ \ \ \
| * | | | | | | Add ability to create volume from image by image nameSean McGinnis2014-11-251-1/+10
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow on change from cinder code to all specifying an image by either its name or ID for creating a volume. This patch adds the --image option that will take either ID or name as input. The service then will attempt to locate the image to use and make sure there are not multiple images with the same name. If both --image-id and --image are used it will favor the more explicit ID. DocImpact Closes-Bug: 1377823 Change-Id: I77d58258144970fac47da6da6b7870f2931e1823
* | | | | | | Merge "client HTTPClient __init__ fails if auth_url None"Jenkins2014-12-022-1/+5
|\ \ \ \ \ \ \
| * | | | | | | client HTTPClient __init__ fails if auth_url NoneBill Arnold2014-09-092-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The initializer for the python-cinderclient HTTPClient class fails if passed a None for an auth_url. This patch modifies the inializer to not call rstrip if auth_url is None, matching the initalizers in python-novaclient and python-neutronclient Change-Id: I19dd6911816639a0e0d6175ba910e9777a4b5981 Closes-bug: #1358926
* | | | | | | | Merge "Use newer features from keystoneclient"Jenkins2014-12-021-28/+4
|\ \ \ \ \ \ \ \