-
Notifications
You must be signed in to change notification settings - Fork 8
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
[Security Hardened Shoot Cluster] Rule 2003 implementation #343
Conversation
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! There are some nits to fix.
@@ -0,0 +1,182 @@ | |||
// SPDX-FileCopyrightText: 2023 SAP SE or an SAP affiliate company and Gardener contributors |
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.
// SPDX-FileCopyrightText: 2023 SAP SE or an SAP affiliate company and Gardener contributors | |
// SPDX-FileCopyrightText: 2024 SAP SE or an SAP affiliate company and Gardener contributors |
func() {}, | ||
rule.RuleResult{RuleID: ruleID, RuleName: ruleName, CheckResults: []rule.CheckResult{{Status: rule.Passed, Message: "Default kubelet config does not disable kernel protection.", Target: rule.NewTarget()}}}, | ||
), | ||
Entry("should pass when shoot does sets default 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.
Entry("should pass when shoot does sets default kubelet config", | |
Entry("should pass when shoot sets default kubelet config", |
{Status: rule.Passed, Message: "Worker kubelet config enables kernel protection.", Target: rule.NewTarget("worker", "worker1")}, | ||
}}, | ||
), | ||
Entry("should pass when shoot worker enables kernel defaults protection in 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.
Entry("should pass when shoot worker enables kernel defaults protection in kubelet config", | |
Entry("should fail when shoot worker disables kernel defaults protection in kubelet config", |
@@ -0,0 +1,17 @@ | |||
// SPDX-FileCopyrightText: 2023 SAP SE or an SAP affiliate company and Gardener contributors |
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.
// SPDX-FileCopyrightText: 2023 SAP SE or an SAP affiliate company and Gardener contributors | |
// SPDX-FileCopyrightText: 2024 SAP SE or an SAP affiliate company and Gardener contributors |
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
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note: