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

[8.0] JobStatus: allow WAITING -> KILLED #7276

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Nov 6, 2023

closes #7264

BEGINRELEASENOTES

*WMS
FIX: JobStatus: allow WAITING -> KILLED

ENDRELEASENOTES

@fstagni fstagni requested a review from atsareg as a code owner November 6, 2023 16:33
@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Nov 6, 2023
aldbr
aldbr previously approved these changes Nov 7, 2023
@fstagni fstagni merged commit 157cd6c into DIRACGrid:rel-v8r0 Nov 7, 2023
20 checks passed
@DIRACGridBot DIRACGridBot added sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention labels Nov 7, 2023
@DIRACGridBot
Copy link

Sweep summary

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

Failed:

  • integration
    cherry-pick 157cd6c into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-157cd6c32-integration
    git cherry-pick -x -m 1 157cd6c32
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #7276 JobStatus: allow WAITING -> KILLED' --author='fstagni <[email protected]>'
    git push -u origin cherry-pick-2-157cd6c32-integration
    
    # If you have the GitHub CLI installed the PR can be made with
    gh pr create \
         --label 'sweep:from rel-v8r0' \
         --base integration \
         --repo DIRACGrid/DIRAC \
         --title '[sweep:integration] JobStatus: allow WAITING -> KILLED' \
         --body 'Sweep #7276 `JobStatus: allow WAITING -> KILLED` to `integration`.
    
    Adding original author @fstagni as watcher.
    
    BEGINRELEASENOTES
    
    *WMS
    FIX: JobStatus: allow WAITING -> KILLED
    
    ENDRELEASENOTES
    Closes #7281'

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 sweep:failed Sweeping failed and needs manual intervention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

killJob returns a dict with OK even if it fails
3 participants