criblio / helm-charts

Repository for Cribl Helm Charts
MIT License
33 stars 38 forks source link

Wrong value for ingress "prefix" variable #200

Open mpangburn34 opened 3 weeks ago

mpangburn34 commented 3 weeks ago

The default value in logstream-leader/values.yml for the prefix variable is /*. According to Kubernetes docs, the value / means all paths. When testing with the default value in my environment, nginx would give a 404 error on all paths. Once I changed the value to /, the reverse proxy worked as expected.

bdalpe commented 3 weeks ago

You are correct @mpangburn34. The prefix is implementation specific and may require a change to work correctly.

The default value of /* has been maintained for historical reasons and probably won't be changed in this chart as at some point the "logstream" name will be deprecated.