-
Notifications
You must be signed in to change notification settings - Fork 402
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
Update v1alpha1 RayCluster CRD and controllers #22
Conversation
Bump the dependency versions - Golang 1.15 - Kubernetes 1.19.14 - controller-runtime 0.7.2 - controller-tools 0.5.0 Signed-off-by: Jiaxin Shan <[email protected]>
- Rename `WorkerGroupsSpec` to `WorkerGroupSpecs` - Remove HeadService and add ServiceType and EnableIngressin headGroupSpec - Add EnableInTreeAutoscaling field - Add rich information in RayClusterStatus Signed-off-by: Jiaxin Shan <[email protected]>
1. Refactor some internal methods to make invoking chains clear 2. Clean up some stale codes Signed-off-by: Jiaxin Shan <[email protected]>
Signed-off-by: Jiaxin Shan <[email protected]>
I am running and adding more tests to make sure it has good coverage. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
I know this PR is an exception, since it is the first big change. In the future we can try to keep the PRs smaller and more granular.
8ed8290
to
b3665af
Compare
This PR doesn't include ingress & in-tree autoscaler support in controllers. A separate PR will be cut later. After this change, future PRs will be small and manageable. |
* Bump k8s version to 1.19.x and golang to 1.15 Bump the dependency versions - Golang 1.15 - Kubernetes 1.19.14 - controller-runtime 0.7.2 - controller-tools 0.5.0 Signed-off-by: Jiaxin Shan <[email protected]> * Update v1alpha1 RayCluster CRD - Rename `WorkerGroupsSpec` to `WorkerGroupSpecs` - Remove HeadService and add ServiceType and EnableIngressin headGroupSpec - Add EnableInTreeAutoscaling field - Add rich information in RayClusterStatus Signed-off-by: Jiaxin Shan <[email protected]> * Update raycluster controller 1. Refactor some internal methods to make invoking chains clear 2. Clean up some stale codes Signed-off-by: Jiaxin Shan <[email protected]> * Update deploy manifests and examples Signed-off-by: Jiaxin Shan <[email protected]> * Address review comments
Address #13 #17 #21
/cc @akanso @chenk008 @chaomengyuan