-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
cAdvisor logs errors when a container it monitors is removed #1772
Comments
Still seeing this behaviour with version 0.28.3 (1e567c2) while running the binary natively on an EC2 instance running ubuntu 16.04 and using aufs storage driver . |
The reason for this is that cAdvisor attempts to collect metrics on all filesystems on the host when it starts up. We should lower the verbosity of the log message or ignore devicemapper filesystems. |
I'm still getting my logs filled with this message every second. Any fix possible while the image is not updated? |
#1847 any plans on a new release soon? |
+++ |
@dashpole while it doesn't look like #1847 is necessarily a fix for this, is it likely there might be a change to lower the log verbosity of said error in the instance of devicemapper/zfs filesystems? At this point, even if we run cadvisor with --v=0 option, the error still appears. You can imagine if containers are being removed frequently, without cadvisor being restarted it can result in hundreds of lines of error logs per second. |
FYI, having this problem with the btrfs driver as well. |
In addition to lowering the log verbosity, I suggest adding an option like prometheus-node-exporter's |
Lower log verbosity for statfs failures, fixes #1772
Hi,
cAdvisor version -
v0.25.0-17543be
On AWS AMI -
amzn-ami-2017.03.f-amazon-ecs-optimized (ami-9eb4b1e5)
Using the
devicemapper
storage driver.We're seeing our cAdvisor logs full of:
Which appears to be coming from here - https://github.com/google/cadvisor/blob/v0.25.0/fs/fs.go#L333
This starts happening after containers have been removed.
To replicate:
This will continue until the cAdvisor container is restarted
Is this the intended behaviour?
Thanks!
The text was updated successfully, but these errors were encountered: