Remove doozer env vars from ci-openshift golang images

ci_images/rhel-8/ci-openshift-build-root/Dockerfile

@@ -73,3 +73,8 @@ RUN mkdir -p $GOPATH && \
 # Some image building tools don't create a missing WORKDIR
 RUN mkdir -p /go/src/github.com/openshift/origin
 WORKDIR /go/src/github.com/openshift/origin
+
+# These services run on the build farms and serve RPMs to CI workloads.
+# This environment variable is used by dnf_wrapper.sh to know which service
+# to acquire repo information from.
+ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc

ci_images/rhel-8/ci-openshift-golang-builder/Dockerfile

@@ -45,3 +45,8 @@ ADD .oit/unsigned.repo $DNF_WRAPPER_DIR/
 # Some image building tools don't create a missing WORKDIR
 RUN mkdir -p /go/src/github.com/openshift/origin
 WORKDIR /go/src/github.com/openshift/origin
+
+# These services run on the build farms and serve RPMs to CI workloads.
+# This environment variable is used by dnf_wrapper.sh to know which service
+# to acquire repo information from.
+ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc

ci_images/rhel-9/ci-openshift-build-root/Dockerfile

@@ -73,3 +73,8 @@ RUN mkdir -p $GOPATH && \
 # Some image building tools don't create a missing WORKDIR
 RUN mkdir -p /go/src/github.com/openshift/origin
 WORKDIR /go/src/github.com/openshift/origin
+
+# These services run on the build farms and serve RPMs to CI workloads.
+# This environment variable is used by dnf_wrapper.sh to know which service
+# to acquire repo information from.
+ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc

ci_images/rhel-9/ci-openshift-golang-builder/Dockerfile

@@ -38,3 +38,7 @@ ENV PATH=$DNF_WRAPPER_DIR:$PATH
 # it is not running on a build farm.
 ADD .oit/unsigned.repo $DNF_WRAPPER_DIR/
 
+# These services run on the build farms and serve RPMs to CI workloads.
+# This environment variable is used by dnf_wrapper.sh to know which service
+# to acquire repo information from.
+ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc

images/ci-openshift-build-root-latest.rhel8.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-8/ci-openshift-build-root/Dockerfile
     git:
@@ -14,6 +16,10 @@ content:
       mirror: true
       mirror_manifest_list: true
       upstream_image: registry.ci.openshift.org/openshift/release:rhel-8-release-golang-{GO_LATEST}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
 from:
   member: ci-openshift-golang-builder-latest.rhel8
 labels:
@@ -41,12 +47,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false

images/ci-openshift-build-root-latest.rhel9.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-9/ci-openshift-build-root/Dockerfile
     git:
@@ -14,6 +16,11 @@ content:
       mirror: true
       mirror_manifest_list: true
       upstream_image: registry.ci.openshift.org/openshift/release:rhel-9-release-golang-{GO_LATEST}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
+
 from:
   member: ci-openshift-golang-builder-latest.rhel9
 labels:
@@ -41,12 +48,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false

images/ci-openshift-build-root-previous.rhel8.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-8/ci-openshift-build-root/Dockerfile
     git:
@@ -14,6 +16,11 @@ content:
       mirror: true
       mirror_manifest_list: true
       upstream_image: registry.ci.openshift.org/openshift/release:rhel-8-release-golang-{GO_PREVIOUS}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
+
 from:
   member: ci-openshift-golang-builder-previous.rhel8
 labels:
@@ -41,12 +48,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false

images/ci-openshift-build-root-previous.rhel9.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-9/ci-openshift-build-root/Dockerfile
     git:
@@ -14,6 +16,11 @@ content:
       mirror: true
       mirror_manifest_list: true
       upstream_image: registry.ci.openshift.org/openshift/release:rhel-9-release-golang-{GO_PREVIOUS}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
+
 from:
   member: ci-openshift-golang-builder-previous.rhel9
 labels:
@@ -41,12 +48,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false

images/ci-openshift-golang-builder-latest.rhel8.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-8/ci-openshift-golang-builder/Dockerfile
     git:
@@ -16,6 +18,10 @@ content:
       upstream_image: registry.ci.openshift.org/ocp/builder:rhel-8-golang-{GO_LATEST}-openshift-{MAJOR}.{MINOR}
       upstream_image_mirror:
       - registry.ci.openshift.org/ocp-private/builder-priv:rhel-8-golang-{GO_LATEST}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
 
 from:
   stream: rhel-8-golang-{GO_LATEST}
@@ -45,12 +51,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false

images/ci-openshift-golang-builder-latest.rhel9.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-9/ci-openshift-golang-builder/Dockerfile
     git:
@@ -16,6 +18,10 @@ content:
       upstream_image: registry.ci.openshift.org/ocp/builder:rhel-9-golang-{GO_LATEST}-openshift-{MAJOR}.{MINOR}
       upstream_image_mirror:
       - registry.ci.openshift.org/ocp-private/builder-priv:rhel-9-golang-{GO_LATEST}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
 
 from:
   stream: rhel-9-golang-{GO_LATEST}
@@ -44,12 +50,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false

images/ci-openshift-golang-builder-previous.rhel8.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-8/ci-openshift-golang-builder/Dockerfile
     git:
@@ -16,6 +18,11 @@ content:
       upstream_image: registry.ci.openshift.org/ocp/builder:rhel-8-golang-{GO_PREVIOUS}-openshift-{MAJOR}.{MINOR}
       upstream_image_mirror:
       - registry.ci.openshift.org/ocp-private/builder-priv:rhel-8-golang-{GO_PREVIOUS}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
+
 
 from:
   stream: rhel-8-golang-{GO_PREVIOUS}
@@ -45,12 +52,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel8.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false

images/ci-openshift-golang-builder-previous.rhel9.yml

@@ -1,6 +1,8 @@
 content:
-  # set_build_variables is necessary in order to set CI_RPM_SVC (see .envs below) in the Dockerfile.
-  set_build_variables: true
+  # set_build_variables to false, as setting git commit env vars break certain
+  # workloads. Set env vars in the Dockerfile, and use modifications if they
+  # should be variable
+  set_build_variables: false
   source:
     dockerfile: ci_images/rhel-9/ci-openshift-golang-builder/Dockerfile
     git:
@@ -16,6 +18,10 @@ content:
       upstream_image: registry.ci.openshift.org/ocp/builder:rhel-9-golang-{GO_PREVIOUS}-openshift-{MAJOR}.{MINOR}
       upstream_image_mirror:
       - registry.ci.openshift.org/ocp-private/builder-priv:rhel-9-golang-{GO_PREVIOUS}-openshift-{MAJOR}.{MINOR}
+    modifications:
+    - action: replace
+      match: "ENV CI_RPM_SVC=base-MAJOR-MINOR-rhelX.ocp.svc"
+      replacement: "ENV CI_RPM_SVC=base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
 
 from:
   stream: rhel-9-golang-{GO_PREVIOUS}
@@ -44,12 +50,6 @@ owners:
 maintainer:
   component: Release
 
-envs:
-  # These services run on the build farms and serve RPMs to CI workloads.
-  # This environment variable is used by dnf_wrapper.sh to know which service
-  # to acquire repo information from.
-  CI_RPM_SVC: "base-{MAJOR}-{MINOR}-rhel9.ocp.svc"
-
 # The from: stanza is exactly what we want to use. So ignore anything in
 # the Dockerfile.
 canonical_builders_from_upstream: false