summaryrefslogtreecommitdiff
path: root/openstackclient/tests/unit
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add project field in image list subcommand"Zuul2020-12-221-0/+15
|\
| * Add project field in image list subcommandokozachenko2020-12-151-0/+15
| | | | | | | | | | | | The motivation is to filter the image by owner Change-Id: I1f08da175a06e62a844f76b0ec18cb3332efef86
* | Merge "Add name and enabled param in ListDomain parser"Zuul2020-12-221-0/+55
|\ \
| * | Add name and enabled param in ListDomain parserVishakha Agarwal2020-07-211-0/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when doing openstack domain list --name xyz_id, and openstack domain list --enabled CLI raising error unrecognized arguments, whereas in api-ref document [1], user can pass name and enabled as optional query param. This addresses the above issue, by adding param --name and --enabled in parser of ListDomain. [1]https://docs.openstack.org/api-ref/identity/v3/?expanded=list-domains-detail#list-domains Change-Id: I3cdb511d3c7059ddfb802ca025188d8976c9302c
* | | Merge "Add option to filter instances by AZ"Zuul2020-12-221-0/+1
|\ \ \ | |_|/ |/| |
| * | Add option to filter instances by AZDmitriy Rabotyagov2020-12-011-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | Since nova API microversion 2.83 it is possible for users to filter instances by AZ. However even before that this functionality was available for admin role. Change-Id: Ife4c8e81aad2ff1dde50d9f23913d9dd9397b00c
* | | Make use of comparable 'FormattableColumn' subclassesStephen Finucane2020-12-0930-301/+290
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This requires fixes found in cliff 3.5.0 [1] and osc-lib 2.3.0 [2]. With these fixes in place, we can remove the icky, still broken 'assertItemEqual' and 'assertListItemEqual' helpers. [1] https://review.opendev.org/761421 [2] https://review.opendev.org/761394 Change-Id: Id6c26b37c3c7d5ec6761361abca57f9219b76838 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | Switch compute aggregate functions to SDKArtem Goncharov2020-12-021-85/+172
| | | | | | | | | | | | | | | | | | | | | Continue journey towards having OSC consuming SDK for nova part. Depends-On: https://review.opendev.org/#/c/762131/ Change-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80
* | | Switch compute flavors from novaclient/direct to SDKArtem Goncharov2020-12-022-215/+307
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let's switch flavors from novaclient or direct API requests onto using SDK. Microversion agreement comes out of the box. SDK normalizes property names, while OSC uses server side names. In order not to break OSC users continue using server-side names. Depends-On: https://review.opendev.org/#/c/762989/ Change-Id: I62b2ed8488ee4ac9c42051311bcfb455506ddd90
* | | Merge "stop image downloads to memory"Zuul2020-12-021-1/+2
|\ \ \
| * | | stop image downloads to memorySimon Merrick2020-11-251-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | + Fixes issue with large images hogging memory + stream image downloads + output to stdout if file not specified Change-Id: Ia01ff9b21a2dac5d0ccf2bd58a8640e88c5cbb36 Story: 2007672 Task: 39776
* | | | Merge "Remove None valued network quota entries"Zuul2020-12-011-0/+23
|\ \ \ \
| * | | | Remove None valued network quota entriesasarfaty2020-09-221-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since the openstack SDK still has the neutron-lbaas entries in the network quota, but those are already deprecated [1], the 'opentack quota show' command shows those as None value. This fix removes those empty deprecated values from the output. [1] https://review.opendev.org/#/c/658494/ Change-Id: I8dbdba2a029ea8e6a268ddf29627e1466a7e3a8a
* | | | | Merge "Add id and enabled param in ListIdentityProvider parser"Zuul2020-12-011-0/+55
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add id and enabled param in ListIdentityProvider parserVishakha Agarwal2020-07-241-0/+55
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when doing openstack identity provider list --name xyz_id, and openstack identity provider list --enabled CLI raising error unrecognized arguments, whereas in api-ref document [1], user can pass name and enabled as optional query param. This addresses the above issue, by adding param --id and --enabled in parser of ListIdentityProvider. [1] https://docs.openstack.org/api-ref/identity/v3-ext/?expanded=list-identity-providers-detail#list-identity-providers Change-Id: I59ce3a5f54700ba5a735f0b3b4b3b73b3a8658fa
* | | | Add "fields" parameter to ListPort queryRodolfo Alonso Hernandez2020-11-181-19/+55
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new query parameter will allow to send a query to the Neutron server filtering only by those parameters needed by the list command: ID, name, MAC address, fixed IPs and status. When using input parameter "long", security groups IDs, device owner and tags will be added to the fields filter. With 4500 ports, those are the execution times for the command "openstack port list" (average values in a development environment): Neutron API (seconds) CLI (seconds) Without filter: 3.05 10.15 With filter: 2.76 8.19 Depends-On: https://review.opendev.org/#/c/754113/ Change-Id: I1cccf0bc3533f8085e8dd61bf2fbe78c49b74b31 Closes-Bug: #1897100
* | | Merge "Add a few selectable fields to the "openstack server list" output"Zuul2020-11-181-0/+19
|\ \ \
| * | | Add a few selectable fields to the "openstack server list" outputjay2020-11-051-0/+19
| |/ / | | | | | | | | | | | | | | | | | | | | | Added ``-c project_id | user_id | created_at`` to ``openstack server list`` command to get these columns as an output. Change-Id: I18991adf899c7b72c98bb89871bf0715d35943f0 Story: 2007925
* | | Merge "Switch 'openstack keypair' ops to use SDK"Zuul2020-11-171-108/+77
|\ \ \
| * | | Switch 'openstack keypair' ops to use SDKArtem Goncharov2020-11-091-108/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let's continue our journey and start using SDK for the keypair operations Depends-On: https://review.opendev.org/#/c/761883/ Change-Id: Id411e70b8e1a79c0e88a0e22be7ff37e5c30fcda
* | | | Merge "tests: Remove 'agent' functional tests"Zuul2020-11-161-1/+1
|\ \ \ \
| * | | | tests: Remove 'agent' functional testsStephen Finucane2020-11-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'os-agents' API was recently removed from nova [1]. Remove the functional tests, since they will always fail going forward but will continue to run on older stable branches. Also Squeeze https://review.opendev.org/#/c/762559/ inside, since those 2 are simultaneously blocking gate [1] https://review.opendev.org/#/c/749309/ Change-Id: I0bf7d4c0ba2a9d4637db0d209d8d52163d772f12 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | Merge "Resolve issues with 'server migration list'"Zuul2020-11-121-9/+3
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Resolve issues with 'server migration list'Stephen Finucane2020-11-091-9/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'os-migrations' API accepts 'instance_uuid' and 'migration_type' query string parameters, not 'server' and 'type'. For the former, as the name would suggest, the value should be a server UUID, not a name. In addition, this is a list command and therefore should subclass the 'Lister' base class. Change-Id: I736f5575156fc04d7ada7783a1865ab3b438396f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | Merge "Add 'openstack server evacuate' command"Zuul2020-11-091-0/+161
|\ \ \ \ | |/ / / |/| | |
| * | | Add 'openstack server evacuate' commandSean Mooney2020-11-021-0/+161
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds a new 'openstack server evacuate' command to provide parity with the 'nova evacuate' command. The term "evacuate" is notoriously poor, in that it implies the instance is moved rather than recreated, but it is retained since people are familiar with it now. Change-Id: I1e32ca51036c501862d8e89b3144a9695d98a06f
* | | | Merge "Compute: Add user id support for keypair"Zuul2020-11-041-1/+162
|\ \ \ \
| * | | | Compute: Add user id support for keypairHe Jie Xu2020-11-021-9/+162
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds functionality of specific the user id when create, delete, show and list keypairs. Change-Id: Ib826f1f4f5a73d1875ba0f02e124b3222c4d05ed Co-Authored-By: tianhui <tianhui@awcloud.com>
* | | | Merge "Add 'openstack server migration list --type' option"Zuul2020-11-031-0/+2
|\ \ \ \
| * | | | Add 'openstack server migration list --type' optionStephen Finucane2020-10-131-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Another gap with novaclient closed. Change-Id: Id3ca95ceda6f438fa72496ab9ab15ac09bb64fa5 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | Merge "Fix formatting of the flavor properties"Zuul2020-11-031-13/+13
|\ \ \ \ \
| * | | | | Fix formatting of the flavor propertiesArtem Goncharov2020-11-021-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not stringify flavor properties to allow proper output formatting to json/yaml/etc Change-Id: I9f4c42acb85b726af87123134dd19de98fe95074
* | | | | | Merge "Add support for 'keypairs list --project' parameter"Zuul2020-11-021-9/+86
|\ \ \ \ \ \
| * | | | | | Add support for 'keypairs list --project' parameterStephen Finucane2020-10-141-9/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It would be lovely to do this server side but doing so requires a new microversion, a blueprint and a spec. This is less performant but should do the trick for the odd time users want to do this. Change-Id: I26e7d38966304dd67be5da8ed0bb24f87191b82f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | | | Merge "Switch console url show operations to SDK"Zuul2020-11-021-55/+31
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | | |
| * | | | | | Switch console url show operations to SDKArtem Goncharov2020-10-241-55/+31
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Switch from using novaclient to SDK for openstack console url show operation. Depends-On: https://review.opendev.org/756286 Change-Id: Ibe247825148788c549c2c1e991aae92338cdf557
* | | | | | Merge "Add support for 'keypairs list --user' parameter"Zuul2020-11-021-2/+64
|\ \ \ \ \ \ | |/ / / / / |/| / / / / | |/ / / /
| * | | | Add support for 'keypairs list --user' parameterStephen Finucane2020-10-141-2/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has been supported by nova and novaclient since the veritable dark ages. Add it to OSC. Change-Id: Ifc95e7dd6c00807c80e87e10046ab154d0989014 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | Merge "Validate 'server group create --policy' option"Zuul2020-10-211-0/+43
|\ \ \ \ \
| * | | | | Validate 'server group create --policy' optionStephen Finucane2020-10-131-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We were documenting that some of these policies were only supported with specific microversions, however, we weren't actually enforcing that, leading to a poor user experience. Correct this. Change-Id: Ic3c555226a220efd9b0f27edffccf6c4c95c2747 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | | Merge "Cleanup of 'server migration list' command"Zuul2020-10-211-83/+102
|\ \ \ \ \ \ | | |_|/ / / | |/| | | |
| * | | | | Cleanup of 'server migration list' commandStephen Finucane2020-10-131-83/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Address some post merge nits. Change-Id: Ie59521d81fab191194f6c1a114b007fa17f5299f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | | Merge "Switch openstack console log show operation to use OpenStackSDK"Zuul2020-10-211-0/+79
|\ \ \ \ \ \
| * | | | | | Switch openstack console log show operation to use OpenStackSDKArtem Goncharov2020-10-061-0/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A short switch onto SDK for fetching console logs of the server Change-Id: I3f750ea4f13a4e72272aa67ea4506bd7182b13f9
* | | | | | | Merge "Compute: Add tag support for server add fixed ip"Zuul2020-10-151-0/+66
|\ \ \ \ \ \ \
| * | | | | | | Compute: Add tag support for server add fixed iptianhui2020-10-121-0/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I62ed4729dead9f91630d1f568c834c9642965558 Story: 2002195 Task: 21679
* | | | | | | | Merge "Compute: Add tags support for server"Zuul2020-10-141-0/+260
|\ \ \ \ \ \ \ \
| * | | | | | | | Compute: Add tags support for servertianhui2020-10-141-0/+260
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: If065602792958ff0145ae9f2e05f5b7a3177905c Story: 2002006 Task: 19641
* | | | | | | | Merge "Restore behavior of image create with same name."Zuul2020-10-141-1/+5
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Restore behavior of image create with same name.Sam Morrison2020-10-061-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With 60e7c51df4cf061ebbb435a959ad63c7d3a296bf the behaviour of `openstack image create` changed so that you can't create an image with the same name. This patch restores the previous functionality. Story: 2008229 Task: 41069 Change-Id: Ia0f4920371a918e94d1ccf9fcfcbf90ff885a455