[Docs] add docs for prerunning deadline for Kuberay#61552
[Docs] add docs for prerunning deadline for Kuberay#61552edoakes merged 2 commits intoray-project:masterfrom
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds documentation for the new preRunningDeadlineSeconds field in KubeRay. The documentation is added to the RayJob quickstart guide and the kubectl plugin user guide. The changes are clear and accurately describe the new feature. I have one minor suggestion to improve consistency in the documentation style.
Note: Security Review has been skipped due to the limited scope of the PR.
doc/source/cluster/kubernetes/getting-started/rayjob-quick-start.md
Outdated
Show resolved
Hide resolved
Signed-off-by: machichima <nary12321@gmail.com>
db15b8a to
83a82ea
Compare
Future-Outlier
left a comment
There was a problem hiding this comment.
LGTM, cc @edoakes to merge.
| * `K8sJobMode`: The KubeRay operator creates a submitter Kubernetes Job to submit the Ray job. | ||
| * `HTTPMode`: The KubeRay operator sends a request to the RayCluster to create a Ray job. | ||
| * `InteractiveMode`: The KubeRay operator waits for the user to submit a job to the RayCluster. This mode is currently in alpha and the [KubeRay kubectl plugin](kubectl-plugin) relies on it. | ||
| * `InteractiveMode`: The KubeRay operator waits for the user to submit a job to the RayCluster. This mode is currently in alpha and the [KubeRay kubectl plugin](kubectl-plugin) relies on it. To avoid RayJobs staying in `Initializing` or `Waiting` for too long, set `preRunningDeadlineSeconds`. |
There was a problem hiding this comment.
| * `InteractiveMode`: The KubeRay operator waits for the user to submit a job to the RayCluster. This mode is currently in alpha and the [KubeRay kubectl plugin](kubectl-plugin) relies on it. To avoid RayJobs staying in `Initializing` or `Waiting` for too long, set `preRunningDeadlineSeconds`. | |
| * `InteractiveMode`: The KubeRay operator waits for the user to submit a job to the RayCluster. This mode is currently in alpha and the [KubeRay kubectl plugin](kubectl-plugin) relies on it. |
I think we should remove this sentence. The other submissionMode (K8sJobMode, HTTPMode, SidecarMode) only describe the mode itself and don't cross-reference unrelated fields. Also, preRunningDeadlineSeconds applies to all modes, not just InteractiveMode, so calling it out here is misleading. Its own definition in the Automatic resource cleanup section is sufficient.
| * `submitterConfig` (Optional): Additional configurations for the submitter Kubernetes Job. | ||
| * `backoffLimit` (Optional, added in version 1.2.0): The number of retries before marking the submitter Job as failed. The default value is 2. | ||
| * Automatic resource cleanup | ||
| * `preRunningDeadlineSeconds` (Optional): Maximum number of seconds a RayJob can remain before reaching `Running`. If the RayJob does not transition to `Running` within this deadline (for example, while in `Initializing` or `Waiting`), the KubeRay operator transitions the `JobDeploymentStatus` to `Failed` with reason `PreRunningDeadlineExceeded`. The default value is 0 (no pre-running deadline is enforced). |
There was a problem hiding this comment.
| * `preRunningDeadlineSeconds` (Optional): Maximum number of seconds a RayJob can remain before reaching `Running`. If the RayJob does not transition to `Running` within this deadline (for example, while in `Initializing` or `Waiting`), the KubeRay operator transitions the `JobDeploymentStatus` to `Failed` with reason `PreRunningDeadlineExceeded`. The default value is 0 (no pre-running deadline is enforced). | |
| * `preRunningDeadlineSeconds` (Optional): If the RayJob doesn't transition the `JobDeploymentStatus` to Running within `preRunningDeadlineSeconds` seconds, the KubeRay operator transitions the `JobDeploymentStatus` to Failed with reason `PreRunningDeadlineExceeded`. |
I think the description can follow the same pattern as activeDeadlineSeconds for consistency. WDYT?
Signed-off-by: machichima <nary12321@gmail.com>
## Description We introduce new config field `preRunningDeadlineSeconds` in ray-project/kuberay#4525. This PR add the related docs. ## Related issues Related to ray-project/kuberay#4525 ## Additional information docs link: - https://anyscale-ray--61552.com.readthedocs.build/en/61552/cluster/kubernetes/getting-started/rayjob-quick-start.html#rayjob-configuration - https://anyscale-ray--61552.com.readthedocs.build/en/61552/cluster/kubernetes/user-guides/kubectl-plugin.html#submit-a-ray-job-without-a-yaml-file --------- Signed-off-by: machichima <nary12321@gmail.com> Signed-off-by: Parag Ekbote <thecoolekbote189@gmail.com>
## Description We introduce new config field `preRunningDeadlineSeconds` in ray-project/kuberay#4525. This PR add the related docs. ## Related issues Related to ray-project/kuberay#4525 ## Additional information docs link: - https://anyscale-ray--61552.com.readthedocs.build/en/61552/cluster/kubernetes/getting-started/rayjob-quick-start.html#rayjob-configuration - https://anyscale-ray--61552.com.readthedocs.build/en/61552/cluster/kubernetes/user-guides/kubectl-plugin.html#submit-a-ray-job-without-a-yaml-file --------- Signed-off-by: machichima <nary12321@gmail.com>
## Description We introduce new config field `preRunningDeadlineSeconds` in ray-project/kuberay#4525. This PR add the related docs. ## Related issues Related to ray-project/kuberay#4525 ## Additional information docs link: - https://anyscale-ray--61552.com.readthedocs.build/en/61552/cluster/kubernetes/getting-started/rayjob-quick-start.html#rayjob-configuration - https://anyscale-ray--61552.com.readthedocs.build/en/61552/cluster/kubernetes/user-guides/kubectl-plugin.html#submit-a-ray-job-without-a-yaml-file --------- Signed-off-by: machichima <nary12321@gmail.com>
Description
We introduce new config field
preRunningDeadlineSecondsin ray-project/kuberay#4525. This PR add the related docs.Related issues
Related to ray-project/kuberay#4525
Additional information
docs link: