-
Notifications
You must be signed in to change notification settings - Fork 117
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
Adapt k8s to v1.19.3. #1760
base: main
Are you sure you want to change the base?
Adapt k8s to v1.19.3. #1760
Conversation
d14884e
to
84b8051
Compare
Don't do that. The package |
Ok, I will try to fix the version compatibility between |
a8126d1
to
ccc4c99
Compare
Sorry for the previous confusing description and please check the updated one. Actually, the k8s@v1.19 depends on a low version of |
The vineyard scheduler is closely related to the k8s version. As the scheduler configurations of lower versions will be deprecated in higher versions of k8s, we can't support multiple versions (v1beta1, v1beta2, v1beta3, and v1) of scheduler configuration in the same codebase. A possible way is to find the lowest version of scheduler configuration and make sure it can work in the latest k8s cluster. |
/cc @sighingnow, this issus/pr has had no activity for a long time, please help to review the status and assign people to work on it. |
ccc4c99
to
0d95d30
Compare
Signed-off-by: Tao He <linzhu.ht@alibaba-inc.com>
Signed-off-by: Ye Cao <caoye.cao@alibaba-inc.com>
0d95d30
to
e1247b0
Compare
Signed-off-by: Ye Cao <caoye.cao@alibaba-inc.com>
What do these changes do?
Replace the log package ingo/vineyard
withk8s/pkg/log
.k8s/pkg
to replace the log implementation undergo/vineyard
.Related issue number
Fixes #1651