Skip to content

Mark runpod complete on no storage backend#1665

Merged
deep1401 merged 4 commits intomainfrom
fix/mark-runpod-complete-non-storage
Mar 27, 2026
Merged

Mark runpod complete on no storage backend#1665
deep1401 merged 4 commits intomainfrom
fix/mark-runpod-complete-non-storage

Conversation

@deep1401
Copy link
Copy Markdown
Member

No description provided.

@paragon-review
Copy link
Copy Markdown

Paragon Summary

This pull request review identified 1 issue across 1 category in 1 file. The review analyzed code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools.

This PR updates remote job status handling so Runpod jobs are marked complete even when no storage backend is configured. It prevents jobs from getting stuck waiting on unavailable storage-related completion checks.

Key changes:

  • Marks Runpod jobs complete when no storage backend is configured
  • Updates remote job status handling in remote_job_status_service.py
  • Adds logic to avoid waiting on missing storage-backed artifacts
  • Improves completion flow for Runpod remote runs

Confidence score: 4/5

  • This PR has low-moderate risk with 1 medium-priority issue identified
  • Score reflects code quality concerns and maintainability issues
  • Consider addressing medium-priority findings to improve code quality

1 file reviewed, 1 comment

Severity breakdown: Medium: 1


Tip: @paragon-run <instructions> to chat with our agent or push fixes!

Dashboard

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 27, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ansformerlab/services/remote_job_status_service.py 0.00% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@deep1401 deep1401 merged commit 3a788fe into main Mar 27, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants