-
Notifications
You must be signed in to change notification settings - Fork 8k
Closed
Labels
area/networking/cniIstio CNI-related issuesIstio CNI-related issueslifecycle/automatically-closedIndicates a PR or issue that has been closed automatically.Indicates a PR or issue that has been closed automatically.lifecycle/staleIndicates a PR or issue hasn't been manipulated by an Istio team member for a whileIndicates a PR or issue hasn't been manipulated by an Istio team member for a while
Description
CNI logging works by sending logs over a UDS socket from the istio-cni binary up to the user-space install-cni container. There are a few issues here around logging levels and scopes.
istio/cni/pkg/plugin/plugin.go
Line 177 in 764e668
log.FindScope("default").SetOutputLevel(getLogLevel(conf.LogLevel)) logLevel
in the configmap, which isValues.cni.logLevel
. This only handles default scope. if we have other scopes, we cannot configure them. Either we should apply this to all scopes, or allow the standard parsing of scopes that log_output_level does- Daemonset doesn't allow setting the level at all:
command: ["install-cni"] - Maybe not a bug, but something we should consider:
Line 121 in bf2adb0
pluginLog.Debug(m.Msg) scope
of the log. If we send scoped logs, we lose that info. Either we can lookup the original scope, or we should add a key to do the equivalent
Metadata
Metadata
Assignees
Labels
area/networking/cniIstio CNI-related issuesIstio CNI-related issueslifecycle/automatically-closedIndicates a PR or issue that has been closed automatically.Indicates a PR or issue that has been closed automatically.lifecycle/staleIndicates a PR or issue hasn't been manipulated by an Istio team member for a whileIndicates a PR or issue hasn't been manipulated by an Istio team member for a while