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
Unify feature probing packages #25627
Conversation
/test |
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! We can remove moar code!
d1de9a7
to
fdd5b58
Compare
/test Job 'Cilium-PR-K8s-1.16-kernel-4.19' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.16-kernel-4.19/121/ 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. |
/test 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/180/ 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. |
Since Cilium's minimal required kernel version was bumped to 4.19 all supported kernels have full LPM map support. This allows us to remove this feature probe as it would always be true. Signed-off-by: Robin Gögge <r.goegge@isovalent.com>
With the removal of HaveFullLPM every field in the prefilter config struct will always be true. This allows the removal of all logic checks based on those fields and the config struct itself. Signed-off-by: Robin Gögge <r.goegge@isovalent.com>
This also refactors HaveIPv6Support into returning an error instead of a bool to keep feature probing APIs consistent across the package and adds a small test for the probe. Fixes: cilium#25621. Signed-off-by: Robin Gögge <r.goegge@isovalent.com>
/test Job 'Cilium-PR-K8s-1.16-kernel-4.19' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.16-kernel-4.19/172/ 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. |
/test-1.16-4.19 |
With this PR we remove the
HaveFullLPM()
feature probe as it is no longer needed since cilium bumped its minimal supported kernel version to 4.19.We also migrate any leftover APIs from
pkg/probe
topkg/datapath/linux/probes
to only have one gopkg
for feature probing APIs.Fixes: #25621