-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
container stats not working properly on zfs backend, zfs/graph/graph: no such file or directory
#5820
Comments
zfs backend is not really supported, any reason for using it? |
I'm running k8s cluster on my Ubuntu workstation which using the zfs as root file system. but that's not the point . |
There's no guaranteed relationship between the two. It would be best for CRI-O to not make any assumptions about relationships between the two. If you have the container's ID, its |
For future reference for others: CRI-O/Kubernetes works correctly with ZFS >=2.2 where overlayfs support was correctly implemented. You may need to verify your distribution's repository, as in Debian 12 you need to enable |
@morsik, thank you for the follow up! Appreciated. |
What happened?
It cause by
cri-o/internal/lib/stats/stats_server.go
Line 192 in 15afd20
When using zfs backend,
container.MountPoint()
has value like/var/lib/containers/storage/zfs/graph/40f155bca81dd456f379f2a349b707c1b14e87bc5fed94e65182fc4a640a088e
The
id
is always "graph", and produce an error above.But it is works well on overlay driver, on overlay the container mountPoint seems like
/var/lib/containers/storage/overlay/679961db0ea4e298b51261b14c5b5a30252ce5425c4abcda84180f800611acac/merged
I'll be glade to do a PR if needed.
What did you expect to happen?
Expected
id
has a value of519dff473ffe68973c81c754dcf4ef643b6d0305085f3317ffe096d2249c1976
at above case.How can we reproduce it (as minimally and precisely as possible)?
A
zfs
driver base crio runtime.Anything else we need to know?
No response
CRI-O and Kubernetes version
OS version
Additional environment details (AWS, VirtualBox, physical, etc.)
The text was updated successfully, but these errors were encountered: