lucky-sideburn / kubeinvaders

Gamified Chaos Engineering Tool for Kubernetes
Apache License 2.0
1.02k stars 127 forks source link

WIP: feat!(helm): refactor helm chart #50

Closed the-technat closed 2 years ago

the-technat commented 2 years ago

Hey!

While bringing support for securityContext and volumes I discovered that the values.yaml file looks as it grew historically and could need some refactoring.

So I did some refactorings:

I'm not sure if this PR is ready to merge yet. I tested most of the values if they are correctly rendered. But one thing I have no idea if it's correct as is is nameOverride and fullNameOverride. Guess I need to invest some more time into this. Or what do you think?

lucky-sideburn commented 2 years ago

Dear @the-technat,

I release a new version of the helm chart with your changes.

Thank you for contributing!