sleep_time_check script does not return the correct return code in some cases
Bug #1569191 reported by
Pierre Equoy
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox Provider - Base |
Fix Released
|
Medium
|
Pierre Equoy |
Bug Description
In some cases, the sleep_time_check script returns "0" (i.e. no problem) although there are errors, leading Checkbox to think the job using this script has passed when it has actually failed.
Steps to reproduce:
1. Create a empty file:
touch /tmp/test.log
2. Call the script using this file:
./sleep_
ERROR: One or more times was not reported correctly
$ echo $?
0
Related branches
lp:~pieq/checkbox/fix-1569191-stress-test-log-time-check
- Sylvain Pineau (community): Approve
-
Diff: 40 lines (+6/-6)1 file modifiedproviders/plainbox-provider-checkbox/bin/sleep_time_check (+6/-6)
Changed in plainbox-provider-checkbox: | |
importance: | Undecided → Medium |
assignee: | nobody → Pierre Equoy (pierre-equoy) |
Changed in plainbox-provider-checkbox: | |
status: | New → Confirmed |
milestone: | none → 0.28 |
Changed in plainbox-provider-checkbox: | |
status: | Confirmed → In Progress |
Changed in plainbox-provider-checkbox: | |
status: | In Progress → Fix Committed |
milestone: | 0.28 → 0.27 |
Changed in plainbox-provider-checkbox: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.