summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Update release notes for 1.2.0 and 1.2.11.2.1Mike Perez2015-05-061-0/+80
| | | | | | Prepping for tag. Change-Id: I8ae5e5a0e7ea140d84a0838ffd2ce0c08fcc6aad
* Merge "Remove "OPTIONAL:" from optional argument help text (v2)"Jenkins2015-05-061-6/+6
|\
| * Remove "OPTIONAL:" from optional argument help text (v2)Eric Harney2015-04-221-6/+6
| | | | | | | | | | | | | | | | | | | | | | All arguments starting with '--' are optional and listed under the 'Optional arguments:' header by argparse. Remove the extra note shown for each argument. This mirrors the previous change which cleaned this up for v1: 811af8f Remove "OPTIONAL:" from optional argument help text Change-Id: Ie8e9544c1544ff5403f1c29e83d528d896957899
* | Merge "Updated from global requirements"Jenkins2015-05-061-2/+2
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2015-05-041-2/+2
| | | | | | | | | | | | Change-Id: Ie0fbf5656185fb39fb2c3199fbd4ce59fc1dd4c4
* | | Merge "Add CLI read-only functional tests"Jenkins2015-05-066-1/+151
|\ \ \
| * | | Add CLI read-only functional testsJohn Griffith2015-05-046-1/+151
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First pass at functional tests in cinderclient. This patch just takes some of the basic CLI tests from tempest and moves them into the cinderclient functional tests. These are read-only tests, and just do simple field checks on the tables generated by the basic list commands. Note: to run use "tox -efunctional", this requires that you have a full cinderclient env and credentials are set. You may be either running this locally (say in a devstack env) or you may have a remote cloud handy that you can just source the credentials for and run it that way. Change-Id: I2f09a63be265d6a74cb103d80579068b9ab66bf4
* | | Change --force parameter into booleanyatin karel2015-05-042-11/+40
|/ / | | | | | | | | | | | | | | | | Change --force parameter into boolean, now user can use it as boolean, without specifying True or False with it. It is still backward compatible with --force True|False. This can be deprecated in future. Closes-Bug: #1244453 Change-Id: Id245af1fa0db27912439a420c37aefca530894ba
* | Add version removal rule to stop discovery warningMike Perez2015-04-292-3/+11
| | | | | | | | | | | | | | | | | | | | | | Cinder recently added version discovery leveraging Keystone Client. For cases where the service catalog still contains version numbers, this can result in Keystone attempting to do discovery at the base url with the version number and giving warnings. This will set a version removal rule so Keystone can find the correct base url. Change-Id: I71432468fea8bf1e50f180ab7f6dd69ee9aaa7e6 Closes-Bug: #1448244
* | Merge "Support pagination param limit in volume list in V1"Jenkins2015-04-283-2/+14
|\ \
| * | Support pagination param limit in volume list in V1Anton Arefiev2015-04-223-2/+14
| |/ | | | | | | | | | | | | | | Since v1 api supports limit param in volume list, cinderclient should support it too. Change-Id: I60ffe0190a61dee354de5bc60ea3210c666bf6f2 Closes-Bug: #1447162
* | V2:cinder create --image option doesn't workPranaliDeore2015-04-272-1/+10
| | | | | | | | | | | | | | | | | | | | | | In create volume help, --image optional parameter is allowed to be passed to create volume, but even after passing valid image ID or NAME, volume is created without image. Assigned image parameter to the image_ref to fix this issue. Closes-Bug: 1447589 Change-Id: I21b8a51451a2caf230ca04103b2d573d1aa966b4
* | Merge "Remove duplicate find request in find_resource"Jenkins2015-04-242-17/+19
|\ \
| * | Remove duplicate find request in find_resourceGorka Eguileor2015-04-012-17/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | Removed duplicate manager.find method call using search argument display_name that affected any command that searches for a volume by display_name or any command that searches for non existent volumes. Change-Id: I86c99b41dd231b058ed3f6d5d78eafe35a111dec Closes-Bug: #1429102
* | | Merge "Create Consistency Group from CG Snapshot CLI"Jenkins2015-04-245-0/+96
|\ \ \
| * | | Create Consistency Group from CG Snapshot CLIXing Yang2015-04-225-0/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch addressed the following: * Added a new CLI to support creating a CG from a CG snapshot. cinder consisgroup-create-from-src [--cgsnapshot <cgsnapshot>] [--name <name>] [--description <description>] API patch: https://review.openstack.org/#/c/145952/ Partial-Implements: blueprint consistency-groups-kilo-update Change-Id: I03cce04e1d43ea72fc53a8bf614ba4f969f6e32e
* | | | Merge "Update README to work with release tools"Jenkins2015-04-231-0/+5
|\ \ \ \ | |/ / / |/| | |
| * | | Update README to work with release toolsDoug Hellmann2015-04-211-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The README file needs to have links to the project documentation and bug tracker in a parsable format in order for some of the release tools scripts to work (particularly the one that prints the release note email). Change-Id: I37e0acc5ed8e1af565359290fa622456901c735e
* | | | Kilo Consistency Group CLI updateXing Yang2015-04-223-2/+96
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch addresses the following: * Modify Consistency Group * Add CLI to update a consistency group, i.e., changing name and description, adding existing volumes to CG and removing volumes from CG. cinder consisgroup-update --name <name> --description <description> --add-volumes <uuid1,uuid2,...> --remove-volumes <uuid5,uuid6,...> Implements: blueprint consistency-groups-kilo-update Change-Id: I76317dc006c1f80e2e6c83218e9566f1d37d935e
* | | Merge "Add support to incremental backups in cinder"Jenkins2015-04-214-3/+25
|\ \ \ | |/ / |/| |
| * | Add support to incremental backups in cinderXing Yang2015-04-214-3/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a new option to "cinder backup-create" to support incremental backups. Specify --incremental to create an incremental backup. By default, it will be a full backup. Co-Authored-By: Xing Yang <xing.yang@emc.com> Implements: blueprint incremental-backup Change-Id: Id11ecd2982ea838181b4adaa5a742fc65f864acb
* | | 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