-
Notifications
You must be signed in to change notification settings - Fork 98
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
Pod level metrics are not being surfaced #821
Comments
@utam0k do you have any insights on the cgroups path here? |
My insights are:
First of all, I'd like to know the cgroup path passed youki. |
I'm interested in this, can I give it a try? By the way, I thought that the cgroup paths should probably be modified as follows, but am I doing something wrong? A:
For pods that do not have runtimeClassName set, it looks like this: pod level: |
Sure, please go ahead and give it a try! Let me know what questions do you have. |
Of course, I can help you to investigate it! |
Apparently the issue is related to the cgroup driver. Currently using cgroupfs as cgroup driver.
However, the cgroup path It seems like there is a contradiction here. Let's try running K3s with
I could see pod level metrics. Next, try building with
This looks like the behavior we want. Based on this, what changes should I make? Any advice? Finally, thanks to @utam0k and my co-worker @sat0ken for their help. |
@z63d, I set $ cat /proc/25029/cgroup
0::/kubepods-besteffort-pod3f8ece92_6aea_463f_a67b_6040bda55f31.slice:cri-containerd:5f26e5e2bf00307f97229886fa04351d35a17d98d8f4db6dd6e6ea8bc14307a7 |
@kate-goldenring
Am I doing something wrong? |
@z63d thank you for reaffirming your results! The mistake was on my end. I was rerunning the test without building. I can confirm that setting systemd to true resolves the issue. |
@jprendes @Mossaka @jsturtevant since systemd is the recommended default for most clusters, can we change that default in |
I think the thing to be careful about here is that runwasi's
test/k3s/bootstrap.sh :
Start K3s
Runs on systemd
test/k3s/bootstrap.sh :
Start K3s
Runs on cgroupfs
But as @kate-goldenring says, it's true that systemd is recommended for most K8s clusters. Are there any good workarounds? |
I think runwasi should parse the |
@Mossaka |
This is a duplicate of this issue found in the Spin shim: spinframework/containerd-shim-spin#180. I wanted to recreate it here as it is reproducable with the Wasmtime shim and may have to do with how cgroups are being created by Youki.
TLDR: Pod level metrics are not being found by kubelet (even though container level metrics exist).
Repro:
Cleanup:
Based on the above, i think we need to amend the container cgroup paths from A to B:
A: /sys/fs/cgroup//kubepods-besteffort-podb7c2cbdb_9ab5_47ef_977e_97e39a73da6d.slice:cri-containerd:66dda1e97c4fc48204b066f07fc1e28bab94a571820484e01576982d4cb9d32b
B: /sys/fs/cgroup/kubepods.slice/kubepods-besteffort.slice/kubepods-besteffort-podb7c2cbdb_9ab5_47ef_977e_97e39a73da6d.slice
I am not sure how to communicate this to Youki or modify this in Youki
The text was updated successfully, but these errors were encountered: