-
Notifications
You must be signed in to change notification settings - Fork 670
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
Support kuberay v1.0.0 #4656
Support kuberay v1.0.0 #4656
Conversation
Signed-off-by: Yicheng-Lu-llll <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4656 +/- ##
==========================================
- Coverage 58.11% 58.11% -0.01%
==========================================
Files 626 626
Lines 53815 53812 -3
==========================================
- Hits 31277 31275 -2
- Misses 20036 20037 +1
+ Partials 2502 2500 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Yicheng-Lu-llll <[email protected]>
Signed-off-by: Yicheng-Lu-llll <[email protected]>
Signed-off-by: Yicheng-Lu-llll <[email protected]>
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.
Thank you for carrying this on!
Why are the changes needed?
Currently, KubeRay is undergoing significant refactoring of the RayJob, which includes lots of status changes that Flyte depends on. This update could potentially be released in KubeRay v1.2.0. Before refactoring the current Ray plugin to be based on KubeRay v1.2.0, it would be great to first support KubeRay v1.0.0 and all earlier versions. For KubeRay v1.2.0, maintaining backward compatibility will be almost impossible due to the extensive changes in RayJob.
Additionally, the KubeRay maintainers have recently decided to continue maintaining version v1alpha1 (previously deprecated) until the release of KubeRay v1.2.0.
This PR enables users to use Kuberay v1.0.0 and all earlier versions.
What changes were proposed in this pull request?
This PR introduces several new statuses while maintaining backward compatibility.
How was this patch tested?
I have tested this PR with Kuberay v0.5.2 and Kuberay v1.0.0.
Setup process
Screenshots
Check all the applicable boxes
Related PRs
Docs link