-
Notifications
You must be signed in to change notification settings - Fork 24
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
Kubernetes cluster hardening standard (previously "K8s cluster baseline security") #581
Kubernetes cluster hardening standard (previously "K8s cluster baseline security") #581
Conversation
14a726d
to
472a80e
Compare
2d9145b
to
cd76a26
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.
I like the clear explanation of concepts before recommending something, but it could be argued that these in depth details should not be part of the standard itself. I myself find it very handy to have the reasoning in the same file as the decisions itself.
I would like to hear more opinions on this, though (I could imagine @mbuechse has something to say about this from a meta-standard perspective?) because I think the standard at least somewhat deviates in this regard from our other standards.
One solution could be to move the explanations into a dedicated file, but we could as well just keep this as-is. I have no strong opinion on this matter myself, I just wanted to mention it because I think other people might have objections in this regard.
The standard looks mostly good to me, I have made some comments regarding wording and some - I think - incomplete sentences need fixing.
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.
Just small typos...
a2adca5
to
4b431bd
Compare
Rebased the branch. |
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.
Wow, this is impressive. One problem I have is to find what the standard requires of me. There is a lot of recommendations and best practice suggestions. But can we (in principle) make a succinct list of requirements? Could you please work on an MVP for a test? I think tests are very important to make clear what the standard really requires, and whether it's practical.
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.
Sorry, I totally missed the section "Standard". Okay, so we have a succinct list. I think we can drop the introductory paragraph from that section, but I would add a remark somewhere in the beginning of the "Hardening Kubernetes" paragraph that "Hardening Kubernetes" is not authoritative, and that the "Standards" section contains the authoritative part.
4b431bd
to
b602e3a
Compare
Updated the paragraphs just as mentioned by @mbuechse |
Update baseline cluster security Signed-off-by: Hannes Baum <[email protected]>
b602e3a
to
b44a0ee
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.
LGTM!
I'm approving the draft because it's good work and covers a lot of aspects. The standard paragraph makes sense to me.
Of course these kind of guides always have room for debates around scope and requirements. Does not mean I object to other people's inputs and improvement suggestions 😉
Made a small adjustment to read-only port section in order to address some mentions by @bitkeks. Signed-off-by: Hannes Baum <[email protected]>
Made a small adjustment to related documents in order to address some mentions by @bitkeks. Signed-off-by: Hannes Baum <[email protected]>
This is ready to merge. I trust that you did what you said. So I won't review it now. You can proceed. |
Merging! |
The "Baseline K8s cluster security" was created previously. In the PR (#376) there were some discussions about the structure and overall usefulness of the standard.
This issue should adapt the standard (since it is still in the draft phase) in order to better adhere to the requirements brought forward.
The standard was also retitled to "Kubernetes cluster hardening".