diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2020-04-07 11:51:21 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2020-04-07 15:35:15 +0200 |
commit | 8b898063f718b51732c8b47b6f7704fd8659d2a9 (patch) | |
tree | 0e2f9b25e898d75ae7f5a03627d1c2fd26ef42bc /io | |
parent | 21eb6c98bcfb1d453f8eb9f73bd87f6dd87aa882 (diff) |
Revert "UITest: Actually time-out the wait"
This reverts commit 150b67b57bd25ba9c8ec9c28c7aed3cc0b557bfd.
Reason for revert: With the above change, a hung soffice process would cause
the UITest that spawned it to fail, but that would not terminate the hung
soffice process. This is especially problematic for Gerrit Jenkins builds,
where a leftover hung soffice process from a previous build will typically cause
unrelated subsequent builds to fail. So: For a manual developer build, it is
arguably better to leave the hung UITest in the hung state (both the python and
the soffice process) to be able to debug it. And for an automated Gerrit
Jenkins build it is even vital that the hung UITest hangs the build, so that
Jenkins will eventually abort it and lode's bin/kill-wrapper will reliably kill
any remaining processes, incl. the hung soffice process. (But of course it is
unfortunate that we now get less feedback from a hung UITest.)
Change-Id: I6cc577dda362d3cb71846decae9c2bb6bed90b6b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91717
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'io')
0 files changed, 0 insertions, 0 deletions