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

Helm chart #51

Closed
kfox1111 opened this issue Jan 27, 2021 · 20 comments
Closed

Helm chart #51

kfox1111 opened this issue Jan 27, 2021 · 20 comments
Labels

Comments

@kfox1111
Copy link
Contributor

It would be nice if the library could compile into an easy to use helm chart.

@kfox1111
Copy link
Contributor Author

Maybe related to #47

@sozercan
Copy link
Member

Are you looking for helm chart for PSPs? In the past, we discussed aligning them into default, restricted buckets that's detailed under https://kubernetes.io/docs/concepts/security/pod-security-standards/. Is this close to what you are looking for?

@kfox1111
Copy link
Contributor Author

Yeah.

Maybe a chart that includes all the ConstraintTemplate's and then maybe some default disabled but easily enable-able default restricted buckets like those? That way its easy to load the library of ConstraintTemplates and enable them as you need them with some sane defaults?

@kfox1111
Copy link
Contributor Author

@stale
Copy link

stale bot commented Feb 1, 2023

This issue/PR has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Feb 1, 2023
@sathieu
Copy link
Contributor

sathieu commented Feb 1, 2023

Not stale, and related to #47 .

@stale stale bot removed the stale label Feb 1, 2023
@stale
Copy link

stale bot commented Apr 2, 2023

This issue/PR has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 2, 2023
@sathieu
Copy link
Contributor

sathieu commented Apr 5, 2023

not stale.

Having an helm chart would help installation.

@stale stale bot removed the stale label Apr 5, 2023
@ritazh
Copy link
Member

ritazh commented Apr 5, 2023

@kfox1111 @sathieu are you looking for specific groups of policies that can be part of a helm chart? Currently for validating policies, we have psp and general. are you looking for 1 helm chart for all psps (maybe filtered by PSS profile level) and 1 for all general policies or just a single helm chart for all?

Another concern: because each policy has its own version, it would be hard to determine how to update the chart's version whenever one of the policies bumps the version.

@sathieu
Copy link
Contributor

sathieu commented Apr 5, 2023

@ritazh I think only one chart is needed (but I'm ok with having two).

On the versioning maybe we could use the max as chart version, and bump all policies to this same version?

Alternatively, the chart version can be independant, and bumped according to semantic versioning.

@starlightromero
Copy link

I agree with the chart version being independent. Usually the Helm chart version is not tied to the application/library version. In a Chart.yaml file, you will typically find the arguments version and appVersion.

@stale
Copy link

stale bot commented Aug 1, 2023

This issue/PR has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 1, 2023
@sozercan
Copy link
Member

sozercan commented Aug 2, 2023

another approach we discussed in the past was a CLI (similar to krew or brew), where users can add/remove/sync policies.

@stale stale bot removed the stale label Aug 2, 2023
@kfox1111
Copy link
Contributor Author

kfox1111 commented Aug 2, 2023

CLI's are harder to integrate into CI/CD systems.

@sozercan
Copy link
Member

sozercan commented Aug 2, 2023

@kfox1111 would you elaborate more in your concerns? isn't helm a cli?

@kfox1111
Copy link
Contributor Author

kfox1111 commented Aug 2, 2023

It isn't when your using a tool such as https://fluxcd.io/ (like: https://fluxcd.io/flux/guides/helmreleases/) or argocd (like: https://argo-cd.readthedocs.io/en/stable/user-guide/helm/)

Kubernetes objects are how you are causing deployment of the charts.

@starlightromero
Copy link

Or in my case, using the Helm Terraform provider, the policies can be codified and on VCS push, a Terraform run is triggered in Terraform Cloud.

@starlightromero
Copy link

starlightromero commented Sep 3, 2023

@kfox1111 Would appreciate your feedback on PR #356

Copy link

stale bot commented Nov 2, 2023

This issue/PR has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 2, 2023
@stale stale bot closed this as completed Nov 16, 2023
@kfox1111
Copy link
Contributor Author

/repoen
/notcompleted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants