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

[8.6](backport #33697) Fix error spam on AKS #33703

Merged
merged 1 commit into from
Nov 17, 2022
Merged

Commits on Nov 16, 2022

  1. Fix error spam on AKS (#33697)

    This was happening due to the error level logging when the log path
    matcher detected a `log.file.path` that does not start with a standard
    Docker container log folder `/var/lib/docker/containers` because AKS
    dropped support for Docker in September 2022 and switched to containerd.
    
    It looks like this message was not supposed to be on the error level
    in the first place since it just means that the matcher didn't
    match and it's not an error. But it was mistakenly promoted from the
    debug level in #16866 most likely
    because the message started with `Error` and looked confusing.
    
    This a partial fix to unblock our customers, but we still need to come
    up with the full AKS/containerd support in a follow up change.
    
    (cherry picked from commit 29f0b4c)
    rdner authored and mergify[bot] committed Nov 16, 2022
    Configuration menu
    Copy the full SHA
    2c9f028 View commit details
    Browse the repository at this point in the history