Skip to content

[v1] (clean manual backport) ci/functional: fix ubuntu version & add antelope#2783

Merged
mandre merged 5 commits intogophercloud:v1from
EmilienM:backport_ci_fix
Sep 22, 2023
Merged

[v1] (clean manual backport) ci/functional: fix ubuntu version & add antelope#2783
mandre merged 5 commits intogophercloud:v1from
EmilienM:backport_ci_fix

Conversation

@EmilienM
Copy link
Copy Markdown
Contributor

  • acceptance/volumeattachment: don't specify Connector
  • ci/functional: switch LIO target instead of tgt
  • ci/functional: add antelope release in the matrix
  • ci/functional: reduce LOC
  • ci/functional: switch master jobs to run on ubuntu 22.04

With `lioadm` being the recommended iscsi helper, we need a real IQN;
not a fake one. Let's just not specify the connector.

Related gophercloud#2768

(cherry picked from commit 440873f)
tgt has been broken for some time in the upstream CI and we were told by
Cinder devs to use lioadm for the helper. Let's switch to it for more
stable jobs.

(cherry picked from commit f4ed08a)
(cherry picked from commit e2a497e)
Since openstack/devstack@427a4e1
We don't support focal anymore in devstack-master. We need to run 22.04
from now.

Note: some workflows already had it right but it wasn't consistent
across all services.

(cherry picked from commit 9a8b0de)
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 79.122%. remained the same when pulling 5ac547f on EmilienM:backport_ci_fix into 48897e2 on gophercloud:v1.

Copy link
Copy Markdown
Contributor

@mandre mandre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified it's a clean backport.

Just curious about why you had to manually create the backport. I see you added the label to #2762 and the workflow ran and was successful. Is that because we were missing #2780 in the branch?

@mandre mandre merged commit 23e5dbd into gophercloud:v1 Sep 22, 2023
@pierreprinetti pierreprinetti added the v1 This PR targets v1 label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:patch No API change v1 This PR targets v1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants