Skip to content

ci/functional: fix ubuntu version & add antelope#2762

Merged
mandre merged 5 commits intogophercloud:masterfrom
EmilienM:antelope
Sep 8, 2023
Merged

ci/functional: fix ubuntu version & add antelope#2762
mandre merged 5 commits intogophercloud:masterfrom
EmilienM:antelope

Conversation

@EmilienM
Copy link
Copy Markdown
Contributor

@EmilienM EmilienM commented Sep 6, 2023

  • add antelope release in the matrix
  • switch master jobs to run on ubuntu 22.04
  • switch tests to use lioadm (supported and stable) and not tgtd (broken)

Fixes #2756

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.
@EmilienM
Copy link
Copy Markdown
Contributor Author

EmilienM commented Sep 7, 2023

The failures seems to be related to volume attachment on the Ubuntu 22.04, we didn't have these issues before I think and this PR bumps the Ubuntu version.

I'll dig that out.

@EmilienM
Copy link
Copy Markdown
Contributor Author

EmilienM commented Sep 7, 2023

The error seems to happen when running iscsiadm --login against the target which prevents the volume to be attached to the server, therefore all the tests related to volumes fail.

https://paste.opendev.org/show/bZpwmV242HFL1GwNDwbK/

@pierreprinetti
Copy link
Copy Markdown
Member

Do you think we'd get fewer failures by spinning off the Ubuntu bump to another PR?

@EmilienM
Copy link
Copy Markdown
Contributor Author

EmilienM commented Sep 7, 2023

Do you think we'd get fewer failures by spinning off the Ubuntu bump to another PR?

The issue will be the same on Antelope and Master; so I'll use this PR.

@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels Sep 7, 2023
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.
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels Sep 7, 2023
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
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels Sep 7, 2023
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.

LGTM
Let's merge what we have know and fix the remaining failures in subsequent commits.

@mandre mandre merged commit 1177454 into gophercloud:master Sep 8, 2023
@EmilienM EmilienM deleted the antelope branch September 8, 2023 13:21
@EmilienM EmilienM added the backport-v1 This PR will be backported to v1 label Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v1 This PR will be backported to v1 semver:patch No API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CI jobs for new branches of OpenStack

3 participants