summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Add project param in LimitList parser3.17.0Vishakha Agarwal2018-11-081-1/+2
| | | | | | | | | | | | | when doing openstack limit list --project xyz_id, CLI raising error unrecognized arguments, whereas in api-ref document [1], user can pass project_id as query param.This addresses the above issue, by adding param --project in parser of LimitList. [1] https://developer.openstack.org/api-ref/identity/v3/index.html Change-Id: If4644cc99a3803f61f4a688b828aeb73977fc0dd Closes-Bug: #1798744
* Merge "Handle not having cinderclient.v1 available"Zuul2018-10-152-13/+2
|\
| * Handle not having cinderclient.v1 availableSean McGinnis2018-10-122-13/+2
| | | | | | | | | | | | | | | | | | | | | | The Cinder v1 API was deprecated several years ago and may be removed from python-cinderclient in the near future. To handle the case where v1 is no longer present, this updates cinderclient initialization to work without it and give an appropriate error if v1 is requested with a version where it is no longer available. Change-Id: I277d7b48b8ad4cce383ec3722f8117938378f615 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Merge "Allow endpoint filtering on both project and project-domain"Zuul2018-10-121-0/+41
|\ \
| * | Allow endpoint filtering on both project and project-domainJulie Pichon2018-10-091-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The --project and --project-domain flags are currently mutually exclusive for listing endpoints, however the --project-domain argument is supposed to help with filtering projects with colliding names. They should be allowed together. Story: 2004018 Task: 27007 Change-Id: I7340e01f509e3515f07cb46f175fb603f1ce8b67
* | | Merge "Add volume backend pool list command"Zuul2018-10-122-0/+132
|\ \ \
| * | | Add volume backend pool list commandSean McGinnis2018-10-092-0/+132
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds an equivalent for "cinder get-pools" with "volume backend pool list" and "cinder get-pools --detail" with "volume backend pool list --long". Story: 1655624 Task: 136949 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com> Change-Id: I826c9946ffe11340d44ad57914f72fc2a72b6938
* | | | Merge "Add volume backend capability show command"Zuul2018-10-122-0/+134
|\ \ \ \ | |/ / / | | | / | |_|/ |/| |
| * | Add volume backend capability show commandSean McGinnis2018-10-092-0/+134
| |/ | | | | | | | | | | | | | | | | | | | | Adds and equivalend for "cinder get-capabilities" command to show the capabilities supported by a Cinder backend. Story: 1655624 Task: 26947 Change-Id: I38686a26cd503e45ce0102705a6632994ef10274 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Add --property option to 'server rebuild' commandRuby Loo2018-10-051-0/+21
|/ | | | | | | | | | | | Add '--property' option to the 'server rebuild' command, to provide the ability to specify properties of the rebuilt instance. This is equivalent to the '--meta' option of the compute's 'nova rebuild' command. Change-Id: I25ea6622e970416090109316e1e28fab8b0b3f07 Story: #2003979 Task: #26922
* Fix 'project purge' deleting wrong project's servers and volumesJulie Pichon2018-09-141-12/+18
| | | | | | | | | | | | | | Project purge would delete the servers and volumes for the project the user is currently authenticated for, regardless of the --project flag. Note: This change means that no server at all will be deleted if the logged in user doesn't have the get_all_tenants permission set in the Nova policy (default: admin_api). This doesn't appear to be an issue with Cinder as the default rule appears to be admin_or_owner. Change-Id: If1c54e24e1482438b81c3c32fd5fc9fdd7a7be04 Story: 1747988 Task: 13854
* Deprecate volume create --project and --user optionsAlan Bishop2018-08-101-84/+10
| | | | | | | | | | | | | | | | | | Cinder's volume create API does not support overriding the project_id and user_id, and it silently igores those API inputs. Cinder always uses the project and user info in the keystone identity associated with the API request. If a user specifies the --project or --user option, the volume create is aborted and a CommandError exception is raised. This prevents a volume from being created, but without the desired project/user values. A user wishing to specify alternate values can still do so using identity overrides (e.g. --os-username, --os-project-id). Story: 2002583 Task: 22192 Change-Id: Ia9f910ea1b0e61797e8c8c463fa28e7390f15bf9
* Merge "Add DNS support to floating IP commands"Zuul2018-08-081-0/+12
|\
| * Add DNS support to floating IP commandsJens Harbott2018-08-061-0/+12
| | | | | | | | | | | | | | | | | | | | Add the DNS domain and name options to the ``floating ip create`` command. Also add these two columns to the output of the ``floating ip list --long`` command. Change-Id: Id4cb18b51b252f19b87b24ec5d77183771189d17 Story: 1547736 Task: 13114
* | Merge "Add dns-domain support to Network object"Zuul2018-08-072-0/+13
|\ \ | |/
| * Add dns-domain support to Network objectPierre Hanselmann2018-04-232-0/+13
| | | | | | | | | | | | | | | | | | | | | | Add "dns-domain" parameter to Network class. Also check backend extensions and send an error message in case of an argument (like dns-domain) is sent and the extension is missing (dns-integration in this case). Change-Id: I7303658c27d9b9f2d8381ccea0b29e96909cab54 Closes-Bug: 1633214 Partial-Bug: 1547736
* | Fix broken gate jobsFan Zhang2018-08-077-160/+154
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch aims at fixing the broken gate jobs because of cinder and glance patches [1], [2], [3] and [4]. * Remove parameter `--source-replicated` to drop volume replication v1 support * Address some timing issues with volume transfer requests * Only run Image v1 tests when the test cloud has v1 available * Get tolerant of unexpected additional attributes being returned in Image data [1].https://review.openstack.org/#/c/586293/ [2].https://review.openstack.org/#/c/532503/ [3].https://review.openstack.org/#/c/533564/ [4].https://review.openstack.org/#/c/578755/ Co-Authored-By: Dean Troyer <dtroyer@gmail.com> Co-Authored-By: Monty Taylor <mordred@inaugust.com> Depends-on: https://review.openstack.org/588664 Change-Id: I2a785750e92155185d3344e6116c7f5c6fdd3cbe Signed-off-by: Fan Zhang <zh.f@outlook.com>
* | Merge "Don't sent disk_over_commit if nova api > 2.24"Zuul2018-07-251-0/+35
|\ \
| * | Don't sent disk_over_commit if nova api > 2.24Artom Lifshitz2018-07-241-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In API microversion 2.25 Nova removed the disk_over_commit parameter to the live migration server action. This patch makes sure that we don't include it in our request if we're running with 2.25 or higher. Story: #2002963 Task: #22966 Change-Id: I1bbdd33be96d82422a05982508e370237c3560f3
* | | Merge "Add command to show all service versions"Zuul2018-07-251-0/+31
|\ \ \
| * | | Add command to show all service versionsMonty Taylor2018-07-231-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Knowing what services and what versions of those services exist on a cloud isn't always a spectacular experience. Add a command that will use get_all_version_data from keystoneauth to produce a report of the available services and the version info for each service. Depends-On: https://review.openstack.org/584944 Change-Id: I84751c175d0c5f6d857a5473d2db6d5f1b41f946
* | | | Merge "Support --community in openstack image list"Zuul2018-07-251-0/+29
|\ \ \ \
| * | | | Support --community in openstack image listNobuto Murata2018-07-241-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "--community" was added to "image create" and "image set" previously, but was missed in "image list". Change-Id: I959fdd7f67ae62c8326659ce52389228152ec019 Story: 2001925 Task: 14453
* | | | | Merge "Implement support for project limits"Zuul2018-07-254-0/+642
|\ \ \ \ \
| * | | | | Implement support for project limitsLance Bragstad2018-07-234-0/+642
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit let's users manage limits via the command line. bp unified-limits Change-Id: I7c44bbb60557378b66c5c43a7ba917f40dc2b633
* | | | | | Merge "Implement support for registered limits"Zuul2018-07-254-0/+754
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Implement support for registered limitsLance Bragstad2018-07-234-0/+754
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds support for users to manage registered limits via the command line. bp unified-limits Depends-On: https://review.openstack.org/#/c/574391/ Change-Id: Id8377363f7a3248b45aeeba21d2acc02684a0305
* | | | | | Merge "compute: host: expand kwargs in host_set() call"Zuul2018-07-251-4/+3
|\ \ \ \ \ \
| * | | | | | compute: host: expand kwargs in host_set() callBenoƮt Knecht2018-07-201-4/+3
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `host_set()` expects `status` and `maintenance_mode` as keyword arguments, but in `SetHost.take_action()`, it is called without expanding the keyword arguments. So it's called as ``` host_set(host, {'status': 'enable'}) ``` instead of ``` host_set(host, status='enable') ``` Change-Id: If0b37ac60091161a892bfc694fce31a988f66005 Task: 23023
* | | | | | Merge "Support filtering port with IP address substring"Zuul2018-07-251-0/+18
|\ \ \ \ \ \
| * | | | | | Support filtering port with IP address substringHongbin Lu2018-06-291-0/+18
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | Change-Id: I9559f1c0a6db943705bd32aefb60d7ea7054dd1b Related-Bug: #1718605
* | | | | | Merge "Fix error with image show when image name is None"Zuul2018-07-241-1/+2
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Fix error with image show when image name is NoneJake Yip2018-07-231-1/+2
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Need to bump osc-lib to 1.10.0 Closes-Bug: #1736696 Depends-On: I2aab5cc1f550848bda2b90ef7ef9a60f07b88996 Change-Id: I7420204f28d36529354e5671bd88587d9b15bb06
* | | | | Merge "Do not require port argument when updating floating IP"Zuul2018-07-241-6/+51
|\ \ \ \ \
| * | | | | Do not require port argument when updating floating IPyanpuqing2018-06-131-6/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When setting floating ip other properties, port argument is force to use. The patch modifies the command, when setting floating ip other properties, like tags, no need port argument. Change-Id: I908712c8913f32d3dd5fdfefe7347277d72f66de Story: 1751431 Task: 13865
* | | | | | Pass volume snapshot size to volume createMonty Taylor2018-07-231-1/+1
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When creating a volume from a snapshot, the size parameter is required and type is checked. Since we have to pass something and it needs to be a valid data type (None is not acceptable) grab the size from the snapshot object and pass it. Change-Id: Ie23e3d23828919234e40336b5c65b22e140d337c
* | | | | Merge "Slow down and retry aggregate create/delete to lessen race"Zuul2018-07-131-0/+18
|\ \ \ \ \
| * | | | | Slow down and retry aggregate create/delete to lessen raceDean Troyer2018-07-061-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I676894c1bfd3156313d88e6457250b9ff226118b Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | | Skip calls to glance and nova when got no serversPavlo Shchelokovskyy2018-07-091-0/+19
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | save (potentially many) HTTP calls to Glance API for image list and a call to Nova API for flavor list when the server list actually returned no servers. Change-Id: I93a56138c50b82fb4dce67a2f788107f71c5f423 Story: #2002039 Task: #19681
* | | | | Merge "Retry floating IP tests"Zuul2018-07-061-4/+36
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Retry floating IP testsDean Troyer2018-07-061-4/+36
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | test_server_attach_detach_floating_ip() has a test for server add/remove floating IP that seems to be racy, add a retry loop to let neutron and nova do their thing before calling it bad. Change-Id: I999a0d7dae1706d746053bafb7ab4e3b791d0042
* | | | Merge "Adding api_version to FakeApp"Zuul2018-06-291-0/+1
|\ \ \ \
| * | | | Adding api_version to FakeAppTelles Nobrega2018-06-241-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to use app.api_version['<some_service>'] we need to have the api_version on FakeApp otherwise the test will fail. Patch that will benefit from this: https://review.openstack.org/#/c/572536/ Change-Id: Ibb3c548daf2a62b6c4aefb5e257372dd5c56521e
* | | | | Merge "Add ability to filter image list by tag"Zuul2018-06-292-7/+36
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add ability to filter image list by tagAdam Harwell2018-06-282-7/+36
| | | | | | | | | | | | | | | | | | | | Change-Id: I2e222d3e69df9d8d7cd472663caaee31bedd848c
* | | | | Merge "Compute: Add description support for flavor"Zuul2018-06-282-3/+134
|\ \ \ \ \
| * | | | | Compute: Add description support for flavortianhui2018-06-202-3/+134
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Fan Zhang <zh.f@outlook.com> Change-Id: I0dc80bee3ba6ff4ec8cc3fc113b6de7807e0bf2a Story: 2002196 Task: 21681
* | | | | Merge "Network: Add tag support for security group"Zuul2018-06-252-4/+171
|\ \ \ \ \
| * | | | | Network: Add tag support for security groupDongcan Ye2018-06-142-4/+171
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Icccb23429913724c6a8bd15d4737672b47a5f13a Closes-Bug: #1750983
* | | | | | Merge "compute: limit the service's force down command above 2.10"Zuul2018-06-241-1/+9
|\ \ \ \ \ \