-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
v1.13 backport for #26197 #26200
v1.13 backport for #26197 #26200
Conversation
[ upstream commit 3eaca5d ] Commit 8531c5a ("bpf,datapath: read jiffies from /proc/schedstat") switched package ctmap over to using /proc/schedstat for getting kernel jiffies. During the conversion, a shift by a magic 'scaler' value was missed, leading to strange output in `cilium bpf ct list global -d` (.. (remaining: -4391264792 sec(s)) ..) and the CT GC never collecting any flows, since the current jiffies were always perceived to be orders of magnitude larger than the flow expiry time. This patch wraps probes.Jiffies() in a documented scaledJiffies() helper and declares the BPF_MONO_SCALER macro as a Go constant. Signed-off-by: Timo Beckers <timo@isovalent.com> Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/test-backport-1.13 Job 'Cilium-PR-K8s-1.24-kernel-4.19' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.24-kernel-4.19/39/ If it is a flake and a GitHub issue doesn't already exist to track it, comment Then please upload the Jenkins artifacts to that issue. Job 'Cilium-PR-K8s-1.26-kernel-net-next' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.26-kernel-net-next/778/ If it is a flake and a GitHub issue doesn't already exist to track it, comment Then please upload the Jenkins artifacts to that issue. |
i'm guessing that |
all the tests passed, including |
Once this PR is merged, you can update the PR labels via: