summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Fix parameter handling in server add fixed ip cmd"Zuul2022-12-152-8/+61
|\
| * Fix parameter handling in server add fixed ip cmdDr. Jens Harbott2022-12-122-8/+61
| | | | | | | | | | | | | | | | | | | | | | | | The fixed_ip_address parameter needs to be passed in a hash with key "ip_address" in order to be processed by the server, the previous arg was simply being ignored. Added a functional test for better coverage. Closes-Bug: 1998927 Change-Id: I6956d2642d8e80fc10c3739f0a571aa7ba276b1a
* | Merge "Migrate hypervisor stats commands to SDK"Zuul2022-12-152-68/+44
|\ \
| * | Migrate hypervisor stats commands to SDKAriel-Berkowicz2022-12-122-68/+44
| |/ | | | | | | Change-Id: I43b2071f5108c28f6881c8e99d4b06e87c83ddfa
* | Merge "Create a functional test case for hypervisor list and show"Zuul2022-12-151-0/+52
|\ \
| * | Create a functional test case for hypervisor list and showAriel-Berkowicz2022-12-141-0/+52
| | | | | | | | | | | | | | | | | | | | | This functional test case for hypervisor list and show makes sure it works with the openstackclient sdk. Change-Id: Ibaeaf68d6056fcf904b546f30287c65173b5a368
* | | Fix server list error with --long and -c optionsThobias Salazar Trevisan2022-12-141-0/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | Using options --long and -c and specifying same columns added by --long option, it passes duplicated column names to prettytable and report the following error: Field names must be unique! This patch removes duplicated columns. Change-Id: I9c0bd09c50dac568ca1980a6b53a6c544b85c2aa
* | Fix functional-tips jobStephen Finucane2022-12-141-5/+3
|/ | | | | | | | | | A recent change to cliff [1] means we're now stripping periods when generating the summary line of command help strings. Account for this. [1] https://review.opendev.org/c/openstack/cliff/+/867274 Change-Id: I45b39b9fe38914497505f157e91d84cd2f84f547 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Merge "image: Add 'image import' command"Zuul2022-12-052-5/+223
|\
| * image: Add 'image import' commandStephen Finucane2022-11-092-5/+223
| | | | | | | | | | | | | | | | Note that we require some additional functionality in SDK for this to work properly, but it's a start. Change-Id: I87f94db6cced67f36f71685e791416f9eed16bd0 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Merge "compute: Allow users to manually specify bootable volumes"Zuul2022-12-051-6/+38
|\ \
| * | compute: Allow users to manually specify bootable volumesStephen Finucane2022-12-011-6/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When creating a server with an attached volume, you can specify a block device with a 'boot_index' of '0' and this will become the bootable device. OSC allows users to do this by using either the '--volume' option or a combination of the '--image' and '--boot-from-volume' options, but we should also allow them to do it the "hard way" via the '--block-device' option. For example: openstack server create \ --block-device uuid=0a89ecd8-1fe2-45f0-94da-7789067911c9,boot_index=0 \ --block-device uuid=589266ef-fd88-46e9-b7b2-94503ce8f88f,boot_index=1 \ ... \ my-server Make this possible. Change-Id: Ia48449fecbc590346630807b1c7da40102d53b33 Signed-off-by: Stephen Finucane <sfinucan@redhat.com> Story: 2010376 Task: 46617
* | | Merge "tests: Add test for multiple blocks devices"Zuul2022-12-051-19/+35
|\ \ \ | |/ /
| * | tests: Add test for multiple blocks devicesStephen Finucane2022-12-011-19/+35
| | | | | | | | | | | | | | | | | | | | | | | | The 'server create' command should support multiple '--block-device' parameters. Prove it. Change-Id: I1bd83287efdbbe11774053b694bae99b6a4ebdf5 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | Merge "tests: Convert network tests to use 'parse_output'"Zuul2022-12-0227-892/+1130
|\ \ \
| * | | tests: Convert network tests to use 'parse_output'Stephen Finucane2022-11-2927-892/+1130
| | | | | | | | | | | | | | | | | | | | Change-Id: I93e2a4e0a4c7ec07da0c78a171f3d787125af053 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | Merge "tests: Convert compute tests to use 'parse_output'"Zuul2022-12-027-403/+503
|\ \ \ \ | |/ / /
| * | | tests: Convert compute tests to use 'parse_output'Stephen Finucane2022-11-297-403/+503
| | | | | | | | | | | | | | | | | | | | Change-Id: Ib5b2f46639f14877a9ec295b26cae01a05395d4d Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | Merge "tests: Convert identity tests to use 'parse_output'"Zuul2022-12-021-7/+7
|\ \ \ \ | |/ / /
| * | | tests: Convert identity tests to use 'parse_output'Stephen Finucane2022-11-291-7/+7
| | | | | | | | | | | | | | | | Change-Id: I10711b911986af0348946f6254cf36773110b0ab
* | | | Merge "tests: Convert volume tests to use 'parse_output'"Zuul2022-12-0219-759/+923
|\ \ \ \ | |/ / /
| * | | tests: Convert volume tests to use 'parse_output'Stephen Finucane2022-11-2919-759/+923
| | | | | | | | | | | | | | | | | | | | Change-Id: Iec8ca873f6bc3993e0ba557f68895d9aefb6f9c6 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | Merge "tests: Convert image tests to use 'parse_output'"Zuul2022-12-022-125/+120
|\ \ \ \ | |/ / /
| * | | tests: Convert image tests to use 'parse_output'Stephen Finucane2022-11-292-125/+120
| | | | | | | | | | | | | | | | | | | | Change-Id: I5f256d466d503d70d1f380016f9c8f5a0d9e395f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | Merge "Use the compute SDK in server list"Zuul2022-12-022-137/+126
|\ \ \ \ | |_|/ / |/| | |
| * | | Use the compute SDK in server listDaniel Wilson2022-11-292-137/+126
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update server list to use the compute component of the OpenStack SDK instead of directly using the nova interface. This change depends on SDK version 0.102.0 for automatic client-side query filters. Change-Id: Ib9985812bfd98320b75f3a82bb594a0daa6e4d93
* | | | Add test for creating volume from sourcewhoami-rajat2022-11-231-0/+37
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds a test to create a new volume from source. We also include code changes to pass the right size i.e. either size passed by the user via --size argument or the source volume size. This case is already handled at the API layer[1] but it helps being consistent with passing the right size value as in case of creating a volume from snapshot or backup. [1] https://github.com/openstack/cinder/blob/7c1a5ce7b11964da4537fd6a7d157ede646b9e94/cinder/api/v3/volumes.py#L381-L382 Change-Id: Idc71636dad6bb678fe24f19b0836d2e9bd92d7d2
* | | Add option to create volume from backupwhoami-rajat2022-11-211-0/+73
|/ / | | | | | | | | | | | | | | Support for creating a volume from backup was added in microversio 3.47. This patch adds a --backup option to the volume create command to add that support. Change-Id: Ib26d2d335475d9aacbf77c0fd7b7cda2ba743943
* | Merge "image: Add 'image stage' command"Zuul2022-11-162-25/+80
|\ \ | |/
| * image: Add 'image stage' commandStephen Finucane2022-11-092-25/+80
| | | | | | | | | | | | | | This is the equivalent of the 'image-stage' glanceclient command. Change-Id: I10b01ef145740a2f7ffe5a8c7ce0296df0ece0bd Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Merge "image: Simplify handling of data provided via stdin"Zuul2022-11-161-64/+30
|\ \ | |/
| * image: Simplify handling of data provided via stdinStephen Finucane2022-11-091-64/+30
| | | | | | | | | | | | | | This was unnecessarily complex. Change-Id: I8289d5ce7356d8bc89425590a7f71bca91a6d396 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Merge "image: Ignore '--progress' if providing image data from stdin"Zuul2022-11-161-0/+31
|\ \ | |/
| * image: Ignore '--progress' if providing image data from stdinStephen Finucane2022-11-091-0/+31
| | | | | | | | | | | | | | | | You can provide data via stdin when creating an image. Using this with '--progress' makes no sense and causes an error currently. Fix this. Change-Id: I3c2d658b72a7c62931b779b0d19bb97f60a0c655 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Merge "Moved hypervisor to the SDK"Zuul2022-11-162-163/+201
|\ \
| * | Moved hypervisor to the SDKViolet Kurtz2022-11-092-163/+201
| | | | | | | | | | | | Change-Id: Ie955fb4d27c30e044626732a1f3e0f141cb85aa5
* | | Merge "Use the compute SDK in usage commands"Zuul2022-11-161-27/+18
|\ \ \
| * | | Use the compute SDK in usage commandsDaniel Wilson2022-10-251-27/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Update usage list and usage show to use the compute component of the OpenStack SDK instead of directly using the nova interface. Change-Id: I1c4d2247c9c1a577ed9efad7e8332e7c9b974ad5
* | | | Merge "compute: Add missing microversion check for networks"Zuul2022-11-161-17/+83
|\ \ \ \
| * | | | compute: Add missing microversion check for networksStephen Finucane2022-11-081-17/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'auto' and 'none' network allocation policies are only supported on compute API microversion 2.37 or later. Enforce this in the code. Change-Id: I90f8fb1e61ead4bd406ea76bbeb731b913805b13 Signed-off-by: Stephen Finucane <sfinucan@redhat.com> Story: 2010385 Task: 46657
* | | | | Merge "compute: Fix '--network none/auto' handling"Zuul2022-11-161-12/+37
|\ \ \ \ \ | |/ / / /
| * | | | compute: Fix '--network none/auto' handlingStephen Finucane2022-11-081-12/+37
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should lookup a network called 'none' or 'auto', not do the equivalent on '--nic none' or '--nic auto'. Correct this. Change-Id: I3c5acc49bfe8162d8fb6110603da56d56090b78f Signed-off-by: Stephen Finucane <sfinucan@redhat.com> Story: 2010385 Task: 46658
* | | | tests: Convert more functional tests to use 'parse_output'Stephen Finucane2022-11-086-113/+105
| | | | | | | | | | | | | | | | | | | | Change-Id: I1d968181eb196c6df4583c772c67ed58bc7ba585 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | tests: Move json decoding to base test classStephen Finucane2022-11-082-30/+48
|/ / / | | | | | | | | | | | | | | | | | | | | | We do this everywhere. Add a simple knob to simplify the pattern. Only one use is migrated initially. The rest will be done separately. Change-Id: Ic3b8958bd4fb1459a8ac3adaff216c2a26628491 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | Merge "compute: Add '--no-network', '--auto-network' flags"Zuul2022-11-081-14/+44
|\ \ \
| * | | compute: Add '--no-network', '--auto-network' flagsStephen Finucane2022-10-261-14/+44
| |/ / | | | | | | | | | | | | | | | | | | These are aliases for '--nic none' and '--nic auto', respectively. Change-Id: I7b4f7e5c3769a813bd8b2b9cd6090c6fe501e13d Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | Merge "quota: Deprecate "force" behavior for network quotas"Zuul2022-11-052-5/+6
|\ \ \
| * | | quota: Deprecate "force" behavior for network quotasStephen Finucane2022-10-112-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In change Idc1b99492d609eb699d0a6bef6cd760458a774f6, we added a '--check-limit' option to enable "no force" behavior for network quotas. This was already the default for compute quotas. Provide a path for harmonizing the behavior of the two options by instead using a '--no-force' option which will become the eventual default for the network quotas also. Change-Id: I25828a3d68e2e900f498e17a0d01fb70be77548e Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | Merge "quota: Fix issues with delete quota command"Zuul2022-11-051-13/+14
|\ \ \ \
| * | | | quota: Fix issues with delete quota commandStephen Finucane2022-10-111-13/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We were passing a project object rather than just the ID. Also correct a typo in the call to delete network quotas. Change-Id: I2292db7932ec01026f0e54014e3d02218792617a Signed-off-by: Stephen Finucane <sfinucan@redhat.com>