fix: remove -e from set options in parse-test-results.sh (#6)

edespino · web-flow · commit f6fb4296c392 · 2024-11-19T18:02:14.000-08:00
Allow test_results.txt to be processed even when tests fail. Previously,
-e was causing immediate script exit on test failures before GitHub
outputs could be set, preventing proper status reporting and regression
log uploads. Now the script properly handles both pass and fail cases.
diff --git a/build_automation/cloudberry/scripts/parse-test-results.sh b/build_automation/cloudberry/scripts/parse-test-results.sh
@@ -76,7 +76,7 @@
 #
 # --------------------------------------------------------------------
 
-set -euo pipefail
+set -uo pipefail
 
 # Default log file path
 DEFAULT_LOG_PATH="build-logs/details/make-${MAKE_NAME}.log"

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@`
`76`	`76`	`#`
`77`	`77`	`# --------------------------------------------------------------------`
`78`	`78`
`79`		`-set -euo pipefail`
	`79`	`+set -uo pipefail`
`80`	`80`
`81`	`81`	`# Default log file path`
`82`	`82`	`DEFAULT_LOG_PATH="build-logs/details/make-${MAKE_NAME}.log"`