Skip to content

[9.1] fix (WMS): clear the signal mask before starting the payload - #8840

Merged
fstagni merged 1 commit into
DIRACGrid:integrationfrom
chrisburr:fix/reset-signal-mask-for-payload-integration
Oct 26, 2026
Merged

fstagni merged 1 commit into
DIRACGrid:integrationfrom
chrisburr:fix/reset-signal-mask-for-payload-integration

Conversation

@chrisburr

Copy link
Copy Markdown
Member

Port of #8838 to integration.

BEGINRELEASENOTES
*WorkloadManagement
FIX: the payload no longer inherits signals blocked in the JobWrapper (e.g. by gfal2), so it can be stopped with SIGTERM and abort itself with SIGQUIT
ENDRELEASENOTES

Creating a gfal2 context leaves most signals (SIGTERM, SIGINT, SIGQUIT,
SIGALRM, ...) blocked in the calling thread. The payload inherits the
signal mask of the thread that starts it, so it couldn't be stopped with
SIGTERM, and Gaudi's EventWatchdog could not abort stuck jobs with SIGQUIT
(gaudi/Gaudi#428).
@chrisburr
chrisburr force-pushed the fix/reset-signal-mask-for-payload-integration branch from 869da71 to 77e4a55 Compare October 20, 2026 12:43
@chrisburr
chrisburr marked this pull request as ready for review October 26, 2026 09:30

@fstagni fstagni left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI green.

@fstagni
fstagni merged commit b32edb7 into DIRACGrid:integration Oct 26, 2026
25 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Oct 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:ignore Prevent sweeping from being ran for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants