summaryrefslogtreecommitdiff
path: root/openstackclient/tests/unit/compute/v2
Commit message (Collapse)AuthorAgeFilesLines
* Merge "compute: Fix bug with start/stop server"Zuul2023-05-171-4/+4
|\
| * compute: Fix bug with start/stop serverStephen Finucane2023-05-171-4/+4
| | | | | | | | | | | | | | | | | | | | | | A mistake was introduced during the conversion from novaclient to SDK in change I5ebfa6b2468d5f20b99ea0eab1aea9377be09b8c. Fix the issue and add functional tests to prevent it being reintroduced. Change-Id: I6b314eab31bcf452e88b8b6a239ac2e296497cb9 Signed-off-by: Stephen Finucane <stephenfin@redhat.com> Story: 2010750 Task: 48004
* | Merge "Allow server rebuild --wait for SHUTOFF servers"Zuul2023-05-171-0/+82
|\ \
| * | Allow server rebuild --wait for SHUTOFF serversPavlo Shchelokovskyy2023-05-171-0/+82
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | currently the command is waiting only for ACTIVE server status, but if the server was SHUTOFF before, it will be SHUTOFF after rebuild as well, so the command is stuck in waiting forever. Additionally, we now also pre-validate the server status on client side, and raise an error if the server to be rebuilt is not in ACTIVE, ERROR or SHUTOFF state. Change-Id: If90a4bbba9a7ecd972f8b594c52fee4f75a0ae5e Co-Authored-By: Oleksiy Molchanov <omolchanov@mirantis.com> Story: 2010751 Task: 48005
* | tests: Use SDK objects where expectedStephen Finucane2023-05-166-150/+133
| | | | | | | | | | | | | | | | | | We had not migrated a number of tests to use SDK objects instead of fake novaclient-like objects when migrating the commands themselves. Address this now. Change-Id: Ib0da07fd9d793968b111986bd36a6d4311469d4e Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | tests: Remove unnecessary nesting of compute resourcesStephen Finucane2023-05-1615-1192/+1078
| | | | | | | | | | Change-Id: I27326b8f2e0d84d3ef2c2fbdcb5c96a5ac1d79e4 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | tests: Reorder compute fakesStephen Finucane2023-05-161-71/+71
|/ | | | | | | This will make future changes slightly less complex. Change-Id: Ia69fd9d05796cce5ed48a8d12902cb54caec9779 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstackclient.computeStephen Finucane2023-05-1018-2948/+3544
| | | | | | | | | | Black used with the '-l 79 -S' flags. A future change will ignore this commit in git-blame history by adding a 'git-blame-ignore-revs' file. Change-Id: I9af45c062d179ab3dc2a5e969e1c467932753a2b Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* compute: Generate SSH keypairs ourselvesStephen Finucane2023-05-022-27/+26
| | | | | | | | | | Starting with the 2.92 microversion, nova will no longer generate SSH keys. Avoid breaking users by generating keypairs ourselves using the cryptography library, which was already an indirect dependency through openstacksdk. Change-Id: I3ad2732f70854ab72da0947f00847351dda23944 Implements: blueprint keypair-generation-removal
* Migrate 'server event *' commands to SDKHuda Irshad2023-04-202-105/+146
| | | | | | | | | This one is tricky since the ServerAction resources includes a nested ServerActionEvent resource which requires a custom formatter in order for things to render as expected. Change-Id: I3d24851303222af9efcee8d7e1565278b1018efd Co-authored-by: Stephen Finucane <stephenfin@redhat.com>
* compute: Migrate 'reboot server' to SDKStephen Finucane2023-04-041-0/+114
| | | | | Change-Id: Ibad4078f680d3b2615b9ca6f6c72c4fd28030b55 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Merge ""hypervisor list --matching" showed the wrong result"Zuul2023-04-041-6/+6
|\
| * "hypervisor list --matching" showed the wrong resultdevMuscle2023-03-261-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, using the "--matching" option shows empty results. Previously, the "--matching" option called the "find_hypervisor method", so we used to call the "hypervisor method" like any other "--limit, --marker" options. Depending on the nova api version, the api that is basically called is as follows 2.53 >= : /os-hypervisors/detail?hypervisor_hostname_pattern=$HOSTNAME 2.53 < : /os-hypervisors/{pattern}/search Hypervisor Type and Host IP are not returned when using microversion 2.52 or lower Co-authored-by: Jipyo Hong <hongsbien@naver.com> Co-authored-by: Jieon Lee <dlwldjs7544@naver.com> Co-authored-by: YoonSoo LIM <msdbtjd123@naver.com> story: 2010670 task: 47726 Change-Id: I7b47acf48def7d4c5f4b74e4dba1c23d8ac7abf2
* | Merge "Remove unused helpers for legacy novaclient"Zuul2023-04-031-20/+0
|\ \
| * | Remove unused helpers for legacy novaclientStephen Finucane2022-12-191-20/+0
| | | | | | | | | | | | | | | | | | | | | These are no longer used. Change-Id: I9514e98a5549130c12f3b785e90debf3437e6933 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | Merge "Switch server shelve, unshelve to SDK"Zuul2023-04-031-244/+192
|\ \ \ | |/ /
| * | Switch server shelve, unshelve to SDKStephen Finucane2022-12-191-244/+192
| | | | | | | | | | | | | | | | | | | | | This one is a little more complicated because we support waiting. Change-Id: I5bd65b44c23bfee1e0144dbd060563ecc3cfb942 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | Merge "compute: Switch server restore to SDK"Zuul2023-04-031-8/+3
|\ \ \ | |/ /
| * | compute: Switch server restore to SDKStephen Finucane2022-12-191-8/+3
| | | | | | | | | | | | | | | | | | Change-Id: I8df9711b736991c01136988aa06c8540d640f52f Signed-off-by: Stephen Finucane <sfinucan@redhat.com> Depends-On: https://review.opendev.org/c/openstack/openstacksdk/+/868108
* | | Merge "Remove unnecessary 'self.methods'"Zuul2023-03-301-24/+4
|\ \ \ | |/ /
| * | Remove unnecessary 'self.methods'Stephen Finucane2022-12-191-24/+4
| | | | | | | | | | | | | | | | | | | | | This was used by 'run_method_with_servers', not the SDK-equivalent. Change-Id: Ia3333a95e406f430beb4f2cfb887e81ab7b72cfe Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | Merge "Switch server start, server stop to SDK"Zuul2023-03-301-31/+19
|\ \ \ | |/ /
| * | Switch server start, server stop to SDKThrivikram Mudunuri2022-12-191-31/+19
| | | | | | | | | | | | | | | | | | Switch the server start and server stop commands from novaclient to SDK. Change-Id: I5ebfa6b2468d5f20b99ea0eab1aea9377be09b8c
* | | Merge "Switch server lock, unlock to sdk"Zuul2023-03-301-70/+68
|\ \ \ | |/ / | | / | |/ |/|
| * Switch server lock, unlock to sdksuneethravi2022-12-191-70/+68
| | | | | | | | | | | | | | Switch server lock commands from novaclient to SDK. Change-Id: I042db99c9d7a0d8d207f8cdf69d786bd4fe904e7 Depends-On: https://review.opendev.org/c/openstack/openstacksdk/+/867890
* | Merge "Update 'host list' and 'host show' command to use sdk"Zuul2023-02-221-42/+63
|\ \
| * | Update 'host list' and 'host show' command to use sdkHarsh Mutha2022-12-201-42/+63
| |/ | | | | | | Change-Id: I3813ff604ba46112ebd358509ea4f28ee38ca3ee
* | Merge "Finish switching server migration to sdk"Zuul2023-02-212-137/+140
|\ \
| * | Finish switching server migration to sdkViolet Kurtz2022-12-142-137/+140
| | | | | | | | | | | | Change-Id: Ic0e6dfdc986989599bfc73b8c274631232161c25
* | | Merge "tests: Use fake SDK Migration object"Zuul2023-02-212-60/+42
|\ \ \ | |/ /
| * | tests: Use fake SDK Migration objectStephen Finucane2023-01-042-60/+42
| |/ | | | | | | | | | | | | | | We missed this when migrating the 'server migration list' command across. Change-Id: Ide05a0289020764829f1dbc416be5336fdf37d84 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Merge "Switch server volume update to sdk"Zuul2023-02-211-49/+66
|\ \
| * | Switch server volume update to sdkRitvik Vinodkumar2022-12-151-49/+66
| | | | | | | | | | | | | | | | | | Switch the server volume update command from novaclient to SDK. Change-Id: Ib9876775bcf8268344da1a58ab0dd1695cb83ece
* | | Merge "Switch list server volume to sdk"Zuul2023-02-213-152/+81
|\ \ \ | |/ / | | / | |/ |/|
| * Switch list server volume to sdkRitvik Vinodkumar2022-12-153-152/+81
| | | | | | | | | | | | | | Switch the server volume list command from novaclient to SDK. Modified functional test for server add/remove volume. Change-Id: I5b4ab7d0275aec2e02451c5371319ac350af6a5f
* | Merge "Switch server dump create to using sdk"Zuul2022-12-152-9/+29
|\ \
| * | Switch server dump create to using sdkDiwei Zhu2022-12-152-9/+29
| |/ | | | | | | | | Change-Id: I8e2a4651bff1f739fa98533cd47074b143b2e3d6 Co-authored-by: Stephen Finucane <stephenfin@redhat.com>
* | Merge "Use the SDK in server migration list"Zuul2022-12-151-62/+58
|\ \
| * | Use the SDK in server migration listDaniel Wilson2022-11-301-62/+58
| | | | | | | | | | | | | | | | | | | | | Update server migration list to use the OpenStack SDK instead of directly using the nova interface. Change-Id: I40dc95ee47e7c33ebf596f8ad437228b4bb0ab33
* | | Merge "Use the SDK for server show"Zuul2022-12-151-12/+5
|\ \ \
| * | | Use the SDK for server showDaniel Wilson2022-12-111-12/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the SDK for the server show command. This change modifies a helper function that is used by server show as well as other commands that print information about an individual server. The helper still uses novaclient APIs when additional OpenStack requests are needed since some of its callers are still using the nova client. Depends-On: https://review.opendev.org/c/openstack/openstacksdk/+/864340 Change-Id: Ic253184ee5f911ec2052419d328260dc4664b273
* | | | Merge "Fix parameter handling in server add fixed ip cmd"Zuul2022-12-151-8/+5
|\ \ \ \
| * | | | Fix parameter handling in server add fixed ip cmdDr. Jens Harbott2022-12-121-8/+5
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | 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
* | | 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>
View Lorry Controller Status