ceph / ceph-csi

CSI driver for Ceph
Apache License 2.0
1.19k stars 527 forks source link

ci: update centos stream 8 baseurl #4658

Closed iPraveenParihar closed 3 weeks ago

iPraveenParihar commented 3 weeks ago

Describe what this PR does

Since CentOS Stream 8 is EOL, this commit updates the config to use vault.centos.org for CentOS Stream 8. This should be removed once the base image (ceph) is updated to a version with a newer CentOS.

update baseurl with: http://vault.centos.org

Checklist:


Show available bot commands These commands are normally not required, but in case of issues, leave any of the following bot commands in an otherwise empty comment in this PR: * `/retest ci/centos/`: retest the `` after unrelated failure (please report the failure too!)
iPraveenParihar commented 3 weeks ago

FYI, CI was failing to build ceph-csi image

#8 [updated_base 4/4] RUN dnf -y update --nobest        && dnf -y install nfs-utils        && dnf clean all        && rm -rf /var/cache/yum
#8 0.555 CentOS Stream 8 - AppStream                     490  B/s |  38  B     00:00    
#8 0.563 Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
#8 ERROR: process "/bin/sh -c dnf -y update --nobest        && dnf -y install nfs-utils        && dnf clean all        && rm -rf /var/cache/yum" did not complete successfully: exit code: 1
------
 > [updated_base 4/4] RUN dnf -y update --nobest        && dnf -y install nfs-utils        && dnf clean all        && rm -rf /var/cache/yum:
0.555 CentOS Stream 8 - AppStream                     490  B/s |  38  B     00:00    
0.563 Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist

CentOS 8 was EOL on 31 May 2024. And quay.io/ceph/ceph:v18 is based on CentOS 8 stream. Probably that is the reason.

pm@dhcp53-176:~$ podman run --rm -it quay.io/ceph/ceph:v18 cat /etc/os-release
NAME="CentOS Stream"
VERSION="8"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="8"
PLATFORM_ID="platform:el8"
PRETTY_NAME="CentOS Stream 8"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:8"
HOME_URL="https://centos.org/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux 8"
REDHAT_SUPPORT_PRODUCT_VERSION="CentOS Stream"

Workaround for now will be to update the centos repo url.

iPraveenParihar commented 3 weeks ago

/test ci/centos/mini-e2e/k8s-1.29

iPraveenParihar commented 3 weeks ago

/test ci/centos/mini-e2e/k8s-1.30

iPraveenParihar commented 3 weeks ago

@Madhu-1, can we skip e2e tests for this PR? I have run one to check if image is getting build or not. That should be enough?

Madhu-1 commented 3 weeks ago

@Madhu-1, can we skip e2e tests for this PR? I have run one to check if image is getting build or not. That should be enough?

@iPraveenParihar yes we can skip it, lets one E2E pass which you have already started

iPraveenParihar commented 3 weeks ago

@Madhu-1, can we skip e2e tests for this PR? I have run one to check if image is getting build or not. That should be enough?

@iPraveenParihar yes we can skip it, lets one E2E pass which you have already started

test has passed!

iPraveenParihar commented 3 weeks ago

@Mergifyio queue

mergify[bot] commented 3 weeks ago

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at *5809628523cd8b68d010fd84a3e95cf0e3832394*
Rakshith-R commented 3 weeks ago

/test ci/centos/k8s-e2e-external-storage

Rakshith-R commented 3 weeks ago

/test ci/centos/mini-e2e

Rakshith-R commented 3 weeks ago

/test ci/centos/mini-e2e-helm

Rakshith-R commented 3 weeks ago

/test ci/centos/upgrade-tests-cephfs

Rakshith-R commented 3 weeks ago

/test ci/centos/upgrade-tests-rbd