sleep_time_check.py failed to parse suspend/resume time
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox Provider - Base |
Expired
|
High
|
Pierre Equoy | ||
OEM Priority Project |
New
|
Undecided
|
Unassigned |
Bug Description
[Description]
sleep_time_check.py failed to parse
# find our times
for line in lines:
try:
if "Average time to sleep" in line:
elif "Average time to resume" in line:
except ValueError as e:
return 1
I guess the "Average time to sleep/resume" likely missing or showing unexpected output.
In the failure case, the checkbox submission doesn't include "pm_test.
Thus, we won't know what's going on.
[Summary]
power-managemen
[Steps to reproduce]
Run checkbox-cli stress test
[Checkbox job `com.canonical.
stdout
------
ERROR: One or more times was not reported correctly:
could not convert string to float: 'N/A'
tags: | added: oem-priority originate-from-1981804 somerville |
tags: | added: originate-from-1950728 stella |
tags: | added: originate-from-1950737 |
tags: | added: originate-from-1970877 |
Changed in plainbox-provider-checkbox: | |
importance: | Undecided → High |
status: | New → Confirmed |
assignee: | nobody → jeremyszu (os369510) |
tags: | added: originate-from-1950729 |
tags: | added: ce-qa-concern |
Changed in plainbox-provider-checkbox: | |
assignee: | jeremyszu (os369510) → Pierre Equoy (pieq) |
Changed in plainbox-provider-checkbox: | |
milestone: | 2.0.0 → 2.1.0 |
tags: | added: originate-from-1999908 |
tags: | added: originate-from-1999916 |
tags: | added: originate-from-1999918 |
tags: | added: originate-from-1999919 |
tags: | added: originate-from-1999923 |
tags: | added: originate-from-1999927 |
tags: | added: originate-from-2008204 |
I suspect the logs generated by fwts_test.py spit out "N/A", and it makes checking the time impossible:
https:/ /github. com/canonical/ checkbox/ blob/main/ checkbox- support/ checkbox_ support/ scripts/ fwts_test. py#L128- L130
I will need to run this on an actual device and capture the logs to see what's the content of the file before being processed.
We see this error more and more, so I think this is important to get fixed.