buildkite: don't customize runtime name

Use the same runtime name for all jobs. The runtime name is passed to test and
if it is different each time, test results are not cached and test are re-run
even without code changes.

PiperOrigin-RevId: 607439704
This commit is contained in:
Andrei Vagin
2024-02-15 13:40:36 -08:00
committed by gVisor bot
parent 631cdcef17
commit 5388e58e1f
2 changed files with 5 additions and 7 deletions
+1 -2
View File
@@ -56,8 +56,7 @@ if test "${BUILDKITE_COMMAND_EXIT_STATUS}" -ne "0"; then
set -x
dmesg -kT | tail -n 50
# LINT.IfChange
runtime="buildkite_runtime_${BUILDKITE_BRANCH}-${BUILDKITE_BUILD_ID}"
runtime="$(echo "$runtime" | sed -r 's~[^-_a-z0-9]+~_~g')"
runtime="buildkite_runtime"
# LINT.ThenChange(pre-command)
if [ -d "/tmp/${runtime}/" ]; then
tar -czf "/tmp/${BUILDKITE_JOB_ID}.tar.gz" -C /tmp/ "${runtime}"
+4 -5
View File
@@ -64,13 +64,12 @@ if [[ "${BUILDKITE_BRANCH}" =~ ^test/ ]]; then
# LINT.ThenChange(../../tools/make_release.sh)
fi
# Set the system-wide Docker runtime name after the BuildKite branch name.
# Remove any slashes and other characters that are not supported in Docker
# runtime names. The runtime name must be a single valid path component.
# LINT.IfChange
export RUNTIME="buildkite_runtime_${BUILDKITE_BRANCH}-${BUILDKITE_BUILD_ID}"
export RUNTIME="$(echo "$RUNTIME" | sed -r 's~[^-_a-z0-9]+~_~g')"
export RUNTIME="buildkite_runtime"
# LINT.ThenChange(post-command)
if [ -d "/tmp/${RUNTIME}/" ]; then
sudo rm -rf "/tmp/${RUNTIME}/"
fi
# If running in a container, set the reload command appropriately.
if [[ -x /tmp/buildkite-reload-host-docker/reload ]]; then