-
Notifications
You must be signed in to change notification settings - Fork 111
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
Can't get container info for PID #1277
Comments
I can verify that it works now with Now, unfortunately, I got another issue.
But I gonna address this in a separate issue or at first just in the beyla slack channel, because this probably hasn't got anything to do with this fix. Thanks for your quick action! |
Thanks for reporting @jkruke we are working on a fix. |
It's fixed now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
I am using the embedded Grafana Beyla of Grafana Alloy in a minikube kubernetes cluster.
After updating minikube, kubernetes and docker, Beyla cannot detect container info anymore for containers running in the same cluster.
Configuration
An excerpt of the alloy configuration file:
Log Extracts
Alloy log (beyla.ebpf.debug=true):
Relevant Versions
Analysis
This part of Beyla code tries to get the docker container ID of a given PID by parsing the content of
/proc/<PID>/cgroup
:beyla/pkg/internal/helpers/container/container.go
Line 30 in e2e9c3f
With Docker Engine 25.0.4, Beyla container ID detection still works. Here is a cgroup file content of a container ("kube-scheduler"):
With Docker Engine 27.3.1, Beyla cannot get container info anymore (see log extracts,
msg="can't get container info for PID" component=discover.watcherKubeEnricher pid=3381 error="/proc/3381/cgroup: couldn't find any docker entry for process
).When I inspect the cgroup file of the kube-scheduler container again, it looks a bit different:
This matches to none of the both regexes in container.go.
Suggested Solution
I assume, the first regex for dockerCgroup (
^\d+:.*:.*/.*-([\da-fA-F]+)\.scope
) should be modified because the cgroup does not necessarily end with.scope
anymore.The text was updated successfully, but these errors were encountered: