robjuz / helm-charts

https://robjuz.github.io/helm-charts/index.yaml
34 stars 30 forks source link

Nominatim: minor fixes #16

Closed mausch closed 1 year ago

robjuz commented 2 years ago

@mausch this chart does not support nominatim 4 yet. 3.7 is the latest version I testet it

mausch commented 2 years ago

I can remove https://github.com/robjuz/helm-charts/pull/16/commits/70766522028848db65208ad48d4d2c97e985c418 if you want?

Good news is, it does work with Nominatim 4 :slightly_smiling_face: I have some other changes I've not included in this PR in https://github.com/mausch/nominatim-helm-charts/tree/icu . Maybe we can parameterise things a bit more to support both 3.x and 4.x ?