influxdata / influxdata/helm-charts

helm chart does not follow default file layout for engine-path

Open
#529 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

https://github.com/influxdata/helm-charts/blob/a42e6022de5cf0d6a020ae3c1a986396b5bc1122/charts/influxdb2/templates/statefulset.yaml#L103-L104

should be
```
value: {{ .Values.persistence.mountPath }}/engine
```
ref: https://docs.influxdata.com/influxdb/v2.5/reference/internals/file-system-layout/
Also mentioned #474

The consequence of this for our deployment was that we migrated to the helm-chart at some point, and then ended up with the previous metrics in ./engine/data and the new metrics in ./data. All of the old metrics were then not able to be queried.

Contributor guide

Open the contributing guide

Research direction

Inspect charts/influxdb2/templates/statefulset.yaml at the referenced lines and compare the engine path with the InfluxDB file-system layout documentation. Confirm the rendered path uses the persistence mount path followed by /engine, and verify that existing metrics remain under the expected layout.

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
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.