Skip to content

Commit

Permalink
Intel/CI: Resolve OneCCL summary bug
Browse files Browse the repository at this point in the history
A bug was found in oneccl summary in counting passed tests
logic because the log file logged it differently.
Expected log:
[0] [  PASSED  ] [0] 384 tests.
Logged line:
[0] [  PASSED  ]
[0] 384 tests.

The fix is to check whether there are any lines having 'tests.'
but not having [0] [  PASSED  ] and [1] [  PASSED  ] in line.

Signed-off-by: Juee Himalbhai Desai <[email protected]>
  • Loading branch information
Juee14Desai authored and zachdworkin committed Feb 13, 2024
1 parent 1c47802 commit d6f8859
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion contrib/intel/jenkins/summary.py
Original file line number Diff line number Diff line change
Expand Up @@ -417,7 +417,9 @@ def check_pass(self, line):
if '[0] PASSED' in line or "All done" in line:
self.passes += 1
self.passed_tests.append(f"{self.name}: 1")
if '[0] [ PASSED ]' in line:
if ("[0] [ PASSED ]" in line and "tests." in line) or \
("tests." in line and "[1] [ PASSED ]" not in line and \
"[0] [ PASSED ]" not in line):
token = line.split()
no_of_tests = f"{token[token.index('tests.') - 1]} "
self.passes += int(no_of_tests)
Expand Down

0 comments on commit d6f8859

Please sign in to comment.