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

ci: workaround for centos stream 8 being EOLed #4304

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

kolyshkin
Copy link
Contributor

@kolyshkin kolyshkin commented Jun 4, 2024

CS8 is EOL. As a temp workaround, fix repo URLs to point to vault.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin kolyshkin added area/ci backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 labels Jun 4, 2024
@kolyshkin kolyshkin marked this pull request as ready for review June 4, 2024 00:40
@AkihiroSuda
Copy link
Member

DCO check is hanging 🤔

@AkihiroSuda AkihiroSuda closed this Jun 4, 2024
@AkihiroSuda AkihiroSuda reopened this Jun 4, 2024
@thaJeztah
Copy link
Member

DCO check is hanging 🤔

Oh, I saw someone else mention that for another repository; perhaps some outage on GitHub?

Do we need to keep the stream-8, or should we just remove it? (Or is it purely an approximation for "RHEL 8" here?)

@AkihiroSuda
Copy link
Member

Do we need to keep the stream-8, or should we just remove it? (Or is it purely an approximation for "RHEL 8" here?)

Let's keep it for now, this is an approximation for Rocky/Alma/Oracle Linux 8 too.

@thaJeztah
Copy link
Member

Yeah, probably ok for now; CentOS 7 will be EOL soon as well (but IIRC, so will RHEL 7, so ..)

For the DCO check; looks like it's currently broken; also running into that in other repositories now 😞

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Member

@AkihiroSuda
Copy link
Member

Let's merge this to unblock other PRs

@AkihiroSuda AkihiroSuda merged commit 3e8f48d into opencontainers:main Jun 4, 2024
74 checks passed
@lifubang lifubang added backport/1.1-done A PR in main branch which has been backported to release-1.1 and removed backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 labels Jun 4, 2024
@lifubang lifubang mentioned this pull request Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci backport/1.1-done A PR in main branch which has been backported to release-1.1 easy-to-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants