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

containers: work around centos 8 stream getting archived #995

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

phlogistonjohn
Copy link
Collaborator

@phlogistonjohn phlogistonjohn commented Jun 4, 2024

The centos 8 stream distribution is now EOL and the yum repo mirrors are now offline. However, our CI is designed to test older released versions of ceph where the container images are still using centos 8 stream. Borrow and adapt a line from the ceph-csi project in order to use the centos 8 archives (vault) until either we stop testing these versions of ceph or the ceph project changes the base distro for these release versions.

Infra only change

@phlogistonjohn phlogistonjohn added the no-API This PR does not include any changes to the public API of a go-ceph package label Jun 4, 2024
@phlogistonjohn phlogistonjohn marked this pull request as ready for review June 4, 2024 13:51
@phlogistonjohn
Copy link
Collaborator Author

I observe that the jobs that were failing yesterday have gotten past the package updates and were actually running the go test suites. Therefore I think the workaround is functioning as intended.

The centos 8 stream distribution is now EOL and the yum repo mirrors are
now offline. However, our CI is designed to test older released versions
of ceph where the container images are still using centos 8 stream.
Borrow and adapt a line from the ceph-csi project in order to use the
centos 8 archives (vault) until either we stop testing these versions of
ceph or the ceph project changes the base distro for these release
versions.

Signed-off-by: John Mulligan <[email protected]>
Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for the quick hack.

@mergify mergify bot merged commit e0e4115 into ceph:master Jun 4, 2024
16 checks passed
@phlogistonjohn phlogistonjohn deleted the jjm-centos-vault branch July 2, 2024 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-API This PR does not include any changes to the public API of a go-ceph package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants