Skip to content

SIGTERM and wait 10 sec prior to sending SIGKILL.

Greg Smethells requested to merge gregsmethells/gitlab-runner:sigterm-3031 into master

We are going to split this merge request into multiple issues to make it easier to review and schdule and prevent it from being too large:

What does this MR do?

Send a SIGTERM and wait 10 seconds prior to sending a SIGKILL so that the process group has a chance to clean up by catching the SIGTERM signal and reacting appropriately for its circumstance.

Why was this MR needed?

To satsify #3031 (closed)

Are there points in the code the reviewer needs to double check?

Only three lines changed.

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

#3031 (closed)

Edited by Steve Xuereb

Merge request reports