grahamc/hydra2 - Change 5MVWRWHT7U57UQS4EP3AEDFUL7BDLMWFCMYK32IT6XXS7OS6MQUQC

hydra-evaluator: Don't use the "timeout" command

"timeout" does not propagate PR_SET_PDEATHSIG, so it prevents hydra-eval-jobset from being killed when hydra-evaluator quits.

Created by Eelco Dolstra on March 13, 2017

5MVWRWHT7U57UQS4EP3AEDFUL7BDLMWFCMYK32IT6XXS7OS6MQUQC

Dependencies

In channels

main

Change contents

Deletion in src/hydra-evaluator/hydra-evaluator.cc at line 29 [2.575]
B:BD[2.1072] → [2.1072:1101]
```
    int evalTimeout = 3600;
```

Replacement in src/hydra-evaluator/hydra-evaluator.cc at line 88 [2.575]

B:BD[2.2877] → [2.2877:3022]

            Strings args = { "timeout", "-s", "KILL", std::to_string(evalTimeout), "hydra-eval-jobset", jobset.name.first, jobset.name.second };

[2.2877]

[2.3022]

            Strings args = { "hydra-eval-jobset", jobset.name.first, jobset.name.second };

Insertion in src/script/hydra-eval-jobset at line 33 [5.1319]
[4.500]
[3.294]
```
alarm 3600; # FIXME: make configurable
```