Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v8r0] Allow jobs to be killed in more states #7794

Merged
merged 2 commits into from
Sep 17, 2024
Merged

Conversation

chrisburr
Copy link
Member

@chrisburr chrisburr commented Sep 17, 2024

We should expect jobs to be able to be killed from most states as a way of "cancelling" them.

BEGINRELEASENOTES

*WorkloadManagement
FIX: Allow jobs to be KILLED from more states
FIX: KILLED is a final job state

ENDRELEASENOTES

@chrisburr chrisburr changed the title fix: KILLED is a final job state [v8r0] KILLED is a final job state Sep 17, 2024
@chrisburr chrisburr changed the title [v8r0] KILLED is a final job state [v8r0] Allow jobs to be killed in more states Sep 17, 2024
@chrisburr chrisburr changed the base branch from integration to rel-v8r0 September 17, 2024 08:59
@chrisburr chrisburr marked this pull request as ready for review September 17, 2024 09:03
@fstagni fstagni merged commit 19202d8 into rel-v8r0 Sep 17, 2024
26 checks passed
@fstagni fstagni deleted the cburr/killed-final branch September 17, 2024 09:36
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Sep 17, 2024
@chrisburr chrisburr added alsoTargeting:rel-v8r0 Cherry pick this PR to rel-v8r0 after merge alsoTargeting:integration Cherry pick this PR to integration after merge and removed sweep:ignore Prevent sweeping from being ran for this PR alsoTargeting:rel-v8r0 Cherry pick this PR to rel-v8r0 after merge labels Sep 18, 2024
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Sep 18, 2024
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Sep 18, 2024
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/10902080612

Successful:

  • integration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants