This is an automated email from the ASF dual-hosted git repository. zhangduo pushed a commit to branch branch-2.4 in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2.4 by this push: new 43486956443 HBASE-27874 Problem in flakey generated report causes pre-commit run to fail (#5244) 43486956443 is described below commit 4348695644355fb4b4c57bff6cb2f48ba6d722da Author: Wellington Ramos Chevreuil <wchevre...@apache.org> AuthorDate: Mon May 22 03:57:10 2023 +0100 HBASE-27874 Problem in flakey generated report causes pre-commit run to fail (#5244) Signed-off-by: Peter Somogyi <psomo...@apache.org> Signed-off-by: Duo Zhang <zhang...@apache.org> Reviewew-by: Kota-SH <shanmukhaharipr...@gmail.com> (cherry picked from commit 0ba562ab4d2045e7377e8dea9dff94fb560b3ab3) --- dev-support/flaky-tests/findHangingTests.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/dev-support/flaky-tests/findHangingTests.py b/dev-support/flaky-tests/findHangingTests.py index bb0a9a5be61..e07638ac7d7 100755 --- a/dev-support/flaky-tests/findHangingTests.py +++ b/dev-support/flaky-tests/findHangingTests.py @@ -62,7 +62,8 @@ def get_bad_tests(console_url): for line in response.content.decode("utf-8").splitlines(): result1 = re.findall("Running org.apache.hadoop.hbase.(.*)", line) if len(result1) == 1: - test_case = result1[0] + # Sometimes the maven build output might have some malformed lines. See HBASE-27874 + test_case = result1[0].split("WARNING")[0].strip() if test_case in all_tests_set: print(("ERROR! Multiple tests with same name '{}'. Might get wrong results " "for this test.".format(test_case))) @@ -71,7 +72,8 @@ def get_bad_tests(console_url): all_tests_set.add(test_case) result2 = re.findall("Tests run:.*?- in org.apache.hadoop.hbase.(.*)", line) if len(result2) == 1: - test_case = result2[0] + # Sometimes the maven build output might have some malformed lines. See HBASE-27874 + test_case = result2[0].split("WARNING")[0].strip() if "FAILURE!" in line: failed_tests_set.add(test_case) if test_case not in hanging_tests_set: