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

enrichment error message #3870

Open
josedonizetti opened this issue Feb 19, 2024 · 3 comments
Open

enrichment error message #3870

josedonizetti opened this issue Feb 19, 2024 · 3 comments
Assignees
Labels
Milestone

Comments

@josedonizetti
Copy link
Collaborator

josedonizetti commented Feb 19, 2024

I keep seem messages like when running on minikube. Not sure if they are a bug because the cgroup that they are trying to enrich isn't from a container. Thought it gives the wrong message to a user, it seems like something is not working. Does it make sense to keep logging them as errors? Maybe change the severity? Anything else?

{
    "level":"error",
    "ts":1708089649.258455,
    "msg":"error enriching container in control plane",
    "error":"containers.(*Containers).EnrichCgroupInfo: cgroup 74730: no containerId (path /user.slice/user-1000.slice/user@1000.service/app.slice/snap.kubectl.kubectl-186be11b-11d3-462e-821c-83ce6d742804.scope)",
    "cgroup_id":74730
}
@josedonizetti
Copy link
Collaborator Author

@NDStrahilevitz WDYT?

@NDStrahilevitz
Copy link
Collaborator

Seems to me there's a bug here, as this pattern should not match to a container but to a pod.
So the log did its work in identifying the bug. I'll add this to the milestone and likely self assign later.

@NDStrahilevitz NDStrahilevitz added this to the v0.21.0 milestone Feb 19, 2024
@josedonizetti
Copy link
Collaborator Author

josedonizetti commented Feb 19, 2024

@NDStrahilevitz AH, nice, so it is indeed a bug! Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants