From c61385f5271ae2bbed89ab456d622b10d86f6e24 Mon Sep 17 00:00:00 2001
From: David Galloway <dgallowa@redhat.com>
Date: Tue, 1 Feb 2022 13:36:23 -0500
Subject: [PATCH] ceph-dev*build: Stop using docker-mirror

Non-stream CentOS8 containers no longer work because the BaseOS and AppStream repos moved.  We now use the centos/centos:stream8 container image as our BASEOS_TAG.  Except that container lives on quay.io and not dockerhub.

docker-mirror.front.sepia.ceph.com:5000 can only mirror one registry.  We'll keep it as mirroring dockerhub.

Signed-off-by: David Galloway <dgallowa@redhat.com>
---
 ceph-dev-build/build/build_rpm     |  4 +---
 ceph-dev-new-build/build/build_rpm |  4 +---
 scripts/build_utils.sh             | 12 ------------
 3 files changed, 2 insertions(+), 18 deletions(-)

diff --git a/ceph-dev-build/build/build_rpm b/ceph-dev-build/build/build_rpm
index e07d214c..269ee13e 100644
--- a/ceph-dev-build/build/build_rpm
+++ b/ceph-dev-build/build/build_rpm
@@ -100,10 +100,8 @@ if [[ "$CI_CONTAINER" == true && $DISTRO == "centos" ]] ; then
       # exit 1
     fi
 
-    use_internal_container_registry
-
     cd $WORKSPACE/ceph-container
-    sudo -E CI_CONTAINER=${CI_CONTAINER} SHA1=${SHA1} OSD_FLAVOR=${FLAVOR} CONTAINER_FLAVOR=${BRANCH},${DISTRO},${RELEASE} BASEOS_REGISTRY=${REGISTRY} \
+    sudo -E CI_CONTAINER=${CI_CONTAINER} SHA1=${SHA1} OSD_FLAVOR=${FLAVOR} CONTAINER_FLAVOR=${BRANCH},${DISTRO},${RELEASE} \
       /bin/bash ./contrib/build-push-ceph-container-imgs.sh
     cd $WORKSPACE
     sudo rm -rf ceph-container
diff --git a/ceph-dev-new-build/build/build_rpm b/ceph-dev-new-build/build/build_rpm
index 740daec0..cd9febaf 100644
--- a/ceph-dev-new-build/build/build_rpm
+++ b/ceph-dev-new-build/build/build_rpm
@@ -100,10 +100,8 @@ if [[ $CI_CONTAINER == "true" && $DISTRO == "centos" ]] ; then
       # exit 1
     fi
 
-    use_internal_container_registry
-
     cd $WORKSPACE/ceph-container
-    sudo -E CI_CONTAINER=${CI_CONTAINER} SHA1=${SHA1} OSD_FLAVOR=${FLAVOR} CONTAINER_FLAVOR=${BRANCH},${DISTRO},${RELEASE} BASEOS_REGISTRY=${REGISTRY} \
+    sudo -E CI_CONTAINER=${CI_CONTAINER} SHA1=${SHA1} OSD_FLAVOR=${FLAVOR} CONTAINER_FLAVOR=${BRANCH},${DISTRO},${RELEASE} \
       /bin/bash ./contrib/build-push-ceph-container-imgs.sh
     cd $WORKSPACE
     sudo rm -rf ceph-container
diff --git a/scripts/build_utils.sh b/scripts/build_utils.sh
index a435ecdf..53765a2d 100644
--- a/scripts/build_utils.sh
+++ b/scripts/build_utils.sh
@@ -1555,18 +1555,6 @@ maybe_reset_ci_container() {
     fi
 }
 
-# dockerhub started aggressively rate limiting in November 2020. We can use an internal docker mirror if the builder is in the upstream lab.
-use_internal_container_registry() {
-  if curl -s -k "https://docker-mirror.front.sepia.ceph.com:5000"; then
-    REGISTRY="docker-mirror.front.sepia.ceph.com:5000/library"
-  else
-    if [ -n "${DOCKER_HUB_USERNAME}" ] && [ -n "${DOCKER_HUB_PASSWORD}" ]; then
-      docker login -u "${DOCKER_HUB_USERNAME}" -p "${DOCKER_HUB_PASSWORD}" docker.io
-    fi
-    REGISTRY="docker.io"
-  fi
-}
-
 # NOTE: This function will only work on a Pull Request job!
 docs_pr_only() {
   pushd .
-- 
2.39.5