netbox-community/netbox-chart

allowedHostsIncludesPodIP currently broken

Zaunei opened this issue · 0 comments

After deploying the current helm chart, I also ran into #166 and had a deeper look at why the PodIP is missing in my allowedHosts.

There are currently two problems with allowedHostsIncludesPodIP. The P in IP is capitalized in values.yaml, but not in the templates. If you want to use allowedHostsIncludesPodIp in your values.yaml as a workaround, helm template/install fails:

Error: YAML parse error on netbox/templates/deployment.yaml: error converting YAML to JSON: yaml: line 96: did not find expected key

This is caused by incorrect indentation in the deployment template, as there is one space too many.

I create a PR straight away to fix both problems.