Skip to content

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 <jmulligan@redhat.com>
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
@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