Skip to content

[bugfix] Fix crash in slurm backend when cancelling pending jobs#3695

Merged
vkarak merged 1 commit into
reframe-hpc:masterfrom
ekouts:bugfix/pending_reasons_slurm_backend
Jul 8, 2026
Merged

[bugfix] Fix crash in slurm backend when cancelling pending jobs#3695
vkarak merged 1 commit into
reframe-hpc:masterfrom
ekouts:bugfix/pending_reasons_slurm_backend

Conversation

@ekouts

@ekouts ekouts commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Closes #3694 .

Signed-off-by: Eirini Koutsaniti <ekoutsaniti@gmail.com>
@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 91.90%. Comparing base (36c0527) to head (a70e796).

Files with missing lines Patch % Lines
reframe/core/schedulers/slurm.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3695      +/-   ##
==========================================
- Coverage   91.94%   91.90%   -0.04%     
==========================================
  Files          62       62              
  Lines       13782    13782              
==========================================
- Hits        12672    12667       -5     
- Misses       1110     1115       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ekouts ekouts force-pushed the bugfix/pending_reasons_slurm_backend branch from 06c944c to a70e796 Compare June 29, 2026 09:32
@ekouts ekouts requested a review from gppezzi June 30, 2026 09:12

@gppezzi gppezzi 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.

lgtm

@github-project-automation github-project-automation Bot moved this from Todo to In Progress in ReFrame Backlog Jul 1, 2026
@vkarak vkarak changed the title [bugfix] Fix regression in pending-job polling crashing the slurm backend [bugfix] Fix crash in slurm backend when cancelling pending jobs Jul 7, 2026

@vkarak vkarak 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.

Thanks for the fix!

@vkarak vkarak merged commit 40be6b7 into reframe-hpc:master Jul 8, 2026
31 of 33 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in ReFrame Backlog Jul 8, 2026
@vkarak vkarak linked an issue Jul 8, 2026 that may be closed by this pull request
@ekouts ekouts deleted the bugfix/pending_reasons_slurm_backend branch July 8, 2026 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

KeyError when cancelling blocked pending jobs

3 participants