pomerium / pomerium-helm

Official helm charts for Pomerium.
https://helm.pomerium.io/
55 stars 70 forks source link

helm chart uses deprecated cache_service_url #160

Closed tobiaskohlbau closed 4 years ago

tobiaskohlbau commented 4 years ago

What happened?

The helm chart to deploy pomerium uses deprecated _cache_serviceurl configuration option. This results in pomerium complaining at startup that this option will be removed in v0.11.0.

{"level":"warn","time":"2020-10-13T09:36:10Z","message":"config: cache url will be deprecated in v0.11.0"

What did you expect to happen?

Use replacement option _databroker_serviceurl instead.

Steps to reproduce

  1. Deploy pomerium with helm chart.

What's your environment like?

Additional context

https://github.com/pomerium/pomerium-helm/blob/47d9ff850e32f4ac03456b6599a5ff5c4612da17/charts/pomerium/templates/_helpers.tpl#L406

Should switch over to _databroker_serviceurl.

tobiaskohlbau commented 4 years ago

Feel free to assign this issue to me. I will try to get the PR merged and will apply any requested changes.