| Commit message (Collapse) | Author | Age |
|---|
| |
|
|
|
|
|
| |
* Use `errors.Is()` for unwrapping errors.
* Use `%w` error verb in internal error formatting.
Signed-off-by: Ben Kochie <superq@gmail.com>
|
| |
|
| |
Signed-off-by: yeya24 <yb532204897@gmail.com>
|
| |
|
|
| |
Signed-off-by: Mark Knapp <mknapp@hudson-trading.com>
|
| |
|
|
|
| |
Change-type: patch
Connects-to: #1418
Signed-off-by: dt-rush <nickp@balena.io>
|
| |
|
|
|
|
| |
The filesystem is read-only and is often used for a virtual FS
with a configuration file for a virtual machine.
Signed-off-by: Leonid Evdokimov <leon@darkk.net.ru>
|
| |
|
|
|
|
|
|
| |
According to the golang docs, the syscall package is deprecated.
https://golang.org/pkg/syscall
This updates collectors to use the x/sys/unix package instead.
Also updates the vendored x/sys/unix module to latest.
Signed-off-by: Paul Gier <pgier@redhat.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add fallback for missing /proc/1/mounts
On some systems, `/proc/1/mounts` is hidden from non-root users due to
the `hidepid` procfs feature. Attempt to fallback to `/proc/mounts` if
`/proc/1/mounts` is not found.
Signed-off-by: Ben Kochie <superq@gmail.com>
* Add tests.
Signed-off-by: Ben Kochie <superq@gmail.com>
* Add CHANGELOG entry.
Signed-off-by: Ben Kochie <superq@gmail.com>
|
| |
|
|
|
|
| |
The pull request #1002 changed the logic used on Linux servers to determine if a filesystem is
read-only. As a result of this change, the variable `readOnly` is now unused and can be removed.
Signed-off-by: Jerome Froelich <jeromefroelich@hotmail.com>
|
| |
|
| |
Signed-off-by: Matt Layher <mdlayher@gmail.com>
|
| |
|
|
|
|
|
|
|
| |
Add more virtual filesystems to the default ignore list
* bpf
* cgroup2
* selinuxfs
* squashfs
Signed-off-by: Ben Kochie <superq@gmail.com>
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* strip rootfs prefix for run in docker
* Use `/` as default value of path.rootfs, and parse mounts from `/proc/1/mounts`.
* No need to mount `/proc` and `/sys` because we share host's PID
namespace, which allows processes within the container to see all of the
processes on the system.
Closes: #66
Signed-off-by: Ivan Mikheykin <ivan.mikheykin@flant.com>
Signed-off-by: Yecheng Fu <cofyc.jackson@gmail.com>
|
| |
|
|
|
|
|
|
|
|
|
| |
When starting Docker containers a whole bunch of netns (network
namespace) mounts are created that the node exporter can't make any
sense of (and can't read either).
This ignores all nsfs filesystems.
Fixes #875
Signed-off-by: Daniele Sluijters <daenney@users.noreply.github.com>
|
| |
|
|
|
|
| |
It is quite common to put /var/lib/docker itself on a separate partition
and that should be monitored as well.
Signed-off-by: Johannes Wienke <languitar@semipol.de>
|
| |
|
|
|
|
|
|
|
|
| |
While the statfs(2) approach is reliable for normally mounted filesystems, the
flags returned can be inconsistent when filesystem has been remounted read-only
after encountering an error. The returned flags do accurately represent the
internal state of the filesystem, but they do not reflect whether the VFS layer
will accept writes. Instead, it makes sense to parse the current VFS mount
state from the options field in /proc/mounts since it takes precedence.
Signed-off-by: Brandon Gilmore <bgilmore@valvesoftware.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
it will stop being queried until it returns. (#997)
Fixed spelling mistakes.
Update transport_generic.go
Changed to a mutex approach instead of channels and added a timeout before declaring a mount stuck.
Removed unnecessary lock channel and clarified some var names.
Fixed style nits.
Signed-off-by: Mark Knapp <mknapp@hudson-trading.com>
|
| |
|
| |
Signed-off-by: Brad Beam <brad.beam@b-rad.info>
|
| |
|
|
|
|
|
|
| |
* Add overlay to defIgnoredFSTypes
To avoid statfs() errors if node_exporter is running as non privileged user.
* Updated defIngoredFSTypes values in sorted order
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
The node exporter runs unprivileged, so it cannot statfs any filesystems
under this directory causing log spam. In addition there tends to be
high churn in the filesystems here (as it's basically application
monitoring) which can cause high cardinaltiy and in one case caused
Prometheus's index symbol table to get very large.
Accordingly this should be ignored to reduce log spam and avoid
performance issues. The filesystems themselves can in principle be
monitored via container oriented exporters, and the underlying
filesystems will still be monitored.
|
| |
|
| |
Add additional Linux virtual filesystem types to the default list.
|
| |
|
|
|
|
| |
* [FIX] Issue 63, error on excluding proc filesystem on linux, improving regexp
* [FIX] Reordering filter order
|
| |
|
|
|
|
| |
Instead of maintaining a counter metric for device errors in memory,
this change exports a gauge and uses const metrics to avoid leaking
metrics for unmounted filesystems.
|
| |
|
|
|
|
| |
Named return variables should only be used to describe the returned type
further, e.g. `err error` doesn't add any new information and is just
stutter.
|
| | |
|
| | |
|
| |\
| |
| | |
Add node_filesystem_device_errors_total metric
|
| | |
| |
| |
| |
| | |
This metric is the total number of errors occurred when getting stats
for the given device.
|
| |/
|
|
|
|
|
|
| |
node_exporter currently triggers autofs to mount the underlying
filesystem on every scrape. This is undesirable. Better ignore autofs.
The underlying filesystem that autofs mounts will be monitored though,
when the (real) filesystem is mounted.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
This is a boolean metric which is set to 1 when the filesystem is flagged as
read-only.
|
| | |
|
| | |
|
| |
|
|
|
|
| |
Current behaviour throws away all stats on any Statfs error. In practice
this is not useful. This turns such errors into debug log messages -
though silently ignoring them seems even more valid to me.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Remove all hardcoded references to `/proc`. For all collectors that do
not use `github.com/prometheus/procfs` yet, provide a wrapper to
generate the full paths.
Reformulate help strings, errors and comments to remove absolute
references to `/proc`.
This is a breaking change: the `-collector.ipvs.procfs` flag is removed
in favor of the general flag. Since it only affected that collector it
was only useful for development, so this should not cause many issues.
|
| | |
|
| | |
|
| |
|
