fix(deps): update module k8s.io/klog to v2 - abandoned#184
fix(deps): update module k8s.io/klog to v2 - abandoned#184renovate-bot wants to merge 2 commits intoGoogleCloudPlatform:mainfrom
Conversation
Summary of ChangesHello @renovate-bot, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This PR attempts to upgrade k8s.io/klog to v2, but it introduces a critical issue. The upgrade is a major version change that requires updating import paths in the Go source code, which is missing from this PR. This will cause the build to fail. Additionally, the change creates a duplicate dependency in go.mod. I've recommended reverting the change until the necessary source code updates can be included.
| k8s.io/klog/v2 v2.130.1 | ||
| k8s.io/klog/v2 v2.130.1 |
There was a problem hiding this comment.
This dependency update has two issues:
- Breaking Change: Upgrading
k8s.io/klogfrom v1 to v2 is a major version change that requires updating import paths in the source code (fromk8s.io/klogtok8s.io/klog/v2). Sincemain.gostill importsk8s.io/klog, this change will break the build. - Duplicate Dependency: This change also introduces a duplicate requirement for
k8s.io/klog/v2.
This PR is incomplete and will break the build. It should be reverted. To fix this properly, the import paths in the Go source files need to be updated to k8s.io/klog/v2 and then go mod tidy should be run to clean up the dependencies.
| k8s.io/klog/v2 v2.130.1 | |
| k8s.io/klog/v2 v2.130.1 | |
| k8s.io/klog v1.0.0 | |
| k8s.io/klog/v2 v2.130.1 |
c4d8201 to
071be94
Compare
071be94 to
0e8b291
Compare
0e8b291 to
2be2e60
Compare
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
Autoclosing SkippedThis PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error. |
This PR contains the following updates:
v1.0.0→v2.130.1Release Notes
kubernetes/klog (k8s.io/klog)
v2.130.1: Prepare klog release for Kubernetes v1.31 (Take 2)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.130.0...v2.130.1
v2.130.0: Prepare klog release for Kubernetes v1.31 (Take 1)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.120.1...v2.130.0
v2.120.1: Prepare klog release for Kubernetes v1.30 (Take 2)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.120.0...v2.120.1
v2.120.0: Prepare klog release for Kubernetes v1.30 (Take 1)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.110.1...v2.120.0
v2.110.1: Prepare klog release for Kubernetes v1.29 (Take 1)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.100.1...v2.110.1
v2.100.1: Prepare klog release for Kubernetes v1.28 (Take 1)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.90.1...v2.100.1
v2.90.1: Prepare klog release for Kubernetes v1.27 (Take 2)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.90.0...v2.90.1
v2.90.0: Prepare klog release for Kubernetes v1.27 (Take 1)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.80.1...v2.90.0
There are some API differences from previous version
v2.80.1: Prepare klog release for Kubernetes v1.26 (Take 2)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.80.0...v2.80.1
v2.80.0: Prepare klog release for Kubernetes v1.26 (Take 1)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.70.1...v2.80.0
v2.70.1: Prepare klog release for Kubernetes v1.25 (Take 2)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.70.0...v2.70.1
v2.70.0: Prepare klog release for Kubernetes v1.25 (Take 1)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.60.1...v2.70.0
v2.60.1: Prepare klog release for Kubernetes v1.24 (Take 6)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.60.0...v2.60.1
v2.60.0: Prepare klog release for Kubernetes v1.24 (Take 5)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.50.2...v2.60.0
v2.50.2Compare Source
v2.50.1: Prepare klog release for Kubernetes v1.24 (Take 4)Compare Source
What's Changed
Full Changelog: kubernetes/klog@v2.50.0...v2.50.1
v2.50.0: Prepare klog release for Kubernetes v1.24 (Take 3)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.40.1...v2.50.0
v2.40.1: Prepare klog release for Kubernetes v1.24 (Take 2)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.40.0...v2.40.1
v2.40.0: Prepare klog release for Kubernetes v1.24Compare Source
What's Changed
Known Issues
New Contributors
Full Changelog: kubernetes/klog@v2.30.0...v2.40.0
v2.30.0: Prepare klog release for Kubernetes v1.23 (take 2)Compare Source
What's Changed
New Contributors
Full Changelog: kubernetes/klog@v2.20.0...v2.30.0
v2.20.0: Prepare klog release for Kubernetes v1.23Compare Source
Changes are here : kubernetes/klog@v2.10.0...v2.20.0
since we moved to logr v1.0.0, there are incompatible changes:
83653a6Update to newest versions of golang 1.17.xd648c2efix file-based filtering symbolization8ee3d65export ClearLogger4171f3cSwitching to logr tag v1.0.09ab3c2badd serathius as approvers of klogv2.10.0: One more change to support 1.22 releaseCompare Source
Changes are here : kubernetes/klog@v2.9.0...v2.10.0
new function added:
v2.9.0: Prepare release for Kubernetes v1.22Compare Source
Changes are here : kubernetes/klog@v2.8.0...v2.9.0
6a9ef3ffix typo59f7cb5fix byte array display in InfoS and ErrorScf22f1eCall logr with call depthe95c7e3make SetLogger thread-safe2728fe1check usage of format specifier in structured log funca18bc97Fix by pr suggestions4e4135cAdd check for InfoS & ErrorS parametersv2.8.0: Bug fixes for structured logging for Kubernetes v1.21Compare Source
v2.7.0: Miscellaneous fixes for structured logging for Kubernetes v1.21Compare Source
Changes are here : kubernetes/klog@v2.6.0...v2.7.0
v2.6.0: Adding a linter for Kubernetes v1.21Compare Source
Changes are here : kubernetes/klog@v2.5.0...v2.6.0
please see https://github.com/kubernetes/klog/tree/master/hack/tools/logcheck
v2.5.0: Prepare release for Kubernetes v1.21Compare Source
Changes are here : kubernetes/klog@v2.4.0...v2.5.0
klog.go has new API:
klogr/klogr.go has new API:
v2.4.0: Prepare release for Kubernetes v1.20Compare Source
Changes are here : kubernetes/klog@v2.3.0...v2.4.0
v2.3.0: Fix Typo-ed Method Error -> ErrorSCompare Source
Changes are here : kubernetes/klog@v2.2.0...v2.3.0
v2.2.0: Dependency update and bugfix for InfoSCompare Source
2e691ebFix missing fields in verbose InfoS966c986feat use go-logr v0.2.0Changes are here : kubernetes/klog@v2.1.0...v2.2.0
v2.1.0: Better support for Structured LoggingCompare Source
We are now enforcing API compatibility, added Windows based tests, and have tweaked the structured logging methods after some real world experience updating kubernetes main repo.
bbd9ca1Add tests for error in InfoS1ccc0e1fix imported bug time encode format form kvlistFormatdd4d1a6fix typo in README.md49123d4ErrorS(nil, ...) should call loggr.Error(nil, ...)5b199cdFix documentation for V(level)d1eb30fAdd apidiff script to check go signature changesdc505bfSwitch slack channel to #kloga47ebb9Add example for co-existence of klog v1 and v2134f148logName(): lazily lookup userName instead of on init()db06a1bfix serialization of special html chars5727d2aFix Windows integration testsedbc1d3test(*): TestRollover failed randomly on Windows6f99060Add LogToStderr, a programatic way to log exclusively to stderr or notv2.0.0: Release to support Kubernetes v1.19Compare Source
Beware of type change:
VerboseNew Methods:
SetLogger(override logger to set a custom implementation)InfoS(structured logging)ErrorS(structured logging)Changes are here : kubernetes/klog@v2.0.0-rc.1...v2.0.0
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.