8000 Cleanup logs · Issue #8347 · kiali/kiali · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Cleanup logs #8347

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.

Sign up for GitHub

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

Jump to bottom
Open
nrfox opened this issue Apr 17, 2025 · 2 comments
Open

Cleanup logs #8347

nrfox opened this issue Apr 17, 2025 · 2 comments
Labels
backlog Triaged Issue added to backlog enhancement This is the preferred way to describe new end-to-end features.

Comments

@nrfox
Copy link
Contributor
nrfox commented Apr 17, 2025

What do you want to improve?

The trace level logs are extremely noisy today. Messages when Kiali accesses the cache like [Kiali Cache]... are not helpful and should be pruned. Along with the "abnormal workload type" messages.

What is the current behavior?

What is the new behavior?
@nrfox nrfox added the enhancement This is the preferred way to describe new end-to-end features. label Apr 17, 2025
@jshaughn jshaughn added the backlog Triaged Issue added to backlog label Apr 28, 2025
@jmazzitelli
Copy link
Collaborator

Messages when Kiali accesses the cache like [Kiali Cache]... are not helpful and should be pruned.

After the latest structured logging additions, I took at look at all the kiali cache messages, and I see just a bunch of these - Do you think we can prune these? I'm not sure how helpful they are.

...
2025-04-30T13:03:59Z TRC Get [resource: ConfigMap] for [namespace: istio-system] [name: istio-ca-root-cert] group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: ConfigMap] for [namespace: istio-system] [name: istio-sidecar-injector] group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Service] for [namespace: istio-system] = 1 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Pod] for [namespace: istio-system] = 1 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Deployment] for [namespace: default] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: StatefulSet] for [namespace: default] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: DaemonSet] for [namespace: default] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Pod] for [namespace: default] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Pod] for [namespace: dex] = 1 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Deployment] for [namespace: dex] = 1 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: DaemonSet] for [namespace: dex] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: ReplicaSet] for [namespace: dex] = 1 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: StatefulSet] for [namespace: dex] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: StatefulSet] for [namespace: ingress-nginx] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: DaemonSet] for [namespace: ingress-nginx] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Pod] for [namespace: ingress-nginx] = 3 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Deployment] for [namespace: ingress-nginx] = 1 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: ReplicaSet] for [namespace: ingress-nginx] = 1 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: StatefulSet] for [namespace: istio-system] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Pod] for [namespace: istio-system] = 7 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Deployment] for [namespace: istio-system] = 7 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: DaemonSet] for [namespace: istio-system] = 0 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: ReplicaSet] for [namespace: istio-system] = 9 group=kialiCache
2025-04-30T13:03:59Z TRC Get [resource: Pod] for [namespace: oauth2-proxy] = 1 group=kialiCache

...

@nrfox
Copy link
Contributor Author
nrfox commented Apr 30, 2025

Do you think we can prune these? I'm not sure how helpful they are.

Yes they are just noise at this point. I'm guessing they were added when cache support was first being added to know if an object was coming from the cache or not but now that feature has been implemented for years I don't think these are ever used for debugging but they account for a vast majority of the trace logs.

jmazzitelli added a commit to jmazzitelli/kiali that referenced this issue Apr 30, 2025
@jmazzitelli
part of kiali#8347
54b9d3d
@jmazzitelli jmazzitelli mentioned this issue Apr 30, 2025
Merged
jmazzitelli added a commit that referenced this issue Apr 30, 2025
@jmazzitelli
part of #8347 (#8390)
fbbae61
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backlog Triaged Issue added to backlog enhancement This is the preferred way to describe new end-to-end features.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jmazzitelli @jshaughn @nrfox
0