Skip to content

[Feature] REP 54: Replace rayv1.Suspended with RayClusterSuspended condition #2277

Closed
@rueian

Description

@rueian

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 ray-project/enhancements#54, the step 7 in the design doc: https://docs.google.com/document/d/1bRL0cZa87eCX6SI7gqthN68CgmHaB6l3-vJuIse-BrY

Step 7: Remove rayv1.Suspended from Status.State. Make suspending a RayCluster an atomic operation.

type RayClusterStatus struct {
       ...
	Conditions []metav1.Condition `json:"conditions,omitempty"`
}

type RayClusterConditionType string

const (
RayClusterSuspending RayClusterConditionType = "Suspending"
RayClusterSuspended RayClusterConditionType = "Suspended"
)

Introduce RayClusterSuspending and RayClusterSuspended to Status.Conditions. Then, we can refer to RayJob to make the suspend operation atomic.

This new behavior is effective only when #2219 is enabled to maintain backward compatibility.

Use case

No response

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions