Soluto / helm-charts

Soluto's Helm Charts Repository
MIT License
8 stars 12 forks source link

fix: issue #36 googleKms.rotationPeriod converted to int but it's expected to be a string #49

Closed marinkovicpetar closed 3 years ago

marinkovicpetar commented 3 years ago

this should fixed the issue when you omit KeyManagement__GoogleKms__RotationPeriod and chart install fails

shaikatz commented 3 years ago

Hi, @marinkovicpetar thanks for the fix.

Looks like this is a bug introduced with helm3 (helm2 knows how to handle nil values).

Before I'll be able to merge that please bump the chart version.