-
Notifications
You must be signed in to change notification settings - Fork 444
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
[Cloud Posture] EKS rule templates #3701
Conversation
/test |
@@ -1,4 +1,9 @@ | |||
# newer versions go on top | |||
- version: "0.0.20" |
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.
nit we haven't promoted version 0.0.19
consider added the changes to that version backlog
see example here
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.
"API Server" | ||
], | ||
"benchmark": { | ||
"name": "CIS Kubernetes V1.23", |
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.
missing benchmark id "cis_k8s"
- maybe you are missing a merge from main?
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.
@@ -1,7 +1,7 @@ | |||
format_version: 1.0.0 | |||
name: cloud_security_posture | |||
title: "CIS Kubernetes Benchmark" | |||
version: 0.0.18 | |||
version: 0.0.20 |
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.
consider using version 0.0.19 instead
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.
# Conflicts: # packages/cloud_security_posture/changelog.yml # packages/cloud_security_posture/manifest.yml
🌐 Coverage report
|
# Conflicts: # packages/cloud_security_posture/changelog.yml
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.
missing version
field
"impact": "Removal of the read-only port will require that any service which made use of it will need to be re-configured to use the main Kubelet API.\n", | ||
"default_value": "See the Amazon EKS documentation for the default value.\n", | ||
"references": "1. https://kubernetes.io/docs/admin/kubelet/\n", | ||
"section": "Kubelet", |
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.
missing version
"default_value": "See the Amazon EKS documentation for the default value.\n", | ||
"references": "1. https://kubernetes.io/docs/admin/kubelet/\n2. https://github.com/kubernetes/kubernetes/issues/22063\n3. https://kubernetes.io/docs/tasks/administer-cluster/reconfigure-kubelet/\n", | ||
"section": "Kubelet", | ||
"tags": [ |
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.
missing version
"impact": "None\n", | ||
"default_value": "See the Amazon EKS documentation for the default value.\n", | ||
"references": "1. https://github.com/kubernetes/kubernetes/pull/45059\n2. https://kubernetes.io/docs/admin/kubelet-tls-bootstrapping/#kubelet-configuration\n", | ||
"section": "Kubelet", |
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.
missing version
"default_value": "See the Amazon EKS documentation for the default value.\n", | ||
"references": "1. https://kubernetes.io/docs/admin/kubelet/ 2. https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/apis/kubeletconfig/v1beta1/types.go 3. https://kubernetes.io/docs/tasks/administer-cluster/reconfigure-kubelet/\n", | ||
"section": "Kubelet", | ||
"tags": [ |
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.
missing version
"default_value": "See the EKS documentation for the default value.\n", | ||
"references": "1. https://kubernetes.io/docs/admin/kubelet/\n2. https://github.com/kubernetes/kubernetes/pull/18552\n", | ||
"section": "Kubelet", | ||
"tags": [ |
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.
missing version
"default_value": "See the Amazon EKS documentation for the default value.\n", | ||
"references": "1. https://github.com/kubernetes/kubernetes/pull/41912\n2. https://kubernetes.io/docs/reference/command-line-tools-reference/kubelet-tls-bootstrapping/#kubelet-configuration\n3. https://kubernetes.io/docs/imported/release/notes/\n4. https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates/\n5. https://kubernetes.io/docs/tasks/administer-cluster/reconfigure-kubelet/\n", | ||
"section": "Kubelet", | ||
"tags": [ |
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.
missing version
…-rule-templates # Conflicts: # packages/cloud_security_posture/changelog.yml
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.
works for me locally
however we still have the same formatting issues like we had before
https://github.com/elastic/security-team/issues/3672
What does this PR do?
Adding EKS benchmark rule templates
Checklist
I have verified that all data streams collect metrics or logs.changelog.yml
file.I have verified that Kibana version constraints are current according to guidelines.Author's Checklist
Related issues