HDDS-5182. Acceptance test may exit with 0 in case of error

[adoroszlai]

What changes were proposed in this pull request?

If any acceptance test fails, test-all.sh (and in turn acceptance.sh) should exit with error code (1). But if a successful test is run after a failing one, it will now wrongly exit with success (0).

$ export OZONE_TEST_SELECTOR='failing1\|ozone-csi'
$ ./hadoop-ozone/dev-support/checks/acceptance.sh
...
$ echo $?
0

The problem is that the variable ret in the functions run_test_scripts and run_test_script are intended to be independent, but are the same (global) one.

https://issues.apache.org/jira/browse/HDDS-5182

How was this patch tested?

$ export OZONE_TEST_SELECTOR='failing1\|ozone-csi'
$ ./hadoop-ozone/dev-support/checks/acceptance.sh
...
$ echo $?
1

[cxorm]

Thank you @adoroszlai for finding and fixing it, I'm +1.

[adoroszlai]

Thanks @cxorm for the review.