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

Check for orphaned processes by job_id when starting a job #3508

Merged
merged 11 commits into from
Jan 2, 2024

Conversation

busma13
Copy link
Contributor

@busma13 busma13 commented Dec 20, 2023

This PR makes the following changes:

  • Add check for orphaned resources by job_id before starting job.

  • Add listResourcesForJobId function to execution service.

  • Add listResourcesForJobId function to k8s and native cluster backends.

  • Improve messaging to user when using force option on the POST /jobs/{job_id}/_stop endpoint

  • Remove check for native clustering when using force stop.

  • Update docs for _stop endpoint to show an example using force

Copy link
Member

@godber godber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets sort out the response and the native method thing before accepting. Otherwise this looks great.

@busma13 busma13 marked this pull request as ready for review December 21, 2023 20:39
@godber godber merged commit 2b1f2d3 into master Jan 2, 2024
39 checks passed
@godber godber deleted the check-for-orphans-on-job-start branch January 2, 2024 16:21
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