-
Notifications
You must be signed in to change notification settings - Fork 455
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
Upgrade klog to v2 #2469
Comments
@Electronic-Waste: Please ensure the request meets the requirements listed here. If this request no longer meets these requirements, the label can be removed In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/remove-label lifecycle/needs-triage |
I'd like to help with this issue, could you assign it to me? |
Yeah, of course! Feel free to assign it to yourself. /assign @Doris-xm |
FYI, you can check this PR for upgrading klog v2 version: kubeflow/training-operator#2332 Now, we are upgrading the Kubernetes version (#2463) so it would be better to make klog version consistent to the corresponding Kubernetes version. |
/release 0.18 |
What you would like to be added?
I would like to upgrade
klog
version to v2Why is this needed?
As we discussed in #2463 (comment), a dedicated issue for upgrading
klog
version is needed.And also, we rely on both v1 and v2 version, which is not neat enough and may need to be unified into the same version.
katib/go.mod
Line 30 in 3363964
katib/go.mod
Line 153 in 3363964
cc👀 @kubeflow/wg-automl-leads @tariq-hasan @helenxie-bit @Doris-xm
/remove-label lifecycle/needs-triage
/help-wanted
/good-first-issue
Love this feature?
Give it a 👍 We prioritize the features with most 👍
The text was updated successfully, but these errors were encountered: