Skip to content

Pull and store process exit status from jobs #2615

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 12, 2025
Merged

Pull and store process exit status from jobs #2615

merged 1 commit into from
May 12, 2025

Conversation

un-def
Copy link
Collaborator

@un-def un-def commented May 9, 2025

  • Pull the exit status from the job, store in JobModel
  • Display non-zero statuses in dstack apply and dstack ps -v

Closes: #2570

Screenshot_2025-05-09_13-50-31

Screenshot_2025-05-09_14-14-21

@un-def
Pull and store process exit status from jobs
f5d2aaf 
* Pull the exit status from the job, store in JobModel
* Display non-zero statuses in `dstack apply` and `dstack ps -v`

Closes: #2570
@un-def un-def requested a review from r4victor May 9, 2025 14:33
@un-def un-def merged commit 569b99b into master May 12, 2025
25 checks passed
@un-def un-def deleted the issue_2570_exit_status branch May 12, 2025 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@r4victor r4victor r4victor approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature]: Save and show the container exit code in case of an error
2 participants
@un-def @r4victor