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

Node exporter not configured to use correct proc location in some circumstaces #2548

Open
GrantFleming opened this issue Nov 10, 2024 · 0 comments · May be fixed by #2549
Open

Node exporter not configured to use correct proc location in some circumstaces #2548

GrantFleming opened this issue Nov 10, 2024 · 0 comments · May be fixed by #2549
Labels

Comments

@GrantFleming
Copy link

What happened?

In the node-exporter daemonsets:

  • The host filesystem is mounted at /host/root
  • The procfs location is not set, so it uses the default of /proc
  • The procfs location should probably be set to /host/root/proc
  • The difference between /proc and /host/root/proc probably doesn't matter much in most cases, but there are some where it does
  • If your Kubernetes node is an LXC container, only /host/root/proc shows the correct stats for the node, /proc shows status for the host on which the node is running

Did you expect to see some different?

I expected node-exporter to be configured to use /host/root/proc for appropriate stats

How to reproduce it (as minimally and precisely as possible):

Create a Kuberetes cluster where at least one node is an LXC container. Install the quick-start stack. Observe that the wrong cpu/mem values are displayed for the LXC node due to these values being taken from /proc rather than /host/root/proc

GrantFleming pushed a commit to GrantFleming/kube-prometheus that referenced this issue Nov 10, 2024
this sets the procfs location in node-exporter to /host/root/proc
since the host fs is mounted at /host/root

Fixes prometheus-operator#2548
@GrantFleming GrantFleming linked a pull request Nov 10, 2024 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant