Skip to content
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

Expand security-hardening/ subsection, reorder howtos/ section #562

Merged
merged 9 commits into from
Mar 18, 2025

Conversation

viccuad
Copy link
Member

@viccuad viccuad commented Mar 13, 2025

Description

Related to #558.

While working on kubewarden/community#40, I feel it's easier to provide a more featured security-hardening section than to write the contents of this PR in the self-assessment which is not so accessible.

Expanded with a summary on how to harden Kubewarden installations.

Reorder howtos/ section, from more general and newbie topics, to more specific and obscure.

Test

Additional Information

Tradeoff

Potential improvement

viccuad added 3 commits March 13, 2025 14:17
Signed-off-by: Víctor Cuadrado Juan <[email protected]>
Showcases that Kubewarden is not providing the air gap.

Signed-off-by: Víctor Cuadrado Juan <[email protected]>
Signed-off-by: Víctor Cuadrado Juan <[email protected]>
@viccuad viccuad requested a review from a team as a code owner March 13, 2025 13:18
@viccuad viccuad self-assigned this Mar 13, 2025
Copy link

netlify bot commented Mar 13, 2025

Deploy Preview for docs-kubewarden-io ready!

Name Link
🔨 Latest commit 1089337
🔍 Latest deploy log https://app.netlify.com/sites/docs-kubewarden-io/deploys/67d8467c45d8970008bbe91b
😎 Deploy Preview https://deploy-preview-562--docs-kubewarden-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

viccuad added 4 commits March 13, 2025 14:26
The secure-supply-chain page is about securing Kubewarden only.

Signed-off-by: Víctor Cuadrado Juan <[email protected]>
From more general and newbie topics, to more specific and obscure.

Signed-off-by: Víctor Cuadrado Juan <[email protected]>
@viccuad viccuad force-pushed the from-self-assessment branch from d9238f1 to acfc73b Compare March 13, 2025 13:27
@flavio
Copy link
Member

flavio commented Mar 13, 2025

@viccuad JFYI, I think that, starting from 1.23, we will be able to run the whole KW stack inside of a namespace where the restricted PSA is enforced

Copy link
Member

@flavio flavio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for having handled that!

Signed-off-by: Víctor Cuadrado Juan <[email protected]>
@jhkrug
Copy link
Contributor

jhkrug commented Mar 14, 2025

Hi from Orlando airport! I will be able to review in more detail next week. But looks fine so far. The move from 'air-gap' to 'air gap'. No problem, just wondering what's the driver? Is there a reference that recommends one over the other that you have in mind? It's the sort of thing that we would want to be used consistently. Ideally across SUSE.

@viccuad
Copy link
Member Author

viccuad commented Mar 17, 2025

The move from 'air-gap' to 'air gap'. No problem, just wondering what's the driver?

I went for consistency inside docs.kubewarden.io, but I'm ok with any nomenclature!

We can discuss about it in the next days, no rush :).

Copy link
Contributor

@jhkrug jhkrug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, a few suggestions. Thanks.

```

This provides a default PolicyServer and default policies in protect mode to
ensure the Kubewarden stack is safe from other workloads.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is 'safe' the right word? Maybe 'partitioned from' or 'separate to'? Just wondering.

Suggested change
ensure the Kubewarden stack is safe from other workloads.
ensure the Kubewarden stack is safe from other workloads.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say it's correct, by deploying the kubewarden-defaults chart we ensure the safety of Kubewarden. It's already partitioned, but if we allow other workloads to deploy with privileges (because the kubewarden-defaults chart is not deployed for example) then Kubewarden, and by extension the rest of the cluster, is not safe.

@jhkrug
Copy link
Contributor

jhkrug commented Mar 17, 2025

The move from 'air-gap' to 'air gap'. No problem, just wondering what's the driver?

I went for consistency inside docs.kubewarden.io, but I'm ok with any nomenclature!

We can discuss about it in the nexy days, no rush :).

Great. Consistency was a word used a lot in my presentation at SC. It's, in my view, the most important thing in a set of docs. We also have a company-wide technical glossary called termweb, recently adopted, looking for wider usage. Air gap is not in it yet and I'll do something about that.

@jhkrug jhkrug added the area/documentation Improvements or additions to documentation label Mar 17, 2025
Co-authored-by: John Krug <[email protected]>
Signed-off-by: Víctor Cuadrado Juan <[email protected]>
@viccuad viccuad requested a review from jhkrug March 18, 2025 09:49
@viccuad
Copy link
Member Author

viccuad commented Mar 18, 2025

@jhkrug I found Rancher 1.x docs mention "Air gap" mostly, while in 2.x there's "Air gap" and "Air-gap".

Is it ok merging this PR as-is? I'm happy changing the air gap mentions now or later of course.

@jhkrug
Copy link
Contributor

jhkrug commented Mar 18, 2025

@jhkrug I found Rancher 1.x docs mention "Air gap" mostly, while in 2.x there's "Air gap" and "Air-gap".

Is it ok merging this PR as-is? I'm happy changing the air gap mentions now or later of course.

I am happy that we are consistent within KW for now. Consistency over cloud native will come over time. Terms that are in the glossary/termweb will become controlled and set as variable/attributes to be used in the markdown/asciidoc.

Copy link
Contributor

@jhkrug jhkrug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@viccuad
Copy link
Member Author

viccuad commented Mar 18, 2025

merging, thanks!

@viccuad viccuad merged commit e7f40fd into kubewarden:main Mar 18, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants