Allow configuring revisionHistoryLimit

[nbarrientos]

No description provided.

[azatsafin]

_ No description provided. _

It will nice if you provide some scenario why you need this change.
Please add the default value for .Values.revisionHistoryLimit in the values.conf.
Please upgrade minor version of helm chart.

[nbarrientos]

Hi @azatsafin

Thanks for taking a look!

I believe that it's useful to be able to limit the number of residual replicasets to retain. Kubernetes' default (10) might be too much once you have reached an acceptable level of maturity/stability when rarely several roll backs are necessary.

I didn't add a default in this chart's values as I thought it'd be a good idea to rely on Kubernetes' default solely but I can replicate it here if you'd like. Users might expect the chart to rely on Kubernetes' default if not specified -- I'd do, personally.

Thanks!

[nbarrientos]

Please upgrade minor version of helm chart.

Done. Assuming semver.

[azatsafin]

LGTM