dexidp / helm-charts

Dex Helm chart repository
Apache License 2.0
50 stars 82 forks source link

Add spec.revisionHistoryLimit config possibility for Deployment resource #116

Closed mhoyer closed 1 year ago

mhoyer commented 1 year ago

Overview

Especially when using GitOps approach to deploy DexIdP it makes not real sense to keep revisions of the deployments as those are actually kind of persisted inside the underlying SCM, which drives the deployment and its history.

What this PR does / why we need it

It simply adds the possibility to configure the revisionHistoryLimit property of the Deployment resource template, but keeps the default to 10.

Checklist

nabokihms commented 1 year ago

@mhoyer thanks for the PR. Could you please update the doc?

mhoyer commented 1 year ago

Ok. Got it. I forgot to run make docs again after I did the helm chart version bump.