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

kubelet and btrfs snapshots: failed to check if disk space is available on the root partition: failed to get fs info for "root": error trying to get filesystem Device for dir /var/lib/kubelet #47046

Closed
flavio opened this issue Jun 6, 2017 · 4 comments
Labels
sig/storage Categorizes an issue or PR as relevant to SIG Storage.

Comments

@flavio
Copy link

flavio commented Jun 6, 2017

Kubernetes version (use kubectl version): 1.5.6 forward

Environment:

  • Cloud provider or hardware configuration: both bare metal and virtual
  • OS (e.g. from /etc/os-release): SUSE 12+ or openSUSE 42.2+
  • Kernel (e.g. uname -a): 4.4.62, but not really relevant in this case
  • Install tools: SUSE/openSUSE packages
  • Others: none

What happened:

The system is using btrfs as main file system, several btrfs subvolumes are mounted under /, with /var/lib/kubelet being one of them.

cadvisor (and hence kubelet) can't determine the disk space available on the root partition because it gets confused by the subvolume major and minor IDs.

Failed to check if disk space is available on the root partition: failed to get fs info for "root": error trying to get filesystem Device for dir /var/lib/kubelet: err: could not find device with major: 0, minor: 50 in cached partitions map

This is similar to #38337 but fix dc8b6cc is not enough for it.

I've already a fix for this issue, I'll follow up with a PR later today.

What you expected to happen:

No warnings from kubelet.

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

  • Install a system using btrfs as file system for /
  • Create a sub-volume for /var/lib/kubelet
  • Start kubelet (even in disconnected mode)

Anything else we need to know:

I've seen other similar issues, but none of them was about btrfs. The issues were about tmpfs or zfs. My fix for this issue is going to be based on dc8b6cc hence it's going to be btrfs specific. That's why I filed this new issue.

@k8s-github-robot
Copy link

@flavio There are no sig labels on this issue. Please add a sig label by:
(1) mentioning a sig: @kubernetes/sig-<team-name>-misc
(2) specifying the label manually: /sig <label>

Note: method (1) will trigger a notification to the team. You can find the team list here and label list here

@k8s-github-robot k8s-github-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Jun 6, 2017
@gnufied
Copy link
Member

gnufied commented Jun 6, 2017

/sig storage

@k8s-ci-robot k8s-ci-robot added the sig/storage Categorizes an issue or PR as relevant to SIG Storage. label Jun 6, 2017
@k8s-github-robot k8s-github-robot removed the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Jun 6, 2017
flavio added a commit to flavio/cadvisor that referenced this issue Jun 6, 2017
kubernetes/kubernetes#47046

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
flavio added a commit to flavio/cadvisor that referenced this issue Jun 7, 2017
kubernetes/kubernetes#47046

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
flavio added a commit to flavio/cadvisor that referenced this issue Jun 12, 2017
kubernetes/kubernetes#47046

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
@jingxu97
Copy link
Contributor

I see the fix for btrfs is merged. Could we close this issue?

@flavio
Copy link
Author

flavio commented Sep 6, 2017

Yes, the fix reached Kubernetes too because the vendored version of cadvisor has been upgraded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/storage Categorizes an issue or PR as relevant to SIG Storage.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants