-
Notifications
You must be signed in to change notification settings - Fork 322
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
[Refactor] Move MatchingLabels to association.go #2045
Comments
4 tasks
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Following the PR #1948 and #2023,
We like to move all MatchingLabels to the
association.go
, so that we can maintain consistency in the association methods by not scattering MatchingLabels everywhere.This issue is a todo list for tracking refactor progress and collaboration with other contributors.
kuberay/ray-operator/controllers/ray/raycluster_controller.go
Line 441 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller.go
Line 476 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller.go
Line 509 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller.go
Line 1252 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller.go
Line 1305 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller.go
Line 1319 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller.go
Line 1340 in bbb65b4
kuberay/ray-operator/controllers/ray/rayservice_controller.go
Line 436 in bbb65b4
kuberay/ray-operator/controllers/ray/rayservice_controller.go
Line 1232 in bbb65b4
kuberay/ray-operator/controllers/ray/rayjob_controller_suspended_test.go
Line 168 in bbb65b4
kuberay/ray-operator/controllers/ray/rayjob_controller_suspended_test.go
Line 181 in bbb65b4
kuberay/ray-operator/controllers/ray/suite_helpers_test.go
Line 210 in bbb65b4
kuberay/ray-operator/controllers/ray/rayjob_controller_test.go
Line 174 in bbb65b4
kuberay/ray-operator/controllers/ray/rayjob_controller_test.go
Line 182 in bbb65b4
kuberay/ray-operator/controllers/ray/rayservice_controller_test.go
Line 250 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller_fake_test.go
Line 2253 in bbb65b4
kuberay/ray-operator/controllers/ray/raycluster_controller_fake_test.go
Line 2264 in bbb65b4
@evalaiyc98 is already working on the
suite_helpers_test.go
part. cc @kevin85421The text was updated successfully, but these errors were encountered: