Deploy releases/k8s-manifests ce53118#161
Merged
Merged
Conversation
Restructures balancer to the design we settled on before the
Envoy/Gateway migration:
balancer/
kustomization.yaml — wrapper, resources: [app, cnpg]
app/
kustomization.yaml — namespace: balancer, references mapped base
manifests/ — mapped from balancer-main base via hologit
cnpg/
kustomization.yaml — no namespace
database.yaml — Database CR in cloudnative-pg namespace
Why this layout:
- Single hololens (no `balancer-cnpg.toml`)
- `Database` CR lives next to the balancer config in this repo but
cnpg requires it to live in the cluster's namespace (cloudnative-pg).
The cnpg sub-kustomization sets no namespace; database.yaml carries
its own; k8s-normalize routes by resource at deploy time.
- Replaces and supersedes PR #143 (TineoC's two-lens approach with the
sort-order hack and mutable `develop` source ref).
Holosource bumped v1.1.3 → v1.1.5 (latest balancer-main release).
Holomapping filters out `ingress.yaml` (replaced by `_gateways/balancer.yaml`)
and the upstream `kustomization.yaml` (we compose our own).
Also adds `_gateways/balancer.yaml`: per-app Gateway + HTTPRoute on
`balancer.sandbox.k8s.phl.io`. Hostname matches what `secret.template.yaml`
documents and aligns with the per-app pattern used by the other sandbox
apps post-PR-#152.
Database resource will be applied but only fully usable after the
`balancer-db-credentials` SealedSecret is created in the cloudnative-pg
namespace and the balancer app's `balancer-config` is updated to point
SQL_HOST at `shared-cluster-rw.cloudnative-pg.svc.cluster.local` with
the new credentials. That cutover (data migration from the current RDS
host) is a separate follow-up.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
feat(balancer): clean design with shared-cluster cnpg + Envoy Gateway
Author
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
kubectl diffreports that applying ce53118 will change: