EugenMayer / helm-charts

Helm charts for applications no official chart exists for
24 stars 19 forks source link

allow verbose logging #6

Closed johanneskastl closed 2 years ago

EugenMayer commented 2 years ago

Besides you need to upstream merge now, should we just expose 'extraArguments' and let the user add whatever needed via the helm chart? In anyway, i'am fine with -v being a option/flag too

johanneskastl commented 2 years ago

Branch was rebased and conflicts were solved.

I woud propose to merge this now, and if lots of other extra arguments appear we can always introduce a new variable, right?

EugenMayer commented 2 years ago

Thank you for pushing it all through. Very appreciated