diff options
| author | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-04-25 13:06:39 +0200 |
|---|---|---|
| committer | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-04-25 15:06:04 +0200 |
| commit | 61d8ba2b7b60c3181258508954bd83c99aff1397 (patch) | |
| tree | ceceebe83acd18a6b99a16f7726cb3b01ef96201 /test/end-to-end/with_serve_test_runner.py | |
| parent | 6220355db107942f805eef09d9dd3b399f9b0c72 (diff) | |
| download | justbuild-61d8ba2b7b60c3181258508954bd83c99aff1397.tar.gz | |
test remote runners: Add timeout for running execute and serve
The runners used in tests that rely on execution or serve
endpoints to exist can get stuck waiting for these to become
online if for some reason they cannot be set up. This commit fixes
this issue by setting a reasonable timeout, after which we fail
gracefully.
Diffstat (limited to 'test/end-to-end/with_serve_test_runner.py')
| -rwxr-xr-x | test/end-to-end/with_serve_test_runner.py | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/test/end-to-end/with_serve_test_runner.py b/test/end-to-end/with_serve_test_runner.py index a71758fd..a7e7861b 100755 --- a/test/end-to-end/with_serve_test_runner.py +++ b/test/end-to-end/with_serve_test_runner.py @@ -253,7 +253,15 @@ serve_proc = subprocess.Popen( stderr=servestderr, ) +timeout: int = 30 while not os.path.exists(SERVE_INFO): + if timeout == 0: + result = "FAIL" + stdout = "Failed to start serve service" + serve_proc.terminate() + dump_results() + exit(1) + timeout -= 1 time.sleep(1) with open(SERVE_INFO) as f: |