oauth2-proxy / manifests

For hosting manifests to allow for the deployment of OAuth2-Proxy/OAuth2-Proxy
Apache License 2.0
170 stars 153 forks source link

Avoid unnecessary pod restart on each helm chart version #178

Closed jkroepke closed 11 months ago

jkroepke commented 11 months ago

This PR changes the the input of the config checksum which triggers the reload.

On main branch, the whole configmap/secret checksum is included as checksum input. The configmap contains labels which includes the helm chart version. Each new helm chart version forces a pod restart which is not always necessary.

By define a finer input for the pod annotation checksum, unnecessary pod restarts can be avoided.

The PR implements this for

and could be extended to

pierluigilenoci commented 11 months ago

@jkroepke, could you please add an annotation for ArtifactHub to be approved.

jkroepke commented 11 months ago

@jkroepke, could you please add an annotation for ArtifactHub to be approved.

Done!

pierluigilenoci commented 11 months ago

@jkroepke Thank you for your contribution!