feat: add security context for cleanup hook

afrimberger commented 2 months ago

Summary

The operator deployment already supports setting podSecurityContext and securityContext. This PR allows setting both for the cleanup Job.

[x] Categorize the PR by setting a good title and adding one of the labels: bug, enhancement, documentation, change, breaking, dependency as they show up in the changelog
[x] PR contains the label area:chart
[x] PR contains the chart label, e.g. chart:k8up
[x] Variables are documented in the values.yaml using the format required by Helm-Docs.
[x] Chart Version bumped if immediate release after merging is planned
[x] I have run make chart-docs
[ ] Link this PR to related code release or other issues.

TheBigLee commented 2 months ago

Hi @afrimberger Could you fix the DCO (sign the commit) so that all tests pass?

afrimberger commented 2 months ago

Hi @TheBigLee,

DCO is "green", now. Thanks for the hint.

I just looked at the Check list and didn't notice the red DCO build. Would be worth adding an entry "Commits signed off" to the Check list :).

TheBigLee commented 1 month ago

@afrimberger Can you run make chart-docs once more to fix the ChartLint check? once this is done, we'll merge this PR.

Cheers

TheBigLee commented 1 month ago

I just looked at the Check list and didn't notice the red DCO build. Would be worth adding an entry "Commits signed off" to the Check list :).

Thanks for the idea. We've added it to the checklist in #974

afrimberger commented 1 month ago

@TheBigLee : docs are updated and I added another small commit for fixing the labels for the Grafana Dashboard ConfigMap