Skip to content

Commit a87ce83

Browse files
authored
Merge pull request #311 from pohly/prow-update-master
master: update release-tools
2 parents 2a06cb4 + 70b9571 commit a87ce83

File tree

1 file changed

+13
-15
lines changed

1 file changed

+13
-15
lines changed

release-tools/prow.sh

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@
3333
# The expected environment is:
3434
# - $GOPATH/src/<import path> for the repository that is to be tested,
3535
# with PR branch merged (when testing a PR)
36-
# - optional: bazel installed (when testing against Kubernetes master),
37-
# must be recent enough for Kubernetes master
3836
# - running on linux-amd64
3937
# - kind (https://github.com/kubernetes-sigs/kind) installed
4038
# - optional: Go already installed
@@ -160,9 +158,6 @@ kindest/node:v1.16.15@sha256:430c03034cd856c1f1415d3e37faf35a3ea9c5aaa2812117b79
160158
kindest/node:v1.15.12@sha256:8d575f056493c7778935dd855ded0e95c48cb2fab90825792e8fc9af61536bf9
161159
kindest/node:v1.14.10@sha256:6033e04bcfca7c5f2a9c4ce77551e1abf385bcd2709932ec2f6a9c8c0aff6d4f" "kind images"
162160

163-
# Use kind node-image --type=bazel by default, but allow to disable that.
164-
configvar CSI_PROW_USE_BAZEL true "use Bazel during 'kind node-image' invocation"
165-
166161
# By default, this script tests sidecars with the CSI hostpath driver,
167162
# using the install_csi_driver function. That function depends on
168163
# a deployment script that it searches for in several places:
@@ -591,17 +586,12 @@ start_cluster () {
591586
if [ "$version" = "latest" ]; then
592587
version=master
593588
fi
594-
if ${CSI_PROW_USE_BAZEL}; then
595-
type="bazel"
596-
else
597-
type="docker"
598-
fi
599589
git_clone https://github.com/kubernetes/kubernetes "${CSI_PROW_WORK}/src/kubernetes" "$(version_to_git "$version")" || die "checking out Kubernetes $version failed"
600590

601591
go_version="$(go_version_for_kubernetes "${CSI_PROW_WORK}/src/kubernetes" "$version")" || die "cannot proceed without knowing Go version for Kubernetes"
602592
# Changing into the Kubernetes source code directory is a workaround for https://github.com/kubernetes-sigs/kind/issues/1910
603593
# shellcheck disable=SC2046
604-
(cd "${CSI_PROW_WORK}/src/kubernetes" && run_with_go "$go_version" kind build node-image --image csiprow/node:latest $(if [ "$CSI_PROW_KIND_VERSION" != "main" ]; then echo --type="$type"; fi) --kube-root "${CSI_PROW_WORK}/src/kubernetes") || die "'kind build node-image' failed"
594+
(cd "${CSI_PROW_WORK}/src/kubernetes" && run_with_go "$go_version" kind build node-image --image csiprow/node:latest --kube-root "${CSI_PROW_WORK}/src/kubernetes") || die "'kind build node-image' failed"
605595
csi_prow_kind_have_kubernetes=true
606596
fi
607597
image="csiprow/node:latest"
@@ -996,9 +986,17 @@ run_e2e () (
996986
run_sanity () (
997987
install_sanity || die "installing csi-sanity failed"
998988
989+
if [[ "${CSI_PROW_SANITY_POD}" =~ " " ]]; then
990+
# Contains spaces, more complex than a simple pod name.
991+
# Evaluate as a shell command.
992+
pod=$(eval "${CSI_PROW_SANITY_POD}") || die "evaluation failed: CSI_PROW_SANITY_POD=${CSI_PROW_SANITY_POD}"
993+
else
994+
pod="${CSI_PROW_SANITY_POD}"
995+
fi
996+
999997
cat >"${CSI_PROW_WORK}/mkdir_in_pod.sh" <<EOF
1000998
#!/bin/sh
1001-
kubectl exec "${CSI_PROW_SANITY_POD}" -c "${CSI_PROW_SANITY_CONTAINER}" -- mkdir "\$@" && echo "\$@"
999+
kubectl exec "$pod" -c "${CSI_PROW_SANITY_CONTAINER}" -- mkdir "\$@" && echo "\$@"
10021000
EOF
10031001
# Using "rm -rf" as fallback for "rmdir" is a workaround for:
10041002
# Node Service
@@ -1023,8 +1021,8 @@ EOF
10231021
# why it happened.
10241022
cat >"${CSI_PROW_WORK}/rmdir_in_pod.sh" <<EOF
10251023
#!/bin/sh
1026-
if ! kubectl exec "${CSI_PROW_SANITY_POD}" -c "${CSI_PROW_SANITY_CONTAINER}" -- rmdir "\$@"; then
1027-
kubectl exec "${CSI_PROW_SANITY_POD}" -c "${CSI_PROW_SANITY_CONTAINER}" -- rm -rf "\$@"
1024+
if ! kubectl exec "$pod" -c "${CSI_PROW_SANITY_CONTAINER}" -- rmdir "\$@"; then
1025+
kubectl exec "$pod" -c "${CSI_PROW_SANITY_CONTAINER}" -- rm -rf "\$@"
10281026
exit 1
10291027
fi
10301028
EOF
@@ -1043,7 +1041,7 @@ else
10431041
fi
10441042
SCRIPT
10451043
)
1046-
kubectl exec "${CSI_PROW_SANITY_POD}" -c "${CSI_PROW_SANITY_CONTAINER}" -- /bin/sh -c "\${CHECK_PATH}"
1044+
kubectl exec "$pod" -c "${CSI_PROW_SANITY_CONTAINER}" -- /bin/sh -c "\${CHECK_PATH}"
10471045
EOF
10481046
10491047
chmod u+x "${CSI_PROW_WORK}"/*dir_in_pod.sh

0 commit comments

Comments
 (0)