buildscripts, psm interop: Don't fail target if sub-target already failed (grpc#10237)

XuanWang-Amos · web-flow · commit d0ef5feb9db7 · 2023-06-02T16:05:50.000-07:00
We configured TestGrid to file bug separately for each failed sub-target, if we still fail the main target, TestGrid will fail duplicate bugs. The same change in core: grpc/grpc#33222.
diff --git a/buildscripts/kokoro/psm-security.sh b/buildscripts/kokoro/psm-security.sh
@@ -180,9 +180,6 @@ main() {
     run_test $test || (( ++failed_tests ))
   done
   echo "Failed test suites: ${failed_tests}"
-  if (( failed_tests > 0 )); then
-    exit 1
-  fi
 }
 
 main "$@"
diff --git a/buildscripts/kokoro/xds_k8s_lb.sh b/buildscripts/kokoro/xds_k8s_lb.sh
@@ -181,9 +181,6 @@ main() {
     run_test $test || (( ++failed_tests ))
   done
   echo "Failed test suites: ${failed_tests}"
-  if (( failed_tests > 0 )); then
-    exit 1
-  fi
 }
 
 main "$@"

Original file line number	Diff line number	Diff line change
`@@ -180,9 +180,6 @@ main() {`
`180`	`180`	`run_test $test \|\| (( ++failed_tests ))`
`181`	`181`	`done`
`182`	`182`	`echo "Failed test suites: ${failed_tests}"`
`183`		`- if (( failed_tests > 0 )); then`
`184`		`- exit 1`
`185`		`- fi`
`186`	`183`	`}`
`187`	`184`
`188`	`185`	`main "$@"`
Original file line number	Diff line number	Diff line change
`@@ -181,9 +181,6 @@ main() {`
`181`	`181`	`run_test $test \|\| (( ++failed_tests ))`
`182`	`182`	`done`
`183`	`183`	`echo "Failed test suites: ${failed_tests}"`
`184`		`- if (( failed_tests > 0 )); then`
`185`		`- exit 1`
`186`		`- fi`
`187`	`184`	`}`
`188`	`185`
`189`	`186`	`main "$@"`