Altinity / Altinity/helm-charts

`operator.enabled` -> `operator.enable_if_not_found`

Open
#25 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
57
Forks
35
PR merge metrics
No merged PRs in 30d

Description

Hi! I have a problem with operator.enabled which seems to be requiring a value of true only if it is the first time installing the chart, because putting it to true while running helm down or helm upgrade seems to fail.
Changing this value after the first deployment is complicated for environments where builds are CI-automated, so I would like for this field to be automatically set to false if the operator is found for instance.

Does that make sense?

https://github.com/Altinity/helm-charts/blob/f65326bbf32bea1d1491dd2005e7922d188c6f90/charts/clickhouse/values.yaml#L137-L140

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Open charts/clickhouse/values.yaml at the referenced operator.enabled setting and read its surrounding comments. Trace where this value is consumed in the chart before deciding the scope. Done means the setting is renamed to operator.enable_if_not_found and avoids requiring manual changes after the operator is already present during installation, upgrade, or helm down.

Written by the indexing model from the issue text.

Assessment

Tech stack
clickhouse, helm, kubernetes
Domain
databases, devops
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.