influxdata / influxdata/helm-charts

Kapacitor: Non-existent default value for .Values.influxURL prevents deployment from being rendered

Open Beginner friendly
#472 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Mustache
Stars
257
Forks
347
Avg merge
4d 20h
Merged PRs (30d)
9

Description

The deployment for Kapacitor is only rendered if `.Values.influxURL` is set:
https://github.com/influxdata/helm-charts/blob/bae0041c452343c91d6123fca199471a8e1aaf95/charts/kapacitor/templates/deployment.yaml#L1-L2

That value is documented as having a default value of `http://influxdb-influxdb.tick:8086`:

https://github.com/influxdata/helm-charts/blob/bae0041c452343c91d6123fca199471a8e1aaf95/charts/kapacitor/README.md?plain=1#L64

However, it is not set in the default `values.yaml`:
https://github.com/influxdata/helm-charts/blob/bae0041c452343c91d6123fca199471a8e1aaf95/charts/kapacitor/values.yaml

This causes the chart to render without deployment if the default values are used.

Contributor guide

Open the contributing guide

Research direction

Start with charts/kapacitor/values.yaml and templates/deployment.yaml, then compare them with the documented value in charts/kapacitor/README.md. Render the Kapacitor chart using default values and verify that the deployment is present and the documented default is consistent.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm, kubernetes
Domain
devops, infrastructure
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.