Skip to content
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

kubelet: revert contextual logging support #110869

Closed
wants to merge 1 commit into from

Commits on Jun 30, 2022

  1. kubelet: revert contextual logging support

    It turned out that using testing.T for logging had a race condition and
    potential panic because the tests keep goroutines running and testing.T is not
    supposed to be used anymore after test
    completion (kubernetes#110854).
    
    Either the code must be fixed to terminate all goroutines before a test
    ends (seems non-trivial because the usage of goroutines is fairly complex in
    the shutdown manager code), or ktesting must handle this case. A solution for
    this is pending in kubernetes/klog#337.
    
    Either way, solving this will take a bit longer. In the meantime, we should
    revert the change to get unit testing stable again.
    
    In order to make this a local change, ktesting is kept as a dependency of the
    test and thus Kubernetes.
    pohly committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    8384e72 View commit details
    Browse the repository at this point in the history