summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 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-10100-7636/+10507
| | | | | | | | | | 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>
* Use pre-commit for 'pep8' tox target, bump versionsStephen Finucane2023-05-103-33/+12
| | | | | | | | | | | | | | We want to bump the versions of hacking, but doing so requires changes in two places: '.pre-commit-config.yaml' and 'tox.ini'. This is silly: we can simply use tox to handle pre-commit and leave all other dependencies to pre-commit. Do this, migrating bandit to pre-commit and bumping the other dependencies in the process. We remove the 'fast8' target and tool, since pre-commit is plenty fast here. Change-Id: Ifdc582ecc3b4479ea9ed9abcb16577c6e8b2f5c1 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Address pre-commit issuesStephen Finucane2023-05-108-53/+52
| | | | | | | | | We'd like to use pre-commit as our 'pep8' target. Before we do that, we need to address some issues that have crept in over the time since pre-commit was first added. Change-Id: I3bb383a00a44007eaa241e037e38aea16bd517b2 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Merge "compute: Generate SSH keypairs ourselves"Zuul2023-05-046-63/+117
|\
| * compute: Generate SSH keypairs ourselvesStephen Finucane2023-05-026-63/+117
| | | | | | | | | | | | | | | | | | | | 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-215-129/+234
|\ \
| * | Migrate 'server event *' commands to SDKHuda Irshad2023-04-205-129/+234
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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-123-5/+6
|/ | | | Change-Id: I7254d30431894879c8bd20e2263d83b12c99d5c8
* compute: Migrate 'reboot server' to SDKStephen Finucane2023-04-043-14/+133
| | | | | 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 "Add an updated tips job running on py310"Zuul2023-04-041-0/+22
|\ \
| * | Add an updated tips job running on py310Dr. Jens Harbott2023-03-311-0/+22
| | | | | | | | | | | | Change-Id: Iadc32b4e8ea10561784208cdc8bf05ff3b3ca59d
* | | 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-033-278/+245
|\ \ \ \ | |/ / /
| * | | Switch server shelve, unshelve to SDKStephen Finucane2022-12-193-278/+245
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-033-13/+13
|\ \ \ \ | |/ / / | | / / | |/ / |/| |
| * | compute: Switch server restore to SDKStephen Finucane2022-12-193-13/+13
| | | | | | | | | | | | | | | | | | 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-303-45/+54
|\ \ \ | |/ /
| * | Switch server start, server stop to SDKThrivikram Mudunuri2022-12-193-45/+54
| | | | | | | | | | | | | | | | | | Switch the server start and server stop commands from novaclient to SDK. Change-Id: I5ebfa6b2468d5f20b99ea0eab1aea9377be09b8c
* | | Merge "Switch server lock, unlock to sdk"Zuul2023-03-303-87/+99
|\ \ \ | |/ /
| * | Switch server lock, unlock to sdksuneethravi2022-12-193-87/+99
| | | | | | | | | | | | | | | | | | | | | Switch server lock commands from novaclient to SDK. Change-Id: I042db99c9d7a0d8d207f8cdf69d786bd4fe904e7 Depends-On: https://review.opendev.org/c/openstack/openstacksdk/+/867890
* | | Merge "Show Network QoS rules one per line"Zuul2023-03-292-2/+13
|\ \ \
| * | | 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
* | | Docs: Update glance command mappingyaeeee2023-03-251-4/+4
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update missing command in glance Mapping Guide. List of updated command [Glance CLI / OSC] md-namespace-create / image metadef namespace create md-namespace-delete / image metadef namespace delete md-namespace-list / image metadef namespace list md-namespace-show / image metadef namespace show Co-authored-by: sujin01 <a5870771@gmail.com> Co-authored-by: heesom <heesom.hs@gmail.com> Co-authored-by: mpicea <jjh77745997@gmail.com> Change-Id: I4d50b0ae820ea0c8feb1719bbbd9eb06d22cbe20
* | Merge "Add auto-approve option to project cleanup"Zuul2023-03-203-11/+50
|\ \
| * | Add auto-approve option to project cleanupDr. Jens Harbott2023-02-153-11/+50
| | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Update master for stable/2023.1OpenStack Release Bot2023-03-092-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add file to the reno documentation build to show release notes for stable/2023.1. Use pbr instruction to increment the minor version number automatically so that master versions are higher than the versions on stable/2023.1. Sem-Ver: feature Change-Id: I248bcb0342afb8276ba9605963287f620b62b6c5
* | | Merge "zuul: Merge osc-functional-devstack-base into only child"6.2.0Zuul2023-02-231-23/+17
|\ \ \
| * | | zuul: Merge osc-functional-devstack-base into only childStephen Finucane2023-02-221-23/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we removed the osc-functional-devstack-n-net job in change I76efeccec04937c3a68108e2654872e00fadcec4, there's no need to maintain separate osc-functional-devstack-base and osc-functional-devstack jobs. Merge them. Change-Id: Id3dcd3c4f9f47a4020faf0787f44bd5509a2b5a2 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | 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-223-58/+111
|\ \ \ \
| * | | | Update 'host list' and 'host show' command to use sdkHarsh Mutha2022-12-203-58/+111
| | |_|/ | |/| | | | | | | | | | Change-Id: I3813ff604ba46112ebd358509ea4f28ee38ca3ee
* | | | Merge "Fix really long help strings"Zuul2023-02-224-35/+33
|\ \ \ \
View Lorry Controller Status