-
Notifications
You must be signed in to change notification settings - Fork 41
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
Allow for expected kube-bench failures with bottlerocket #179
Conversation
This should be fixed properly with aquasecurity/kube-bench#808 and bottlerocket-os/bottlerocket#1295 But, as these are known failures, this should not be a failure in our CI. Workaround by patching kube-bench config for bottlerocket config paths, and allowing for a known number of failures.
35719cc
to
a00446d
Compare
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.
Note to self - check how the default config has been patched for kube-bench job
- "/etc/kubernetes/kubelet.conf" | ||
- "/var/lib/kubelet/kubeconfig" | ||
- "/etc/kubernetes/kubelet-kubeconfig" | ||
- "/etc/kubernetes/kubelet/kubeconfig" |
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.
Patched addition from default config
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.
- "/var/lib/kubelet/config.yaml" | ||
- "/var/lib/kubelet/config.yml" | ||
- "/etc/kubernetes/kubelet/kubelet-config.json" | ||
- "/etc/kubernetes/kubelet/config" |
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.
Patched addition from default config
- /var/snap/microk8s/current/args/kube-proxy | ||
kubeconfig: | ||
- "/etc/kubernetes/kubelet-kubeconfig" | ||
- "/etc/kubernetes/kubelet/config" |
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.
Patched addition from default config
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.
LGTM! I left a question out of curiosity having worked through the upstream fixes, but this looks 👌
Allow for expected kube-bench failures with bottlerocket
This should be fixed properly with aquasecurity/kube-bench#808 and bottlerocket-os/bottlerocket#1295
But, as these are known failures, this should not be a failure in our CI.
Workaround by patching kube-bench config for bottlerocket config paths, and allowing for a known number of failures.
connects #181
depends #178