Skip to content

Commit d5897da

Browse files
committed
Change junit output to $ARTIFACTS
It seems that's what prow uses Asking prow to set WORKSPACE broke the e2e tests, so I assume that env-var is already used
1 parent f7f44c3 commit d5897da

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

vertical-pod-autoscaler/hack/run-e2e-locally.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,9 +103,9 @@ case ${SUITE} in
103103

104104
echo " ** Running suite ${SUITE}"
105105
if [ ${SUITE} == recommender-externalmetrics ]; then
106-
WORKSPACE=./workspace/_artifacts ${SCRIPT_ROOT}/hack/run-e2e-tests.sh recommender
106+
ARTIFACTS=./workspace/_artifacts ${SCRIPT_ROOT}/hack/run-e2e-tests.sh recommender
107107
else
108-
WORKSPACE=./workspace/_artifacts ${SCRIPT_ROOT}/hack/run-e2e-tests.sh ${SUITE}
108+
ARTIFACTS=./workspace/_artifacts ${SCRIPT_ROOT}/hack/run-e2e-tests.sh ${SUITE}
109109
fi
110110
;;
111111
*)

vertical-pod-autoscaler/hack/run-e2e-tests.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ export GO111MODULE=on
4747
ABSOLUTE_PATH=$(realpath "${SCRIPT_ROOT}")
4848
export GOBIN="${ABSOLUTE_PATH}/e2e/_output/bin"
4949

50-
export WORKSPACE=${WORKSPACE:-/workspace/_artifacts}
50+
export ARTIFACTS=${ARTIFACTS:-/workspace/_artifacts}
5151

5252
SKIP="--ginkgo.skip=\[Feature\:OffByDefault\]"
5353

@@ -62,7 +62,7 @@ case ${SUITE} in
6262
export KUBECONFIG=$HOME/.kube/config
6363
pushd ${SCRIPT_ROOT}/e2e
6464
go install github.com/onsi/ginkgo/v2/ginkgo
65-
${GOBIN}/ginkgo build v1/ && ${GOBIN}/ginkgo --nodes=$NUMPROC --focus="\[VPA\] \[${SUITE}\]" v1/v1.test -- --report-dir=${WORKSPACE} --disable-log-dump ${SKIP}
65+
${GOBIN}/ginkgo build v1/ && ${GOBIN}/ginkgo --nodes=$NUMPROC --focus="\[VPA\] \[${SUITE}\]" v1/v1.test -- --report-dir=${ARTIFACTS} --disable-log-dump ${SKIP}
6666
V1_RESULT=$?
6767
popd
6868
echo v1 test result: ${V1_RESULT}

0 commit comments

Comments
 (0)