openova/openova
1
0
Fork 0
Code Issues Pull Requests Actions 14 Packages Projects Releases Wiki Activity

docs(principles): clarify #14 — HelmRelease.dependsOn cannot reference Kustomizations (empirical t27 finding) (#1878)

Browse Source 
A84 empirical finding (t27 / PR #1875): HelmRelease.spec.dependsOn
strictly references OTHER HelmReleases — it cannot reference Flux
Kustomizations or other resource kinds. PR #1875 added the `sovereign-tls`
Kustomization to a HelmRelease's dependsOn; helm-controller logged
`helmreleases "sovereign-tls" not found` and retried every 30s forever.

Adds a critical sub-rule to principle #14 documenting the cross-kind
limitation, the recommended workaround (wait-HelmRelease shim or move the
gated workload into a Kustomization), and the verbatim helm-controller
error message so the next regression is greppable.

Doc-only.

Co-authored-by: hatiyildiz <claude@openova.io>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
e3mrah 2026-05-19 04:00:25 +04:00 committed by GitHub
parent 2e1826abb4
commit 366d5d2b33
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
docs/INVIOLABLE-PRINCIPLES.md
View File

@ -204,6 +204,17 @@ If a future ticket, agent, or operator session tries to ship a Sovereign without
- The cutover chart must publish a rollback Job — flipping HelmRepository URLs is destructive in the same sense `tofu destroy` is; treat it accordingly.
- Never sequence "flip URLs" before "prove Gateway TLS works" inside the same blueprint. If they share a slot, the slot is wrong.
**Critical sub-rule (empirical 2026-05-19 on t27 — PR #1875 incident)**:
`HelmRelease.spec.dependsOn` references ONLY other HelmReleases. It CANNOT
reference Flux Kustomizations or other resource kinds. If you need to gate
a HelmRelease on a Kustomization, ship a "wait-HelmRelease" (tiny chart
with a Job that runs `kubectl wait …`) and depend on THAT HR. Or move the
gated workload into a Kustomization with cross-kind `dependsOn`.
Empirical verbatim from helm-controller when this rule was violated:
`unable to get 'flux-system/<name>' dependency: helmreleases.helm.toolkit.fluxcd.io "<name>" not found`
→ retries every 30s forever, never resolves.
---
## Self-check before every commit