summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add ability to specify path var to testr"1.2.0Jenkins2015-04-211-1/+1
|\
| * Add ability to specify path var to testrJohn Griffith2015-04-201-1/+1
| | | | | | | | | | | | | | | | | | This keeps cinderclient/tests as the default test dir, but it adds the ability to set specific paths via the OS_TEST_PATH var, this way we can call on unit vs functional tests (once the functional tests are in place of course). Change-Id: Icdc703e4054af6d22f5886581efed07319ec0247
* | Merge "cinder list now supports filter by tenant"Jenkins2015-04-205-145/+186
|\ \
| * | cinder list now supports filter by tenantyatin karel2015-04-125-145/+186
| | | | | | | | | | | | | | | | | | | | | | | | | | | Admin can now filter volumes on the basis of tenant. Also updated help argument, to be consistent over the v1 and v2 shell module. Closes-Bug: #1341411 Change-Id: I647f0c9331ee4cd27acbfada6cdb9a56f80701b4
* | | Merge "Enable version discovery"Jenkins2015-04-207-33/+382
|\ \ \
| * | | Enable version discoveryMike Perez2015-04-207-35/+382
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The service catalog can now have the cinder endpoint x.x.x.x:8776 with service_type volume, without needing to specify a version in the endpoint. Keystone will do discovery of the root / GET of the Cinder API to discover the versions that can be talked to. This also provides backwards compatibility for the previous solution of having v1 enabled on service_type volume and v2 on service_type volumev2. Change-Id: Id0347f8370dbc8fd7fa8096cd5859e10b0c5d67c
* | | | Remove print statement in unit testJohn Griffith2015-04-201-2/+0
|/ / / | | | | | | | | | | | | | | | | | | Stray print statement in test_client, this removes it. Change-Id: I761da5067747d12d506165817fd5eef31a0ab792
* | | Uncap library requirements for libertyDoug Hellmann2015-04-162-4/+4
|/ / | | | | | | | | Change-Id: I64958cfb5234e1e4728e7336600e2389ef4469a6 Depends-On: Ib948b756b8e6ca47a4c9c44c48031e54b7386a06
* | Allow cinderclient to handle exception responsetpatil2015-03-312-1/+123
|/ | | | | | | | | | | | | | | | | For all cinder commands, the error message returned by the cinder service is gobbled by keystoneclient as raise_exc is enabled by default and it doesn't have the knowledge of how to interpret exception returned by the cinder service correctly. Set raise_exc to False explicitly in request method of cinderclient and pass it to the keystoneclient session request method so that keystoneclient raises exception only when the exception is raised by the keystone server else cinderclient should interpret the exception returned by the cinder service. Closes-Bug: 1431693 Closes-Bug: 1428764 Change-Id: I1160a2c998919b669c39a4b739e83f5bd9d8d235
* Move unit tests into test directoryJohn Griffith2015-03-2357-86/+86
| | | | | | | | | | | | This is the first step of moving functional testing into the project tree. This change just moves all of the unit tests to be under a dedicated tests/unit directory. Follow up patches will add the functional directory and start moving tempest CLI tests there. Change-Id: I9dc0b8f761676c1b3bbdeb03e2f44b35f75c693e
* bash_completion now shows only subcommands when subcommand is "help"yatin karel2015-03-171-6/+18
| | | | | | | | | Options and subcommands are filtered. Now Only subcommands are shown when subcommands is "help". Both subcommands and options are shown when subcommand is other than "help" Change-Id: Iad9def1b2909af33fd23b0c89559b8de30fa6547 Closes-Bug: #1399747
* Merge "cinder list now prints dash '-' when data is None"Jenkins2015-03-162-0/+17
|\
| * cinder list now prints dash '-' when data is Noneyatin karel2015-03-162-0/+17
| | | | | | | | | | | | | | | | cinder list used to print None when volume was created without name. Now it prints '-' dash when display_name is None Closes-Bug: #1422244 Change-Id: I195ccc37fe96dbb54a0460527fabf55146170bc7
* | Update to change name for volume type clientGloria Gu2015-03-163-7/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This continues to update the name for volume type: * Update a client method for volume type. update: to update an existing volume type's name * Update a command-line operations. type-update: (adminitrator only) to update a volume type name The corresponding cinder APIs change to update volume type name: https://review.openstack.org/#/c/140906/ Implements: blueprint volume-type-description Change-Id: I66adb7fd2a433e7443cb609046f64dbab3d9d4c7
* | Merge "cinderclient accepts arguments after metadata without -- separator"Jenkins2015-03-163-0/+172
|\ \ | |/ |/|
| * cinderclient accepts arguments after metadata without -- separatorrajiv2015-03-103-0/+172
| | | | | | | | | | | | | | | | Added a function, which parse arguments and add -- separator after metadata subcommand Change-Id: I907eb89209d549dcdad361d88352491110eaadb9 Closes-Bug: #1379486
* | Updated help on cinder reset-state cliyatin karel2015-03-064-22/+32
| | | | | | | | | | | | | | | | | | | | volumes also have 'in-use', 'attaching' and 'detaching' states. These were missing in cinder help for reset-state cli. I have added these in help. Partial-Bug: #1309086 Change-Id: I094a61dd4921bed83e97134a4bfbcdc7d0c9ad63
* | Merge "Adopt CLI sorting argument guidelines"Jenkins2015-03-024-25/+189
|\ \
| * | Adopt CLI sorting argument guidelinesSteven Kaufer2015-02-254-25/+189
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CLI sorting guidelines are defined in the cross project specs repository: [1] This patch deprecates --sort_key and --sort_dir and adds support for --sort in the 'cinder list' command with the following syntax: cinder list --sort <key>[:<direction>] The --sort parameter is comma-separated and is used to specify one or more sort keys and directions. Server-side work for sorting by multiple keys is handled in [2]. [1] https://github.com/openstack/openstack-specs/blob/master/specs/ cli-sorting-args.rst [2] https://review.openstack.org/#/c/141915/ Implements: blueprint cinder-pagination Change-Id: Ie8c2c62b8c129f647f85d66d5bcfe4c8f0f4eedb
* | | Merge "Make cinderclient metadata CLI output consistent"Jenkins2015-03-022-2/+2
|\ \ \
| * | | Make cinderclient metadata CLI output consistentThang Pham2015-01-132-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cinder metadata-update-all and metadata-show produces inconsistent output. The following patch makes the output of each command above consistent, where each metadata property will be printed on a separate table row. Change-Id: I42933bc6c0e4f925c19cfd4db6d1a205de06a73d Closes-Bug: #1366289
* | | | Add missing all-tenants option to transfer-listliyingjun2015-02-246-10/+98
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | cinder transfer-list is missing the --all-tenants option, which makes it inconsistent with 'cinder list'. Without this fix, when running as admin horizon will need to make multiple queries to cinder (one for each tenant present in the volume list) in order to display the transfers that exist for the volumes being shown. Change-Id: Ic4d72a54c450876ff24c64efd83678062c93a24e Closes-bug: 1351084
* | | 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
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
View Lorry Controller Status