summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Blacken openstackclient.apiStephen Finucane2023-05-1010-162/+113
| | | | | | | | | | 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: I1df5bc4c35f02147fe5ac5b4073f0e01e7d51c2f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstack.commonStephen Finucane2023-05-1029-566/+777
| | | | | | | | | | 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: Ifcb3c798666d74d596b8ecb3d6d507f782de7ba5 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstackclient.imageStephen Finucane2023-05-1012-664/+623
| | | | | | | | | | 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: I6420ec6fd550903b03083b9b1f8391511913c86f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstackclient.objectStephen Finucane2023-05-1011-281/+300
| | | | | | | | | | 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: I6658e76d7d1f93149f24fcab28d593195b37779f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstackclient.identityStephen Finucane2023-05-10100-4011/+4901
| | | | | | | | | | 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: I2eeade1ce6653be8e9179ecc40105182c5ff5f16 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstackclient.networkStephen Finucane2023-05-1099-7633/+10504
| | | | | | | | | | 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: I8048746dbc2ef0cb582f68934734db4c1153d779 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstackclient.volumeStephen Finucane2023-05-1084-4438/+5339
| | | | | | | | | | 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: Ic318617c67ab7ce6527f9016b759a1d4b0b80802 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Blacken openstackclient.computeStephen Finucane2023-05-1043-4154/+4902
| | | | | | | | | | 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>
* Merge "compute: Generate SSH keypairs ourselves"Zuul2023-05-044-63/+105
|\
| * compute: Generate SSH keypairs ourselvesStephen Finucane2023-05-024-63/+105
| | | | | | | | | | | | | | | | | | | | 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
* | Merge "Silence warnings from openstacksdk"Zuul2023-05-031-0/+5
|\ \ | |/ |/|
| * Silence warnings from openstacksdkStephen Finucane2023-04-111-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OSC has historically relied on project-specific clients like novaclient for library bindings. These did not support auto-negotiation of versions. Instead, users were advised to set the requested microversion to e.g. '2.latest' to opt into the latest API version supported by the client. We're slowly migrating everything to SDK which *does* support auto-negotiation (well, of sorts). This makes versions like '2.latest' unecessary and SDK correctly warns the user about this. You have a configured API_VERSION with 'latest' in it. In the context of openstacksdk this doesn't make any sense. Unfortunately, we have not yet migrated all commands to SDK, meaning we have a mix of SDK and legacy client-based commands. So long as there are any command using the legacy client, we can't insist on users removing this configuration. This makes the warning both annoying and something the user can't do anything about. We also don't want to remove the warning from SDK so instead we opt to filter it out, along with all other warnings from openstacksdk (which similarly a user can't do anything about). Change-Id: If8a7cf9bc876f84864d66f5aed5f2f61c5d0696a Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Merge "Migrate 'server event *' commands to SDK"Zuul2023-04-214-129/+229
|\ \
| * | Migrate 'server event *' commands to SDKHuda Irshad2023-04-204-129/+229
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | Fix pep issue in the network service providerArtem Goncharov2023-04-201-10/+16
|/ / | | | | | | | | | | | | | | pep gods started complaining (correctfully) about spacing in the old command. Apply `black -l 79` on the file to make it looking nice and passing checks. Change-Id: I716f6a1496fc552b32809c7eb744283f3a3cd5a4
* | Adding ``image import`` commandMridula Joshi2023-04-122-5/+5
|/ | | | Change-Id: I7254d30431894879c8bd20e2263d83b12c99d5c8
* compute: Migrate 'reboot server' to SDKStephen Finucane2023-04-042-14/+129
| | | | | Change-Id: Ibad4078f680d3b2615b9ca6f6c72c4fd28030b55 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Merge ""hypervisor list --matching" showed the wrong result"Zuul2023-04-042-12/+16
|\
| * "hypervisor list --matching" showed the wrong resultdevMuscle2023-03-262-12/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-032-278/+240
|\ \ \ | |/ /
| * | Switch server shelve, unshelve to SDKStephen Finucane2022-12-192-278/+240
| | | | | | | | | | | | | | | | | | | | | 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-032-13/+9
|\ \ \ | |/ /
| * | compute: Switch server restore to SDKStephen Finucane2022-12-192-13/+9
| | | | | | | | | | | | | | | | | | 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-302-45/+49
|\ \ \ | |/ /
| * | Switch server start, server stop to SDKThrivikram Mudunuri2022-12-192-45/+49
| | | | | | | | | | | | | | | | | | Switch the server start and server stop commands from novaclient to SDK. Change-Id: I5ebfa6b2468d5f20b99ea0eab1aea9377be09b8c
* | | Merge "Switch server lock, unlock to sdk"Zuul2023-03-302-87/+95
|\ \ \ | |/ /
| * | Switch server lock, unlock to sdksuneethravi2022-12-192-87/+95
| | | | | | | | | | | | | | | | | | | | | Switch server lock commands from novaclient to SDK. Change-Id: I042db99c9d7a0d8d207f8cdf69d786bd4fe904e7 Depends-On: https://review.opendev.org/c/openstack/openstacksdk/+/867890
* | | Show Network QoS rules one per lineRodolfo Alonso Hernandez2023-03-282-2/+13
| |/ |/| | | | | | | | | | | | | The command "network show qos policy <qos>" now shows the QoS rules printed one per line. Example: https://paste.opendev.org/show/bgam6amfLUVheAMhDxB3/ Change-Id: I65a99da16056902a822804e29b2dd9376a72e091
* | Merge "Add auto-approve option to project cleanup"Zuul2023-03-202-11/+44
|\ \
| * | Add auto-approve option to project cleanupDr. Jens Harbott2023-02-152-11/+44
| | | | | | | | | | | | | | | | | | | | | | | | Sometimes it is useful to be able to use the project cleanup function in scripts and having to add "echo y | openstack" isn't really nice. Signed-off-by: Dr. Jens Harbott <harbott@osism.tech> Change-Id: I3ded9982769b4568b6dcfc49bc31ba67a6227839
* | | Merge "Add missing documentation for state options for "image set""Zuul2023-02-221-2/+8
|\ \ \
| * | | Add missing documentation for state options for "image set"lsmman2023-02-221-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current "image set" with the state option behaves as follows: if you use --project, it updates the membership status of the given project. When this command was first added, it had the following description: If --project is passed, update the membership status for the given project However, the description was missed when moving the documentation for which command to the parser for each command method in commit f055fe67c. Correct this oversight. Co-authored-by: JAE YONG LEE <jaeljy135@gmail.com> Change-Id: I18a29a19ce973971f68a52cbf4020cfa324c7a35
* | | | Merge "Drop default from ask_user_yesno question"Zuul2023-02-221-5/+3
|\ \ \ \ | | |/ / | |/| |
| * | | Drop default from ask_user_yesno questionDr. Jens Harbott2022-12-271-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no default for ask_user_yesno() since an empty answer instead just repeats the question. Drop the unnecessary parameter. Change-Id: I207bccbcbd31f831765e74a20c503b695790c0f5
* | | | Merge "Wait for volume being available to set bootable or readonly"Zuul2023-02-224-14/+164
|\ \ \ \
| * | | | Wait for volume being available to set bootable or readonlymatbu2023-02-224-14/+164
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | This patch add a check to be sure that the volume created is in a available state before trying to set bootable or readonly flag. Story: 2002158 Change-Id: I8db71fd8cf5bd14eb67880f76d2e9135edeb3ed2
* | | | Merge "Update 'host list' and 'host show' command to use sdk"Zuul2023-02-222-58/+107
|\ \ \ \
| * | | | Update 'host list' and 'host show' command to use sdkHarsh Mutha2022-12-202-58/+107
| | |_|/ | |/| | | | | | | | | | Change-Id: I3813ff604ba46112ebd358509ea4f28ee38ca3ee
* | | | Merge "Fix really long help strings"Zuul2023-02-224-35/+33
|\ \ \ \
| * | | | Fix really long help stringsStephen Finucane2022-12-124-35/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Each command should have a summary line followed by a longer description, if needed. Some commands were not following this. Fix them. Change-Id: If1ce7654037d192626460f34c069ea0979919b9b Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | Merge "Add port ranges on floating ip portforwardings cli"Zuul2023-02-223-57/+314
|\ \ \ \ \
| * | | | | Add port ranges on floating ip portforwardings cliPedro Martins2022-07-133-57/+314
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is one of a series of patches to implement floating ip port forwarding with port ranges. The specification is defined in: https://github.com/openstack/neutron-specs/blob/master/specs/wallaby/port-forwarding-port-ranges.rst Change-Id: If9679c87fd8b770fcd960048e091ee8d59205285 Implements: blueprint floatingips-portforwarding-ranges Related-Bug: #1885921
* | | | | | Merge "volume: Remove duplication from 'consistency group create' opts"Zuul2023-02-222-15/+34
|\ \ \ \ \ \
| * | | | | | volume: Remove duplication from 'consistency group create' optsStephen Finucane2023-02-142-15/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The '--consistency-group-source' and '--consistency-group-snapshot' opts are unnecessarily verbose. Shorten them to '--source' and '--snapshot', respectively, maintaining aliases to avoid breaking users. Change-Id: I2b6656a8a09d953eb4406f1d4fd1e804743a8963 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | | | | Merge "Deprecate positional args for 'volume group create'"Zuul2023-02-222-44/+138
|\ \ \ \ \ \ \
| * | | | | | | Deprecate positional args for 'volume group create'Stephen Finucane2023-02-172-44/+138
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are now many ways to create a new volume group, thus the positional arguments don't make sense. Deprecate them. Change-Id: Id0b212426861719db1812b7d07b82613cf591de4 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>