test_hwinject.sh: kill lalpulsar_hwinject on walltime instead of CPU time (!1936) · Merge requests · lscsoft / lalsuite · GitLab

We will be having an extended maintenance of https://git.ligo.org, https://containers.ligo.org, and https://docs.ligo.org on 7 May 2024 starting at approximately 9am PDT and lasting for three to four hours. This is to start the process of syncing the data from the current instance to the re-engineered instance we have been working with GitLab on over the past few months. Starting at approximately 9am GitLab will be put into maintenance mode, meaning that all services will be in read-only mode. Throughout the maintenance there may be several periods of downtime. Please address any comments, concerns or questions to the helpdesk.

Karl Wette requested to merge ANU-CGA/lalsuite:kill-hanging-test-hwinject-walltime into master Aug 03, 2022

Description

Unfortunately !1934 (merged) didn't quite solve the issue of hanging jobs (#575 (closed)), so here's another go:

ulimit -t measures CPU time, no effect if process isn't consuming CPU cycles
Simplest/most portable way to get a walltime timeout is to use alarm()
Use perl to set a 60-second alarm() then execs lalpulsar_hwinject; alarm() will raise SIGALRM which (uncaught) will kill lalpulsar_hwinject

API Changes and Justification

Backwards Compatible Changes

This change does not modify any class/function/struct/type definitions in a public C header file or any Python class/function definitions
This change adds new classes/functions/structs/types to a public C header file or Python module

Backwards Incompatible Changes

This change modifies an existing class/function/struct/type definition in a public C header file or Python module
This change removes an existing class/function/struct/type from a public C header file or Python module

Review Status

cc @david-keitel