cockroachdb / cockroachdb/helm-charts

Helm chart should specify cache and max-sql-memory programmatically

Open
#165 0 comments 0 reactions 0 assignees View on GitHub
C-enhancement
Dominant language
Go
Stars
105
Forks
152
Avg merge
4d 23h
Merged PRs (30d)
5

Description

The `cache` and `max-sql-memory` fields in the [Helm chart](https://github.com/cockroachdb/helm-charts/blob/master/cockroachdb/values.yaml) should use programmatic values like in the [StatefulSet configs](https://github.com/cockroachdb/cockroach/blob/master/cloud/kubernetes/bring-your-own-certs/cockroachdb-statefulset.yaml#L220).

We have been documenting a workaround to override the Helm chart by setting these to explicit values, so having this work as-is would be an improvement.

Jira issue: HELM-53

Contributor guide

No contributing guide indexed for this repository

Research direction

Compare the cache and max-sql-memory entries in values.yaml with the corresponding programmatic values in cockroachdb-statefulset.yaml around line 220. Confirm how the Helm chart currently renders these fields and update it so the chart works without explicit override values.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm
Domain
devops
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.