Comment 6 for bug 1410501

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

zyga: spineau: well, I think we should fix checkbox-ng and look at all the other parts that load a session to see if they are handling errors correctly
zyga: spineau: one other possibility
zyga: spineau: is that the resumed job is local but (?) somehow it has been re-started rather than loaded from the log file
zyga: spineau: and a remote possibility that we've changed something and there's a subtle change in the job checksum
zyga: spineau: but that's rather strange as we have lots of tests for that
zyga: spineau: so it's probably something else
zyga: spineau: hmm
spineau: zyga: iperf_before_suspend_wifi_auto_wlan0 is generated for sure
zyga: spineau: yep
spineau: by a local job
zyga: spineau: but it's generated from a log file
zyga: spineau: (on resume)
zyga: spineau: unless checkbox-gui actually runs local jobs before resuming stuff
zyga: spineau: in any case, for debugging, I think we could store a copy of every job
zyga: spineau: keyed from the checksum of that job
zyga: spineau: so that the recovery path of a case like this can give an informed opinion
zyga: spineau: like a diff of the canonical form of both jobs
zyga: spineau: it should be simple to add and would help us along the way towards making compatible job upgrades possible