summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* 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-023-189/+200
|\ \
| * | Use the compute SDK in server listDaniel Wilson2022-11-293-189/+200
| | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "Improve `server dump create` helptext"Zuul2022-11-291-2/+3
|\ \ \
| * | | Improve `server dump create` helptextArtom Lifshitz2022-10-261-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `server dump create` command instructs Nova to trigger a crash dump in the guest OS. Assuming the guest supports this, the resulting dump file will be located in the guest, in a location dependent on the guest OS. Explain all that in the helptext. Story: 2010384 Change-Id: If940ed5cce6c5ab4193ab1494738149370da9aad
* | | | Merge "Add note about microversion 2.87 in server rescue help"Zuul2022-11-291-1/+5
|\ \ \ \
| * | | | Add note about microversion 2.87 in server rescue helpmelanie witt2022-11-071-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ability to rescue a volume-backed server was added in compute microversion 2.87 [1]. This adds a note to the command help to improve user experience. [1] https://docs.openstack.org/nova/latest/user/rescue.html Change-Id: I5f40c3ca28e13bd1f979bc5f8c337302a3b9a5be
* | | | | Add test for creating volume from sourcewhoami-rajat2022-11-232-4/+46
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Change --size helptext to include backupwhoami-rajat2022-11-221-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Followup from [1]. Modifying help text of --size argument to include --backup option. [1] https://review.opendev.org/c/openstack/python-openstackclient/+/864893 Change-Id: I12cf60079ebcfe1cd059602fbfc1a13c8fe86803
* | | | Add option to create volume from backupwhoami-rajat2022-11-212-3/+99
| |_|/ |/| | | | | | | | | | | | | | | | | | | | 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-163-25/+157
|\ \ \
| * | | image: Add 'image stage' commandStephen Finucane2022-11-093-25/+157
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-162-103/+82
|\ \ \ \ | |/ / /
| * | | image: Simplify handling of data provided via stdinStephen Finucane2022-11-092-103/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-162-1/+34
|\ \ \ \ | |/ / /
| * | | image: Ignore '--progress' if providing image data from stdinStephen Finucane2022-11-092-1/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-163-201/+300
|\ \ \ \
| * | | | Moved hypervisor to the SDKViolet Kurtz2022-11-093-201/+300
| | | | | | | | | | | | | | | | | | | | Change-Id: Ie955fb4d27c30e044626732a1f3e0f141cb85aa5
* | | | | Merge "Use the compute SDK in usage commands"Zuul2022-11-162-66/+48
|\ \ \ \ \
| * | | | | Use the compute SDK in usage commandsDaniel Wilson2022-10-252-66/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-162-17/+91
|\ \ \ \ \ \
| * | | | | | compute: Add missing microversion check for networksStephen Finucane2022-11-082-17/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-162-17/+43
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | compute: Fix '--network none/auto' handlingStephen Finucane2022-11-082-17/+43
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Docstring fix for CreateVolumeAttachment classJadon Naas2022-11-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The command "volume attachment create" has a typo in the docstring. The docstring says to use "server add volume", but the command is actually "server volume add". This change fixes the typo in the docstring. Task: 46781 Story: 2010401 Change-Id: Ie19a24ead100dd9177669653a7a9997772ef4538
* | | | | | 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-082-38/+130
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | compute: Add '--no-network', '--auto-network' flagsStephen Finucane2022-10-262-38/+130
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | 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-053-16/+46
|\ \ \ \
| * | | | quota: Deprecate "force" behavior for network quotasStephen Finucane2022-10-113-16/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-052-16/+17
|\ \ \ \ \
| * | | | | quota: Fix issues with delete quota commandStephen Finucane2022-10-112-16/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | | | | Merge "tests: Remove unnecessary nesting of volume resources"Zuul2022-11-0534-2567/+2378
|\ \ \ \ \ \ | |/ / / / /
| * | | | | tests: Remove unnecessary nesting of volume resourcesStephen Finucane2022-09-3034-2567/+2378
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I210ce7534d161e89115e5cb96e42ab7f27170aa1 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | | Merge "tests: Remove duplicate FakeImagev1Client"Zuul2022-11-051-7/+4
|\ \ \ \ \ \ | |/ / / / /
| * | | | | tests: Remove duplicate FakeImagev1ClientStephen Finucane2022-09-301-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I guess this was defined to avoid a circular import. There are easier ways to do this. Change-Id: Iab215a53691298413ea9d10def792008ab9edbce Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | | Merge "tests: Move fake clients to top of file"Zuul2022-11-053-123/+123
|\ \ \ \ \ \ | |/ / / / /
| * | | | | tests: Move fake clients to top of fileStephen Finucane2022-09-303-123/+123
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are important and they're easier to identify at the top of the file than nested deep inside. Change-Id: I1569c855fadfe4cdacf83ee07d08922bf3edefbc Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | Merge "quota: Trivial style fixups"Zuul2022-11-051-38/+40
|\ \ \ \ \ | |/ / / /
| * | | | quota: Trivial style fixupsStephen Finucane2022-09-301-38/+40
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I4522b54676033dced2b47477238ceb551e11d04a Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | Merge "quota: Deprecate 'quota show --class', 'quota set --class' arguments"Zuul2022-11-051-4/+36
|\ \ \ \ \ | |/ / / /
| * | | | quota: Deprecate 'quota show --class', 'quota set --class' argumentsStephen Finucane2022-09-301-4/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This doesn't do anything special in nova and cinder and is not supported in neutron. For the 'quota show' command, people should use the '--default' argument instead. Change-Id: I0dd38e5cb252a01d5817ed168be040b21b35e348 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>