Skip to content

Conversation

@teojgo
Copy link
Contributor

@teojgo teojgo commented Feb 24, 2020

  • Allow ReFrame to exit with grace when it receives a SIGTERM.

  • Add unittest to simulate a SIGTERM handle.

Fixes #1143

* Allow ReFrame to exit with grace when it receives a SIGTERM.

* Add unittest to simulate a SIGTERM handle.
@teojgo teojgo requested a review from vkarak February 24, 2020 16:16
@teojgo teojgo self-assigned this Feb 24, 2020
@teojgo teojgo added this to the ReFrame sprint 20.02 milestone Feb 24, 2020
Theofilos Manitaras added 2 commits February 25, 2020 11:27
* Adjust unittests to simulate sending of a SIGTERM signal
  to ReFrame.
@codecov-io
Copy link

codecov-io commented Feb 25, 2020

Codecov Report

Merging #1193 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1193   +/-   ##
=======================================
  Coverage   92.12%   92.12%           
=======================================
  Files          81       81           
  Lines       12028    12028           
=======================================
  Hits        11081    11081           
  Misses        947      947
Impacted Files Coverage Δ
reframe/__init__.py 80% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d69c7c1...a5bc32b. Read the comment docs.

Copy link
Contributor

@vkarak vkarak left a comment

Choose a reason for hiding this comment

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

Almost there. I have just a couple of minor comments.

@vkarak vkarak changed the title [bugfix] Handle SIGTERM in Runner class [bugfix] Do not leave stale test jobs when ReFrame receives a TERM signal Feb 25, 2020
@vkarak
Copy link
Contributor

vkarak commented Feb 25, 2020

@teojgo Strangely CI has finished successfully, but here it appears as still running.

Copy link
Contributor

@vkarak vkarak left a comment

Choose a reason for hiding this comment

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

lgtm

@vkarak
Copy link
Contributor

vkarak commented Feb 26, 2020

@teojgo The test fails on Kesch. Moving over to the next sprint.

@vkarak
Copy link
Contributor

vkarak commented Feb 26, 2020

@jenkins-cscs retry kesch

@vkarak vkarak merged commit 0e010db into reframe-hpc:master Mar 1, 2020
@teojgo teojgo deleted the bugfix/handle_sigterm branch March 4, 2020 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ReFrame does not cancel the spawned test jobs if it receives a TERM signal

3 participants