From fdf028167eb3c4c0601901d38123e72d52a95f89 Mon Sep 17 00:00:00 2001 From: youkaichao Date: Tue, 14 Jan 2025 19:24:06 +0800 Subject: [PATCH] [HPU][misc] add comments for explanation (#12034) Signed-off-by: youkaichao Signed-off-by: Bowen Wang --- .buildkite/run-hpu-test.sh | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/.buildkite/run-hpu-test.sh b/.buildkite/run-hpu-test.sh index bdbb5195ce95f..1edcb1d2669e9 100644 --- a/.buildkite/run-hpu-test.sh +++ b/.buildkite/run-hpu-test.sh @@ -8,6 +8,11 @@ set -ex docker build -t hpu-test-env -f Dockerfile.hpu . # Setup cleanup +# certain versions of HPU software stack have a bug that can +# override the exit code of the script, so we need to use +# separate remove_docker_container and remove_docker_container_and_exit +# functions, while other platforms only need one remove_docker_container +# function. EXITCODE=1 remove_docker_container() { docker rm -f hpu-test || true; } remove_docker_container_and_exit() { remove_docker_container; exit $EXITCODE; }