Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update gophercloud client to v2 #951

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

hebelsan
Copy link
Contributor

How to categorize this PR?

/area quality
/kind enhancement
/platform openstack

What this PR does / why we need it:
This PR updates the gophercloud client to v2.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Updates gophercloud client to v2

@hebelsan hebelsan requested review from a team as code owners January 13, 2025 09:27
@gardener-robot gardener-robot added needs/review Needs review area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/enhancement Enhancement, improvement, extension platform/openstack OpenStack platform/infrastructure size/xl Size of pull request is huge (see gardener-robot robot/bots/size.py) needs/second-opinion Needs second review by someone else labels Jan 13, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 13, 2025
@gardener-robot-ci-3 gardener-robot-ci-3 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jan 13, 2025
@hebelsan
Copy link
Contributor Author

/test

@testmachinery
Copy link

testmachinery bot commented Jan 13, 2025

Testrun: e2e-cxfk4
Workflow: e2e-cxfk4-wf
Phase: Failed

+---------------------+-----------------------------+--------+----------+
|        NAME         |            STEP             | PHASE  | DURATION |
+---------------------+-----------------------------+--------+----------+
| infrastructure-test | infrastructure-test         | Failed | 8m15s    |
| infrastructure-test | infrastructure-test-flow    | Failed | 8m15s    |
| infrastructure-test | infrastructure-test-migrate | Failed | 8m15s    |
| infrastructure-test | infrastructure-test-recover | Failed | 8m14s    |
| bastion-test        | bastion-test                | Failed | 8m11s    |
+---------------------+-----------------------------+--------+----------+

@gardener-robot gardener-robot added the needs/rebase Needs git rebase label Jan 13, 2025
@gardener-robot
Copy link

@hebelsan You need rebase this pull request with latest master branch. Please check.

@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 13, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 13, 2025
@gardener-robot-ci-1 gardener-robot-ci-1 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jan 14, 2025
@hebelsan hebelsan force-pushed the feature/os-clientv2 branch from 11fa368 to 339ce0d Compare January 14, 2025 11:32
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2025
@hebelsan
Copy link
Contributor Author

/test

@testmachinery
Copy link

testmachinery bot commented Jan 14, 2025

Testrun: e2e-c7n5l
Workflow: e2e-c7n5l-wf
Phase: Failed

+---------------------+-----------------------------+--------+----------+
|        NAME         |            STEP             | PHASE  | DURATION |
+---------------------+-----------------------------+--------+----------+
| infrastructure-test | infrastructure-test         | Failed | 10m57s   |
| infrastructure-test | infrastructure-test-flow    | Failed | 10m57s   |
| infrastructure-test | infrastructure-test-migrate | Failed | 10m56s   |
| infrastructure-test | infrastructure-test-recover | Failed | 10m57s   |
| bastion-test        | bastion-test                | Failed | 10m51s   |
+---------------------+-----------------------------+--------+----------+

@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2025
@gardener-robot-ci-1 gardener-robot-ci-1 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2025
@hebelsan
Copy link
Contributor Author

the infra test work with basic credentials, but something is still missing to make the client working with app credentials (what the tm is using).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/enhancement Enhancement, improvement, extension needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/rebase Needs git rebase needs/review Needs review needs/second-opinion Needs second review by someone else platform/openstack OpenStack platform/infrastructure size/xl Size of pull request is huge (see gardener-robot robot/bots/size.py)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants