summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge "better format remote IDs for identity providers"Jenkins2015-11-152-7/+8
|\
| * better format remote IDs for identity providersSteve Martinelli2015-11-122-7/+8
| | | | | | | | | | | | | | | | | | remote-ids are a list, and we should format these values as such, rather than python representations of lists/arrays. Closes-Bug: 1478995 Change-Id: Ia6ced0fab2435b8cb486822c676c0dee32613abe
* | Merge "Split the vol_id from a dev mapping"Jenkins2015-11-132-0/+83
|\ \
| * | Split the vol_id from a dev mappingxiexs2015-11-112-0/+83
| |/ | | | | | | | | | | | | Add a split into the dev mapping to get the right vol_id. Change-Id: I1a7bf6351491b1321c5ca0fa4a27f29825400eaf Closes-Bug: #1514394
* | Add capability to update description of an IdPlin-hua-cheng2015-11-121-0/+48
| | | | | | | | | | Change-Id: I854067642bbfde6fdf84b22b9cc1de8afc7767c0 Closes-Bug: #1515815
* | Merge "Add project-name/-id validation for the OSC "openstack quota set""Jenkins2015-11-121-6/+20
|\ \
| * | Add project-name/-id validation for the OSC "openstack quota set"xiexs2015-11-041-6/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | The quota info would be set into DB, even though the project actually does not exist. This patch tried to add a validation to forbid this undesirable behavior. Change-Id: Ia2d8c96527820e25b074e6486d3f39c5ad7eae60 Closes-Bug: #1512638
* | | Merge "Fix the bug of "openstack console log show""Jenkins2015-11-121-0/+55
|\ \ \ | |_|/ |/| |
| * | Fix the bug of "openstack console log show"xiexs2015-11-091-0/+55
| | | | | | | | | | | | | | | | | | | | | | | | The behaviors are inconsistent while different negative line numbers specified. Change-Id: I2573f3e789f5603c896758971830ffc0b94c5e2b Closes-Bug: #1512263
* | | Merge "Add --owner to `image create`"Jenkins2015-11-121-2/+4
|\ \ \
| * | | Add --owner to `image create`NiallBunting2015-11-061-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds --owner to `image create`. This is backwards compatable with v1. Change-Id: I9e79cf880c91a1386419db729818d23dfe632179 Depends-On: I8d572a070bbb04dccdd051b8e0ad199c5754746e
* | | | Merge "Fix issue when displaying image_member"Jenkins2015-11-112-3/+9
|\ \ \ \
| * | | | Fix issue when displaying image_memberlin-hua-cheng2015-10-232-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | image_member doesn't have a _info attribute, glanceclient returns warlock object instead of a Resource object. Change-Id: If6e7c4bd404454bd6cbe8c111879c1afa1380211 Closes-Bug: #1509054
* | | | | Merge "Add compute service delete"Jenkins2015-11-112-0/+58
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add compute service deleteAlex Schultz2015-10-162-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds 'compute service delete <service>' to the openstack client. This is the equivalent of 'nova service-delete <id>' Change-Id: I69ef1cac72cbe125c2114f8e958e22350a70f367 Closes-Bug: #1503510
* | | | | Use fake server name instead of id when testing "server_name" param.Tang Chen2015-11-071-11/+11
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "server create" command takes a server name parameter. The server id is generated by openstack. When we intended to pass a server name to the unit tests of "server create" command, we passed server id. It won't be any problem because the fake server id is also a string, but we should pass a fake server name because we have a string to fake one. Change-Id: I9944f0ea2a6c457e4fad8215a54778bca08965ab
* | | | Import the module not the classSean Perry2015-11-041-5/+5
| |_|/ |/| | | | | | | | | | | | | | As requested during review 226922. Change-Id: Ic5222141e247ce33cf5dbee66667cee3040e1cc3
* | | Merge "Mark arguments for 'credential' commands as required"Jenkins2015-11-042-0/+116
|\ \ \
| * | | Mark arguments for 'credential' commands as requiredSean Perry2015-09-302-0/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to the [1], 'user_id', 'type', and 'blob' are all required arguments for 'credential set' but the code treats them as optional. Set the 'required' flag and remove logic supporting missing arguments. [1]: https://github.com/openstack/keystone-specs/blob/master/api/v3/identity-api-v3.rst#credentials-v3credentials "spec" Change-Id: I597c9616ad744385fc6dd92379feb03daec54458 Closes-Bug: #1418837
* | | | Merge "Trivial clean up: do not use plural form in command arguments."Jenkins2015-11-021-3/+3
|\ \ \ \
| * | | | Trivial clean up: do not use plural form in command arguments.Tang Chen2015-10-241-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is a general rule that we don't use plural form for the name of command arguments. But class DeleteServer() is still using "servers". So use "server instead". Change-Id: I2d76de14ec34b88547b9f728b41e9bd93b2a22c5
* | | | | Merge "Rename context.py to logs.py"Jenkins2015-10-301-32/+32
|\ \ \ \ \
| * | | | | Rename context.py to logs.pyTerry Howe2015-10-301-32/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At one point this file contained the context for logging, but the reason for its existence is now for logging. Implements: blueprint logging-migration Change-Id: I4ba42bbef97b09d31236ac8c01b6fb23827b8bee
* | | | | | Merge "remove url from v3 regions"Jenkins2015-10-292-75/+6
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | remove url from v3 regionsSteve Martinelli2015-10-212-75/+6
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the parameter "url" was removed from Keystone, it was only added for one release as part of an experimental support for adding service providers. BackwardsIncompatibleImpact Closes-Bug: 1506841 Change-Id: I7a62fbf1d9bfa8e6dd8d619e98c32b9860348d2e
* | | | | Merge "Support pagination params for flavor list"Jenkins2015-10-231-5/+15
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Support pagination params for flavor listliyingjun2015-10-211-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Missing 'marker' and 'limit' params for `openstack flavor list` shell command. It would be nice to have this when there are many flavors. Closes-bug: #1505874 Change-Id: I088ac5d24f0d7595f5cbce14f063e296a449eb26
* | | | | Merge "Move session and fixtures to keystoneauth1"Jenkins2015-10-207-7/+7
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Move session and fixtures to keystoneauth1Steve Martinelli2015-10-147-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in an effort to start consuming keystoneauth1, we can move our sesssion and fixture code over to use keystoneauth1 instead of keystoneclient. Change-Id: Ibcbd588ce2e3d864f87ff1eb6e1c3c071a1e06f6
* | | | | Merge "Follow-on for volume list - add tests, clean help"Jenkins2015-10-193-10/+340
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Follow-on for volume list - add tests, clean helpDean Troyer2015-10-163-10/+340
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add volume list tests for v1 (a copy of the v2 tests) * Converts volume v2 tests to use Identity v3 so domains can be tested * Add volume list (v2) tests for new options * Re-orders volume list options (both v1 and v2) to match * MArks the new volume list (v2) options as v2-only in doc Change-Id: I2181b2c48cfde2147d7d0ef135322df8a81e7ce8
* | | | Merge "Fix up object-store show commands"Jenkins2015-10-143-13/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Fix up object-store show commandsSteve Martinelli2015-10-103-13/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1) Change metadata to appear under a common 'properties' key, and use the utility to format them, this applied to object, account and container. 2) Clean up container and object output, which were setting the x-container-meta-owner property, but this is metadata only for the container, so it's pointless to have, removed it. 3) Container show was showing read/write ACLs and sync stuff, but these are not being returned by my swift by default, so I moved these to be checks, so we don't clutter the output. Change-Id: Ife7521fe9c2724035b06963c118bd6016ba2f5b5
* | | | Merge "Add test for role list --inherited"Jenkins2015-10-121-0/+33
|\ \ \ \ | |/ / / |/| | |
| * | | Add test for role list --inheritedDean Troyer2015-10-071-0/+33
| | | | | | | | | | | | | | | | Change-Id: I216ab6c8ac903720ec67870a5171ae57a8f293aa
* | | | Merge "image set should not show the resource"Jenkins2015-10-082-14/+3
|\ \ \ \
| * | | | image set should not show the resourceSteve Martinelli2015-10-062-14/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the rest of OSC set commands do not show the resource after it has been updated. unless the update fails then we report back a failure, otherwise the user should assume everything went fine. Change-Id: I2bd4188450c3853b4a1bc25f80fc9450cda32bdd
* | | | | Merge "Evaluate --inherited in role list"Jenkins2015-10-081-0/+5
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Evaluate --inherited in role listRudolf Vriend2015-10-061-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the --inherited option was not being passed into keystoneclient Closes-Bug: #1502822 Change-Id: I48170dc67b23cc9b0665b1e0f38118eea952f131
* | | | | Merge "Add tags to `image set`"Jenkins2015-10-073-6/+98
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Add tags to `image set`NiallBunting2015-10-063-6/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds --tag to the v2 version of `image set`. This is another step to compatability between the osc image api. Added merge of tags into existing tags and handling duplicates, and tests for same. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Change-Id: Ie800fcbf8bbc0978c54ace3278750a18023e8ce4
* | | | | Merge "Clean up Image v2 image set command"Jenkins2015-10-061-33/+174
|\ \ \ \ \ | |/ / / / | | / / / | |/ / / |/| | |
| * | | Clean up Image v2 image set commandDean Troyer2015-10-061-33/+174
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the Image v2 image set command meet at the intersection of the v1 image set command and the v2 image create command: * Add visibility to the deadopts list and remove the option * Put the options in the same order as v1 image set * Make the help text match * Add --properties * Move the additional options that do not appear in either v1 image set or v2 image create after --property as they are really pre-defined properties * Add tests for v2 image set to match v1 and then some * Put the SetImage class in v2/image.py in alphabetical order Change-Id: I102b914e8ad09a014f6fdd846c5766b6c2eaadb8
* | | Merge "cleanup account ids from container commands"Jenkins2015-10-053-0/+6
|\ \ \
| * | | cleanup account ids from container commandsSteve Martinelli2015-10-023-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | use a common function to determine account ID instead of different ways - depending on the response and command Change-Id: I95adc5dc7d5a82a2cffc570d1ded24d1fc754a11
* | | | Add tests for find_resource()Dean Troyer2015-10-011-0/+54
| |/ / |/| | | | | | | | | | | | | | | | | Add a couple of tests for utils.find_resource() for the odd resources and managers without resource_class. Change-Id: I2ed9b491d1361b5259b3a5f80b4fac787a7087c1
* | | Merge "Glance `image set` Resolve Fracturing"Jenkins2015-09-231-2/+1
|\ \ \
| * | | Glance `image set` Resolve FracturingNiallBunting2015-09-231-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently `image set` uses the new api, where other parts of osc the old api is used. This deprecates the v2 api in favour of the v1 to maintain the same commands across osc. However the functionality now remains there as people could now be using this functionality. This also adds the --unprotected argument, as in the previous version if --protected was not supplied it would just make the argument --unprotected without the users explicit consent. The patch also fixes the documentation for image set as it was outdated. Change-Id: I990d20332c80165102badef7ac94ddbeb7824950 Closes-Bug: 1498092
* | | | Add image create support for image v2Monty Taylor2015-09-211-0/+186
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have it for v1, but v2 is the future. There are two differences, things in v2 do not go into a properties dict, and the actual image data needs to get uploaded as a second step. Closes-Bug: 1405562 Co-Authored-By: Niall Bunting <niall.bunting@hp.com> Co-Authored-By: Sean Perry <sean.perry@hp.com> Change-Id: If7b81c4a6746c8a1eb0302c96e045fb0f457d67b
* | | Merge "Add tests for volume quota set"Jenkins2015-09-191-0/+45
|\ \ \ | |/ / |/| |
View Lorry Controller Status