Skip to content

fix: sync n8n app before namespace resources - #6

Open
arch-err wants to merge 3 commits into
v2from
fix/n8n-syncwave-namespace
Open

fix: sync n8n app before namespace resources#6
arch-err wants to merge 3 commits into
v2from
fix/n8n-syncwave-namespace

Conversation

@arch-err

@arch-err arch-err commented Jun 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • Reverted the broad sync-wave model change; the hardcoded namespace list was rot and has been removed.
  • Keeps only the original n8n-specific mitigation currently on this branch: sync Application/n8n at wave -1 before parent-rendered n8n namespace resources.

Verification

  • helm lint kubernetes/platform -f kubernetes/infra.yaml
  • helm template kubernetes/platform -f kubernetes/infra.yaml

Note

This PR is intentionally narrow again. A proper cluster-wide prerequisite/namespace sync-wave model needs a separate design, not a hardcoded namespace denylist.

@arch-err arch-err changed the title fix: sync n8n app before namespace resources fix: order platform prerequisites before app sync Jun 13, 2026
@arch-err arch-err changed the title fix: order platform prerequisites before app sync fix: sync n8n app before namespace resources Jun 13, 2026
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.

1 participant