Skip to content

Commit

Permalink
changing the place of Dmesg Check
Browse files Browse the repository at this point in the history
  • Loading branch information
umfranci committed Feb 4, 2025
1 parent cc60ac1 commit a6a30e7
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions lisa/testsuite.py
Original file line number Diff line number Diff line change
Expand Up @@ -330,13 +330,6 @@ def _send_result_message(self, stacktrace: Optional[str] = None) -> None:

# get information of default node, and send to notifier.
if self.environment:
nodes = self.environment.nodes
for node in nodes.list():
dmesg_check_result = node.exec_check_dmesg_oops()
if dmesg_check_result:
self.status = TestStatus.FAILED
self.message = f"failed. dmesg oops found: {dmesg_check_result}"
break
# force refresh information, when test result status is changed. The
# refreshed information is not used so far. But in case it's needed
# in future, keep it up to date.
Expand Down Expand Up @@ -766,6 +759,14 @@ def start(
log=case_log,
)

nodes = self.environment.nodes
for node in nodes.list():
dmesg_check_result = node.exec_check_dmesg_oops()
if dmesg_check_result:
case_result.set_status(TestStatus.FAILED, f"failed. dmesg oops found: {dmesg_check_result}")
self.status = TestStatus.FAILED
break

if case_result.status == TestStatus.FAILED:
try:
case_result.capture_serial_console_log()
Expand Down

0 comments on commit a6a30e7

Please sign in to comment.