Skip to content

chore: Bump helm chart versions ahead of 26.7 - #424

Merged
NickLarsenNZ merged 17 commits into
mainfrom
chore/helm-bumps
Jul 9, 2026
Merged

chore: Bump helm chart versions ahead of 26.7#424
NickLarsenNZ merged 17 commits into
mainfrom
chore/helm-bumps

Conversation

@NickLarsenNZ

@NickLarsenNZNickLarsenNZ commented Jul 9, 2026•
edited
Loading

Copy link
Copy Markdown
Member

Part of #422

  • Improve update script
  • Do some chart bumps (untested)
Details
❯ .scripts/update_helm_charts.py
⚠️⚠️⚠️ This script is best-effort! Always check the result using "git diff"! ⚠️⚠️⚠️
Notably, it skips invalid YAMLs, which can be the case because we sometimes use templating syntax, even for helm-chart definitions.
In those cases, use quotes around templated values, or otherwise comments for templated blocks.
Please judge on the skipped files if they contain a helm-chart and should be manually bumped.
The script can be improved to handle such files in the future.
✅ Added/Updated timescale (https://charts.timescale.com/)
✅ Updated ./stacks/_templates/postgresql-timescaledb.yaml
✅ Added/Updated minio (https://charts.min.io/)
✅ Updated ./stacks/_templates/minio.yaml
✅ Added/Updated bitnami (https://charts.bitnami.com/bitnami/)
✅ Updated ./stacks/_templates/postgresql-hive.yaml
⚠️ Error processing ./stacks/_templates/vector-aggregator.yaml: Skipping vector. Ensure it is at the same version as the product sidecars
✅ Added/Updated argo-cd (https://argoproj.github.io/argo-helm)
✅ Updated ./stacks/_templates/argo-cd.yaml
✅ Updated ./stacks/_templates/postgresql-druid.yaml
✅ Updated ./stacks/_templates/postgresql-superset.yaml
✅ Updated ./stacks/_templates/postgresql-hive-iceberg.yaml
✅ Updated ./stacks/_templates/postgresql-airflow.yaml
✅ Added/Updated grafana-community (https://grafana-community.github.io/helm-charts)
✅ Updated ./stacks/observability/grafana-loki.yaml
✅ Updated ./stacks/observability/grafana.yaml
✅ Added/Updated opentelemetry-operator (https://open-telemetry.github.io/opentelemetry-helm-charts)
✅ Updated ./stacks/observability/opentelemetry-operator.yaml
✅ Added/Updated jaeger (https://jaegertracing.github.io/helm-charts)
✅ Updated ./stacks/observability/jaeger.yaml
✅ Updated ./stacks/observability/grafana-tempo.yaml
✅ Added/Updated jupyterhub (https://jupyterhub.github.io/helm-chart/)
✅ Updated ./stacks/jupyterhub-keycloak/jupyterhub-native-auth.yaml
✅ Updated ./stacks/jupyterhub-keycloak/jupyterhub.yaml
✅ Added/Updated grafana (https://grafana.github.io/helm-charts)
✅ Updated ./stacks/signal-processing/grafana.yaml
✅ Updated ./stacks/signal-processing/jupyterhub.yaml
✅ Added/Updated prometheus-community (https://prometheus-community.github.io/helm-charts)
✅ Updated ./stacks/monitoring/prometheus.yaml

@NickLarsenNZ
fix(yaml): Quote/comment templating syntax
db2aba9
The update script (.scripts/update_helm_charts.py) skipped these files
@NickLarsenNZ
chore(script): Formatting
7f9e012
@NickLarsenNZ
fix(script): Handle empty appVersion
a92a6e0
@NickLarsenNZ
fix(script): Don't strip leading indentation
935965f
This was matching on vector-aggregator:
```yaml
version: "2"
```
@NickLarsenNZ
chore(script): mention "appVersion" before the app version
bfe0b03
@NickLarsenNZ
chore(helm): Bump versions
5d4c984
@NickLarsenNZ
fix(script): Combine helm repo add with update
38ddf5c
This had a side-effect: When the `repo.url` is updated (eg: https://grafana.github.io/helm-charts -> https://grafana-community.github.io/helm-charts), but the user has a named repo with the old url (because the `repo.name` wasn't updated), this caused chart downgrades.
Now we do an add and force update (once), along with a repo URL check and bail out with this message:
```
⚠️ Error processing ./stacks/observability/grafana.yaml:
You have a local repo named grafana pointing to https://grafana.github.io/helm-charts, but the stack/demo wants https://grafana-community.github.io/helm-charts
Either delete or update your local repo URL, or update the stack/demo repo name so it doesn't conflict
```
@NickLarsenNZ
fix(script): Ignore bumping templated versions
6616080
@NickLarsenNZ
chore(helm): Update grafana repo name
e331d39
@NickLarsenNZNickLarsenNZ changed the title Chore/helm bumpschore: Bump helm chart versions ahead of 26.7Jul 9, 2026
@Techassi
Techassi self-requested a review July 9, 2026 13:31
Techassi
Techassi reviewed Jul 9, 2026
View reviewed changes

@TechassiTechassi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, just a few comments on the script.

Comment thread.scripts/update_helm_charts.py Outdated
Comment thread.scripts/update_helm_charts.py
Comment thread.scripts/update_helm_charts.py Outdated
Comment thread.scripts/update_helm_charts.py Outdated
Comment thread.scripts/update_helm_charts.py
Comment thread.scripts/update_helm_charts.py
Comment threadstacks/_templates/vector-aggregator.yaml Outdated
@TechassiTechassi moved this to Development: In Review in Stackable EngineeringJul 9, 2026
NickLarsenNZand others added 5 commits July 9, 2026 20:10
@NickLarsenNZ@Techassi
Apply suggestions from code review
767c99a
Co-authored-by: Techassi <git@techassi.dev>
@NickLarsenNZ
chore(script): Dedent and """
ed73a38
@NickLarsenNZ
chore(script): Skip Vector bumps
9e576be
These are currently tied to the vector version shipped with products via docker-images.
Hopefully we can move to OTLP for the vector sidecar -> vector-aggregator.
@NickLarsenNZ
revert(helm): Go back to vector 0.55.0
a1c4de1
@NickLarsenNZ
revert(helm): Last minute ArgoCD bump
e410775
NickLarsenNZ
NickLarsenNZ commented Jul 9, 2026
View reviewed changes
Comment threadstacks/_templates/argo-cd.yaml
name: argo-cd
url: https://argoproj.github.io/argo-helm
version: 9.4.10 # v3.3.3
version: 10.1.3 # appVersion: v3.4.5

@NickLarsenNZNickLarsenNZJul 9, 2026

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, 3.4.5 just landed :D

dervoeti
dervoeti previously approved these changes Jul 9, 2026
View reviewed changes
@NickLarsenNZ
NickLarsenNZ enabled auto-merge (squash) July 9, 2026 18:46
dervoeti
dervoeti previously approved these changes Jul 9, 2026
View reviewed changes

@dervoetidervoeti left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fyi: signal-processing still uses https://grafana.github.io/helm-charts, not community.
Is that correct?

@NickLarsenNZ

NickLarsenNZ commented Jul 9, 2026

Copy link
Copy Markdown
MemberAuthor

fyi: signal-processing still uses https://grafana.github.io/helm-charts, not community. Is that correct?

Fixed in dbaf0d9.

@NickLarsenNZ
NickLarsenNZ merged commit b9c7761 into mainJul 9, 2026
2 checks passed
@NickLarsenNZ
NickLarsenNZ deleted the chore/helm-bumps branch July 9, 2026 19:48
@NickLarsenNZNickLarsenNZ moved this from Development: In Review to Development: Track in Stackable EngineeringJul 9, 2026
@NickLarsenNZNickLarsenNZ moved this from Development: Track to Development: Done in Stackable EngineeringJul 9, 2026
This was referenced Jul 10, 2026
Closed
Merged
@lfranckelfrancke moved this from Development: Done to Done in Stackable EngineeringJul 16, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dervoetidervoetidervoeti approved these changes
@TechassiTechassiAwaiting requested review from Techassi

Assignees

No one assigned

Labels

release/26.7.0

Projects

Stackable Engineering
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@NickLarsenNZ@Techassi@dervoeti@lfrancke