more jobs that use tee but not pipefail discovered

Bug #1096948 reported by Jeff Lane 
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
Medium
Jeff Lane 

Bug Description

Jeffrey Chang pointed out two more jobs that use tee, but don't properly use pipefail to ensure checkbox gets the correct exit code.

He suggests:

bluetooth/detect-output
suspend/network-before-suspend

looking at bluetooth/detect-output, it appears that there are several jobs that depend on this test passing.

with suspend/network_before_suspend, however, I think the behaviour should remain the same as it is interesting to see if network passes before suspend but not after, and vice versa.

Related branches

Jeff Lane  (bladernr)
Changed in checkbox:
status: New → In Progress
assignee: nobody → Jeff Lane (bladernr)
importance: Undecided → Medium
Revision history for this message
Jeff Lane  (bladernr) wrote :

actually, after looking further, the original intent was to have suspend/network_after_suspend only run on success of suspend/network_before_suspend, so I have also applied the pipefail fix to network_before_suspend as well.

Changed in checkbox:
status: In Progress → Fix Committed
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.