Switch ARDAC production portal to ardac2prd - #100
Conversation
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
The cutover lacks a mechanism to remove legacy live Argo Applications and their conflicting managed ingresses.
Review effort: Balanced
Findings: None
What changed in this PR
Switches the canonical ARDAC production portal from ardac1prd to ardac2prd, retaining redirects for legacy hostnames.
Changes:
- Reassigns canonical hostnames and redirects.
- Pins both Gen3 Helm revisions.
- Configures External Secrets
v1beta1forardac1prd.
| File | Description |
|---|---|
ardac2prd/README.md |
Documents the new hostname routing. |
ardac2prd/portal.ardac.org/values.yaml |
Makes portal.ardac.org canonical. |
ardac2prd/portal.ardac.org/app.yaml |
Pins the Gen3 Helm revision. |
ardac2prd/load-balancer/manifests/ingress.yaml |
Redirects new.portal to portal. |
ardac1prd/portal.ardac.org/values.yaml |
Makes archive.portal canonical and pins External Secrets API. |
ardac1prd/portal.ardac.org/app.yaml |
Pins the Gen3 Helm revision. |
ardac1prd/load-balancer/manifests/ingress.yaml |
Redirects stale portal traffic to archive. |
ardac1prd/load-balancer/app.yaml |
Repoints and renames the load-balancer Application. |
ardac1prd/archive.portal.ardac.org/manifests/ingress.yaml |
Removes the obsolete archive redirect manifest. |
ardac1prd/archive.portal.ardac.org/app.yaml |
Removes the obsolete Argo Application manifest. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Copilot review follow-up: the legacy live Applications cannot be removed by deleting these repository manifests because they are bootstrapped independently and automated pruning is disabled. After this PR merges, the cutover runbook explicitly deletes There are no line-level review threads to resolve. |
Summary
archive.portal.ardac.orgthe canonical hostname forardac1prdportal.ardac.orgthe canonical hostname forardac2prdportaltraffic on the old ALB toarchivenew.portal.ardac.orgon the new ALB toportalv1beta1onardac1prdValidation
git diff --checkyamllinton changed YAML (only two pre-existing comment-spacing warnings in values files)ardac1prdat6c432454d71bac8f86190d38eb5d6a686f47a9dfardac2prdat2fab13ff7e477a6600f4f18ec05d781cc93d1349v1resourcesOperations
Route 53 remains a manual cutover step. Live AWS secret and Argo updates will be coordinated after this PR is reviewed and merged.