-
Notifications
You must be signed in to change notification settings - Fork 650
Closed
Labels
Description
Search before asking
- I had searched in the issues and found no similar feature requirement.
Description
This is one of the tracking items of the REP: RayCluster status improvement, the step 5 in the design doc: https://docs.google.com/document/d/1bRL0cZa87eCX6SI7gqthN68CgmHaB6l3-vJuIse-BrY
We should implement the new HeadReady condition by referring to the current utils.IsRunningAndReady
kuberay/ray-operator/controllers/ray/rayservice_controller.go
Lines 1211 to 1217 in a43217b
| func (r *RayServiceReconciler) isHeadPodRunningAndReady(ctx context.Context, instance *rayv1.RayCluster) (bool, error) { | |
| headPod, err := r.getHeadPod(ctx, instance) | |
| if err != nil { | |
| return false, err | |
| } | |
| return utils.IsRunningAndReady(headPod), nil | |
| } |
Once the condition is implemented, we should base on it to refactor the isHeadPodRunningAndReady function.
Use case
No response
Related issues
Are you willing to submit a PR?
- Yes I am willing to submit a PR!