Skip to content

docs: update default ExitMode to DELETE_AFTER_JOBS#170

Merged
jmhsieh merged 4 commits intomainfrom
jon/delete_after_jobs
Feb 27, 2026
Merged

docs: update default ExitMode to DELETE_AFTER_JOBS#170
jmhsieh merged 4 commits intomainfrom
jon/delete_after_jobs

Conversation

@jmhsieh
Copy link
Contributor

@jmhsieh jmhsieh commented Feb 27, 2026

Summary

  • Updates docs to reflect that the default on_exit for KubeRay clusters changed from ExitMode.DELETE to ExitMode.DELETE_AFTER_JOBS
  • Documents the new wait_timeout parameter for capping how long the cluster waits
  • Moves Exit Modes section under KubeRay since it only applies to KubeRay clusters

Test plan

  • Verify docs render correctly on Mintlify

🤖 Generated with Claude Code

jmhsieh and others added 4 commits February 13, 2026 10:16
The implementation now watches for alive JobTracker actors on the
cluster rather than polling DB job status, and works across multiple
processes sharing the same cluster.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…section

- Default on_exit changed from DELETE to DELETE_AFTER_JOBS
- Document new wait_timeout parameter
- Move Exit Modes section under KubeRay since it only applies there

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jmhsieh
Copy link
Contributor Author

jmhsieh commented Feb 27, 2026

Busted link warning is a separate from this patch.

@prrao87
Copy link
Contributor

prrao87 commented Feb 27, 2026

Have opened a support ticket with mintlify to address that. For the short term, we can ignore the busted link and merge by ignoring it. If it's not resolved by next week I'll remove the offending link from the docs. Thanks!

@jmhsieh jmhsieh merged commit eebba25 into main Feb 27, 2026
1 of 2 checks passed
@jmhsieh jmhsieh deleted the jon/delete_after_jobs branch February 27, 2026 23:03
@prrao87
Copy link
Contributor

prrao87 commented Feb 27, 2026

Okay, just for future reference, Yang discovered the fix!

For redirect URLs, just add the actual path to the filename like so: /api-reference/rest/namespace/index. That pleases the link checker because that's the actual filename on disk 👍🏽

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.

3 participants