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

K8SSAND-1361 ⁃ add /etc/mtab to /proc/mounts symlink to account for the containerd runtime collectd disk monitor #181

Closed
andrey-dubnik opened this issue Mar 29, 2022 · 3 comments · Fixed by #182

Comments

@andrey-dubnik
Copy link
Contributor

andrey-dubnik commented Mar 29, 2022

Hi,

There is a change when it comes to using the containerd compare to moby runtime as containerd does not create the default symlinks

collectd:df plugin assumes that /etc/mtab is populated accordingly to the collectd logs inside the k8ssadra managed deployment which breaks the disk stats exporter as no mounts are detected

[2022-03-24 01:05:59] setmntent (/etc/mtab): No such file or directory
[2022-03-24 01:05:59] df plugin: cu_mount_getlist failed.

Does this makes sense to update the Dockerfile and add a symlink /etc/mtab -> /proc/mounts so C* disks can be monitored?

┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1361
┆priority: Medium

@sync-by-unito sync-by-unito bot changed the title add /etc/mtab to /proc/mounts symlink to account for the containerd runtime collectd disk monitor K8SSAND-1361 ⁃ add /etc/mtab to /proc/mounts symlink to account for the containerd runtime collectd disk monitor Mar 29, 2022
@andrey-dubnik
Copy link
Contributor Author

I can do a PR if this helps

@emerkle826
Copy link
Contributor

@andrey-dubnik A PR would be welcome. A few notes:

  1. Please sign the CLA if you want to contribute code changes (CLA is here https://cla.datastax.com/)
  2. If your changes are to Dockerfiles, we are currently maintaining 3 of them in this project:

@andrey-dubnik
Copy link
Contributor Author

@emerkle826 PR and CLI are done

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

Successfully merging a pull request may close this issue.

2 participants