DebugHooksServerSuite.TestRunHook signal: killed

[Curtis Hovey]

As seen at
    http://reports.vapour.ws/releases/issue/57add913749a560da1d91ec1

server_test.go:207:
    c.Assert(err, jc.ErrorIsNil)
... value *exec.ExitError = &exec.ExitError{ProcessState:(*os.ProcessState)(0xc820216560), Stderr:[]uint8(nil)} ("signal: killed")

[Curtis Hovey (sinzui)]

This fails on s390x more often than other arch and it has failed 4 times in one week. Given the good quality tests are in now, this test needs fixing.

[Christian Muirhead (2-xtian)]

I can see the same failure if I add a delay to the goroutine the test starts - this would match the behaviour on a heavily loaded test machine. The flock command in the test has a 5s timeout, but the select loop further down has a 10s timeout, so I've increased the flock command to match.

[Christian Muirhead (2-xtian)]

PR: https://github.com/juju/juju/pull/7038