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
Automated cherry pick of #124289: e2e node: fix -v support #124415
Automated cherry pick of #124289: e2e node: fix -v support #124415
Conversation
Since 43539c8 (first released in v1.30.0-alpha.2), the test/e2e/framework manages -v and -vmodule and uses them for a logger which writes to the Ginkgo output stream. This did not work for test/e2e_node, because: - logs.AddFlags(pflag.CommandLine) registers its own -v and -vmodule flags - pflag.CommandLine.AddGoFlagSet(flag.CommandLine) skips the corresponding flags in the flag.CommandLine - pflag.Parse() initializes the settings in the "logs" package even though those are not used at runtime The solution is to not use the "logs" package.
Even if the textlogger which writes to Ginkgo is installed as the logger in klog, klog still does some verbosity checks itself (for example, klog.V().Enabled). Therefore the framework has to keep the verbosity settings in the textlogger and in klog consistent. This is done by wrapping the Set call instead of replacing it.
/assign @SergeyKanzhelev |
failures are from k8s infra /retest /approve Change is small and test-only. And will make things easier for 1.30. Suitable for cherry pick for sure. |
LGTM label has been added. Git tree hash: 591af649abc0abc6128acbd7e55b3eff23766b22
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pohly, SergeyKanzhelev The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/cc kubernetes/release-managers |
/priority important-soon |
/kind bug |
/release-note-none |
should have done all PR metadata in a single comment. Sorry for the spam |
/triage accepted |
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.
/lgtm
(Release Manager)
Cherry pick of #124289 on release-1.30.
#124289: e2e node: fix -v support
For details on the cherry pick process, see the cherry pick requests page.