[adwk67]

Description

fixes #472

Minimum set of demos to test these changes:

• 🟢 signal-processing
• 🔴 argo-cd-git-ops. The helm chart changes work, though various other things are broken (minio, resources, namespace) but not related to the helm charts. Airflow resources for the gitsync container have been raised as it was OOM-ing.
• 🟢 trino-taxi-data
• 🟢 jupyterhub-keycloak: needs port passed explicitly with the new version. Tested with the spark image.
• 🟢 the observability stack
• 🟢 the monitoring stack

N.B. I haven't yet removed (or tested) the namespace limitiation from the argo-cd demo, nor the switch from minio to garage (though I may push some more commits to this PR).

Docs

Release note

Checklist

Author

• AI review done, findings addressed
• Tests added or updated
• Changelog updated (if the repo keeps one)

Reviewer

• Useful comments and logging
• Docs added or updated per the style guide

Acceptance

• Check for release notes and docs

[Techassi]

Did you by any chance also check running .scripts/update_helm_charts.py? It produces a single-line diff for me locally.

[adwk67]

Did you by any chance also check running .scripts/update_helm_charts.py? It produces a single-line diff for me locally.

Yes, I did - it complained about jupyterhub not being valid yaml but I checked that file in the sample demos anyway.

[Techassi]

Mhm curious, it ran through fine for me. This is the diff:

diff --git a/stacks/observability/grafana-loki.yaml b/stacks/observability/grafana-loki.yaml
index 603bf36..249c658 100644
--- a/stacks/observability/grafana-loki.yaml
+++ b/stacks/observability/grafana-loki.yaml
@@ -8,7 +8,7 @@ repo:
 # To find the latest version:
 # helm repo add grafana-community https://grafana-community.github.io/helm-charts --force-update
 # helm search repo grafana/loki
-version: 21.0.0 # appVersion: 3.7.9
+version: 21.1.1 # appVersion: 3.7.9
 options:
   deploymentMode: SingleBinary
   singleBinary: