From cd695aa09f0e7c2ef4f7f36907b40dda90a22204 Mon Sep 17 00:00:00 2001 From: elliot-barn Date: Mon, 15 Jun 2026 23:10:54 +0000 Subject: [PATCH] 0 retries for exit code 42 (test failures) Signed-off-by: elliot-barn --- raycicmd/bk_pipeline.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/raycicmd/bk_pipeline.go b/raycicmd/bk_pipeline.go index 1378063a..37a11afb 100644 --- a/raycicmd/bk_pipeline.go +++ b/raycicmd/bk_pipeline.go @@ -246,12 +246,13 @@ func makeRayDockerPlugin( } // makeAutomaticRetryConfig creates the retry configuration for rayci pipelines. -// The retry configuration is to retry once for any unknown exit status or -// test failures, and to retry 3 times for known exit statuses. +// The retry configuration is to retry once for any unknown exit status and to +// retry 3 times for known exit statuses. Test failures (exit_status 42) are not +// retried here since bazel already retries those. func makeAutomaticRetryConfig(exitStatus []int) []any { m := []any{ - map[string]int{"exit_status": 1, "limit": 1}, // unknown exist status - map[string]int{"exit_status": 42, "limit": 1}, // test failures + map[string]int{"exit_status": 1, "limit": 1}, // unknown exist status + // exit_status 42 (test failures) omitted: bazel already retries those. } for _, s := range exitStatus { m = append(m, map[string]any{"exit_status": s, "limit": 3})