Skip to content

Commit a0f195c

Browse files
authored
Merge pull request #106 from msau42/fix-canary
Only set staging registry when running canary job
2 parents b3c65f9 + 7100c12 commit a0f195c

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

Diff for: prow.sh

+5-1
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ configvar CSI_PROW_DRIVER_INSTALL "install_csi_driver" "name of the shell functi
212212
configvar CSI_PROW_DRIVER_CANARY "${CSI_PROW_HOSTPATH_CANARY}" "driver image override for canary images"
213213

214214
# Image registry to use for canary images.
215-
# Only valid if CSI_PROW_DRIVER_CANARY is set.
215+
# Only valid if CSI_PROW_DRIVER_CANARY == "canary".
216216
configvar CSI_PROW_DRIVER_CANARY_REGISTRY "gcr.io/k8s-staging-sig-storage" "registry for canary images"
217217

218218
# The E2E testing can come from an arbitrary repo. The expectation is that
@@ -697,7 +697,11 @@ install_csi_driver () {
697697
fi
698698

699699
if [ "${CSI_PROW_DRIVER_CANARY}" != "stable" ]; then
700+
if [ "${CSI_PROW_DRIVER_CANARY}" == "canary" ]; then
700701
images="$images IMAGE_TAG=${CSI_PROW_DRIVER_CANARY} IMAGE_REGISTRY=${CSI_PROW_DRIVER_CANARY_REGISTRY}"
702+
else
703+
images="$images IMAGE_TAG=${CSI_PROW_DRIVER_CANARY}"
704+
fi
701705
fi
702706
# Ignore: Double quote to prevent globbing and word splitting.
703707
# It's intentional here for $images.

0 commit comments

Comments
 (0)