summaryrefslogtreecommitdiff
path: root/cinderclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Incorrect version for hacking package on stable/kiloYuriy Nesenenko2015-07-213-29/+35
| | | | | | | | | | | | 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
* Make <volume-types> required option for create cgXing Yang2014-09-101-2/+2
| | | | Change-Id: I101ec89ab95cedd8b55247301c1c3b9cba2645c5
* Cinder Client for Consistency GroupsXing Yang2014-09-084-1/+173
| | | | | | | | 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 "Add client support in Cinder for volume replication"Jenkins2014-09-053-1/+30
|\
| * Add client support in Cinder for volume replicationRonen Kat2014-08-273-1/+30
| | | | | | | | | | | | | | | | 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 "Support pagination for volume list"Jenkins2014-08-212-0/+34
|\ \ | |/ |/|
| * Support pagination for volume listliyingjun2014-08-022-0/+34
| | | | | | | | | | | | | | | | Since v2 api supports pagination, we should add this support in cinderclient too. Change-Id: I9d4cb9e192523c14cf7df043deabe156dbb8e216 Closes-bug: 1325773
* | 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 "Fix the return code of the command reset-state"Jenkins2014-08-032-0/+32
|\ \
| * | Fix the return code of the command reset-stateZhengguang2014-07-282-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | 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-021-0/+30
|\ \ \
| * | | Mask passwords in client debug outputJay S. Bryant2014-07-301-0/+30
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Add tenant uuid when running cinder list --all-tenants"Jenkins2014-07-301-0/+10
|\ \ \ | |_|/ |/| |
| * | Add tenant uuid when running cinder list --all-tenantsliyingjun2014-06-271-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "Retry when connection to cinder is refused"Jenkins2014-07-251-0/+20
|\ \ \
| * | | Retry when connection to cinder is refusedXu Chen2014-07-241-0/+20
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-243-0/+123
|\ \ \
| * | | Add commands for managing and unmanaging volumesGeraint North2014-07-243-0/+123
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-243-8/+83
|\ \ \ | |/ / |/| |
| * | Optional size parameter for volume creationJuan Manuel Olle2014-07-083-8/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-164-0/+14
|\ \ \ | |_|/ |/| |
| * | Add set-bootable commandHiroyukiEguchi2014-04-024-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Fix version discovery and auth_pluginsHaneef Ali2014-07-012-2/+43
| |/ |/| | | | | | | | | | | | | | | | | 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
* | Added support for keystone v3clienthaneef ali2014-06-174-6/+426
| | | | | | | | Change-Id: I7bbc74c9e73f36f942f5800a7af0da717da0bc64
* | Merge "Use real timestamps in authentication tests"Jenkins2014-06-132-6/+6
|\ \
| * | Use real timestamps in authentication testsJamie Lennox2014-04-042-6/+6
| |/ | | | | | | | | | | | | | | Using a non time string means it won't be correctly parsed by anything in keystoneclient trying to read those auth responses. Change-Id: I6db79d3cf7596044a281cc85de014691f81c1fd1 Blueprint: use-session
* | Merge "Ambiguous option error should not appear if Arg is SUPPRESS"Jenkins2014-05-081-2/+45
|\ \
| * | Ambiguous option error should not appear if Arg is SUPPRESSJuan Manuel Olle2014-04-161-2/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to backward compatibility there are arguments hidden. For example: --tenant-id and --tenant_id On those case the abbreviation mechanism if a user enter --tenant throw that there is an ambiguity but the helps only show --tenant-id as an optional argument. This change remove this ambiguity if one of them is hidden. Change-Id: I413f1ebafc68b3e02b5abadce1608e6d48207b01 Closes-Bug: #1252457
* | | Update help strings for cinder clientDiane Fleming2014-04-231-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make text consistent and clear. Add periods to end of sentences. Adjust test suite for change Closes-Bug: #1275881 Change-Id: I1dfde51636e8dc4b42e4c4810c27c1c4ac513d82
* | | Merge "When there is no error body return the HTTP reason"Jenkins2014-04-181-0/+1
|\ \ \
| * | | When there is no error body return the HTTP reasonRussell Sim2014-03-041-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Return the response reason when there is no message body to return. DocImpact Change-Id: Ia60da4f04b059a13fcbe0059bb42fd77f272d8aa Closes-Bug: 1248773
* | | | Merge "Fix usage of v1 and v2 availability zones"Jenkins2014-04-181-3/+3
|\ \ \ \
| * | | | Fix usage of v1 and v2 availability zonesekudryashova2014-03-141-3/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change usage of availability zones in client. V2 client should use appropriate version of manager of availability zones. Also change tests of v2 availability zones to test version 2 instead of one more time test v1 availability zones. Change-Id: Iec3863a105a20f876a303f789cc1edfafa58f4df Closes-Bug: #1290956
* | | | Merge "Accept deleting multiple snapshots in one shot"Jenkins2014-04-174-0/+14
|\ \ \ \
| * | | | Accept deleting multiple snapshots in one shotXavier Queralt2014-04-034-0/+14
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In bug 1241941 the delete and force-delete commands were updated to accept multiple volumes for deleting in the same call. This patch adds the same behaviour to the snapshot-delete command. Closes-Bug: #1284540 Change-Id: I5e4bf8641d7fd261fa24b03832bad08007e43c12
* | | | Merge "Add auth_plugin support to cinderclient"Jenkins2014-04-171-0/+346
|\ \ \ \
| * | | | Add auth_plugin support to cinderclientCory Stone2014-02-141-0/+346
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With CINDER_RAX_AUTH being rightfully removed, cinderclient is no longer compatible with Rackspace/any non-keystone auth. To fix this, I stole auth_system/auth_plugin from novaclient's implementation. See https://review.openstack.org/#/c/23820/. Change-Id: If5f84003f868ef02bb7eb7da67cf62018602e8f0 Closes-Bug: 1280393
* | | | Fix malformed encryption-type body in test casesPeter Hamilton2014-04-162-12/+10
| |_|/ |/| | | | | | | | | | | | | | | | | | | | This change addresses bug #1286261 and fixes two malformed encryption-type bodies in two test_create test cases. Change-Id: I308561c1d5a70812889a93922e13194597f670f1 Closes-Bug: #1286261
* | | Merge "Wrong hint key in volume create function"Jenkins2014-04-091-0/+18
|\ \ \
| * | | Wrong hint key in volume create functionStephen Ahn2014-02-211-0/+18
| |/ / | | | | | | | | | | | | | | | | | | | | | Hint key has to be 'OS-SCH-HNT:scheduler_hints' not 'scheduler_hints' and scheduler_hints key has to be located in same level with volume. Change-Id: Ic5888545d4b7ee6bb58a22840928ef1fc86fedfa Closes-Bug: #1276088
* | | Merge "CLI for disable service reason"Jenkins2014-04-046-0/+42
|\ \ \
| * | | CLI for disable service reasonJay Lau2014-03-166-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds cli option to allow users to give reason for service-disable. Also adds disabled reason as a column in service list, so any disabled service can be seen with reason. A recent cinder change that allows disable-log-reason allows users to provide reason for disabling service. This just adds the cli option for the method. Implements bp record-reason-for-disabling-service Change-Id: I90f4566764790eeb0c047c4a0fd0108050ab6ad9
* | | | Merge "replace assertTrue(isinstance) to assertIsInstance"Jenkins2014-04-046-24/+24
|\ \ \ \ | |_|_|/ |/| | |
View Lorry Controller Status