Skip to content

chore: regenerate Helm diagram when updating the version#893

Merged
chrisburr merged 1 commit intoDIRACGrid:mainfrom
chaen:update_diagram_pushing_chart
Apr 16, 2026
Merged

chore: regenerate Helm diagram when updating the version#893
chrisburr merged 1 commit intoDIRACGrid:mainfrom
chaen:update_diagram_pushing_chart

Conversation

@chaen
Copy link
Copy Markdown
Contributor

@chaen chaen commented Apr 16, 2026

The automatic pushes to the charts are failing the CI because the version is part of the mermaid diagram.

https://github.com/DIRACGrid/diracx-charts/actions/runs/24500801671/job/71606798020

Run if ! git diff --exit-code docs/admin/explanations/architecture_diagram.md; then
diff --git a/docs/admin/explanations/architecture_diagram.md b/docs/admin/explanations/architecture_diagram.md
index 98286a6..1d08398 100644
--- a/docs/admin/explanations/architecture_diagram.md
+++ b/docs/admin/explanations/architecture_diagram.md
@@ -5,7 +5,7 @@ config:
 flowchart TD
 
     subgraph k8s_instance ["K8s Instance: diracx"]
-        subgraph helm_chart ["Helm Chart: diracx 1.0.22"]
+        subgraph helm_chart ["Helm Chart: diracx 1.0.24"]
             subgraph k8s_app ["K8s Application: diracx"]
                 ing_diracx{{"ing: diracx"}}
                 svc_diracx(("svc: diracx"))
Error: architecture_diagram.md is out of date. Run 'pixi run generate-diagram' and commit the result.
Error: Process completed with exit code 1.

This PR forces the diagram regeneration

@chrisburr chrisburr merged commit 4c7853a into DIRACGrid:main Apr 16, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants