[infra] Kill test runner's batch compilers with sigkill if sigterm fails.

whesse · commit-bot@chromium.org · commit df509bfbf9a3 · 2019-07-25T14:10:23.000Z
Change-Id: Ica226b2b4d2356d6ef0e7a428eed7a7e2d18fe7e Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110224 Auto-Submit: William Hesse <whesse@google.com> Reviewed-by: Jonas Jensen <jonasfj@google.com> Commit-Queue: William Hesse <whesse@google.com>
diff --git a/pkg/test_runner/lib/src/process_queue.dart b/pkg/test_runner/lib/src/process_queue.dart
@@ -1036,7 +1036,11 @@ class BatchRunnerProcess {
   Future<bool> terminate() {
     if (_process == null) return Future.value(true);
     var terminateCompleter = Completer<bool>();
+    final sigkillTimer = Timer(Duration(seconds: 5), () {
+      _process.kill(io.ProcessSignal.sigkill);
+    });
     _processExitHandler = (_) {
+      sigkillTimer.cancel();
       terminateCompleter.complete(true);
     };
     _process.kill();
