Skip to content

Added isCompleted check in getNumSpotInterruptions#6805

Merged
pditommaso merged 1 commit intomasterfrom
6802-deadlock-on-25103-on-aws-batch
Feb 6, 2026
Merged

Added isCompleted check in getNumSpotInterruptions#6805
pditommaso merged 1 commit intomasterfrom
6802-deadlock-on-25103-on-aws-batch

Conversation

@munishchouhan
Copy link
Collaborator

@munishchouhan munishchouhan commented Feb 5, 2026

This Pr fixes the deadlock reported here #6802

Prevents the deadlock because:

  • Spot interruption counts are only meaningful after task completion
  • Returns null immediately for non-completed tasks
  • Avoids calling describeJob() and triggering the batching mechanism
  • Only queries AWS Batch API when the task is completed and we actually need the spot interruption count

Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan munishchouhan linked an issue Feb 5, 2026 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Feb 5, 2026

Deploy Preview for nextflow-docs-staging canceled.

Name Link
🔨 Latest commit 8bca9ae
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/698408eea6b8040008dd18cb

@pditommaso pditommaso merged commit 7655848 into master Feb 6, 2026
25 checks passed
@pditommaso pditommaso deleted the 6802-deadlock-on-25103-on-aws-batch branch February 6, 2026 10:46
ewels pushed a commit to ewels/nextflow that referenced this pull request Feb 6, 2026
Signed-off-by: munishchouhan <hrma017@gmail.com>
@jchorl
Copy link
Contributor

jchorl commented Feb 6, 2026

Thanks for addressing so quickly!

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.

Deadlock on 25.10.3 on AWS Batch

3 participants