Bigli comments

Results 9 comments of


                                            Bigli

Support setting priority class of operator in Helm chart

@DebakelOrakel Please sign your commit for the DCO check to pass. Also don't forget to run `make chart-docs`

Made cleanup hook job honor pullSecret settings

@albimn Please sign the commit to fix the DCO and also run `make chart-docs` to fix the ChartLint check. Furthermore a rebase is required now. Until this is fixed, we...

Helm chart: Adding cleanup.enabled and cleanup.imagePullSecrets

@enrique-ayala Thanks for this PR. Please run `make chart-docs` and commit the change. This should fix the ChartLint check. Furthermore ensure that the commits are all [signed](https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits) to ensure the...

feat: add security context for cleanup hook

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

feat: add security context for cleanup hook

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

feat: add security context for cleanup hook

> 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...

enhancement: Added oportuniy set ImagePullSecret

@eunames Could you rebase your branch so we can have a look at the change again and merge it if everything looks good?

Backups that don't trigger any jobs never complete and aren't cleaned up

Hi @haasad The problem should now be resolved with the latest k8up version

[stable/redis-ha] Add additional ports for haproxy

@DandyDeveloper Is there anything else required from my side to move this forward?