-
Notifications
You must be signed in to change notification settings - Fork 3
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
Check CNCF Green Reviews Cluster and Setup Requirements for Falco #2
Comments
There is not a label identifying the kind of this issue. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@incertum Thanks for writing this up and I agree this is where we need to align.
k3s comes with a local-path-provisioner that supports hostPath. So you should be able to add these mounts. If that doesn't work we can investigate alternatives.
Yes, the knodes will all have containerd (default for k3s)
I like the project label and the sub label adds flexibility. We might need more labels later but this is a great starting point IMO. For the kernel version requirements Equinix Metal has a pretty wide selection of supported OSes We're using ubuntu 22.04 but we can easily specify an alternative OS in the tofu automation. Does that provide you enough control for the kernel version?
Lastly I have a concern on how much Equinix resources we will need. Can we start with We can then add more knodes but I think we should consider provisioning knodes on demand for the duration of the test. So we consume less resources and the approach is more scalable as we onboard more projects. @nikimanoledaki @AntonioDiTuri Please also chime in with your thoughts on this. |
We would love this approach, also easier for us. [By the way I forgot to add "Kernel headers installed" as requirement for the other drivers. We will update our docs shortly. And I noticed still some minor naming hiccups it should now be consistently
Perfect works for us!
@rossf7 mind double-checking the exact path of the socket? Would appreciate it a lot, is it (1)
Great, yes I think we can very easily change or add new labels! |
@incertum That's great, thank you. The socket path is |
Thanks! I'll update the docs once we tag the next release containers and state |
What is left for this issue? :) |
Now that I have access to the In addition, do we want to mark this as complete and open a new issue once we tackle the other 2 drivers Falco has? Ok for us. |
Had a chance to inspect a few things, LGTM. We can refer to this issue in the future when we test the remaining 2 drivers. |
See https://github.com/falcosecurity/cncf-green-review-testing?tab=readme-ov-file#summary-cncf-green-reviews-cluster-requirements
Notes:
kernel.bpf_stats_enabled
by default.ebpf
andkmod
, additional host mounts are required, such as/usr/src/kernels/
and/lib/modules
. Please refer to the respective daemonset configuration for more details.containerd
to be the container runtime socket located at/run/k3s/containerd/containerd.sock
.Clarify each item with the CNCF Green Reviews Working Group, especially the
nodeSelector
.CC @nikimanoledaki
The text was updated successfully, but these errors were encountered: