Skip to content

Commit e12ca41

Browse files
committed
chore: drop dns_zone from site configuration
The only place this was used was for Argo Workflows but we need the Dex URL and we need the Ingress URL for the service so having this as one value would have never worked. To actually do this you need to use the documentation configuration. This leaves dns_zone in place, because that's still used in the ArgoCD deployment.
1 parent 7c39a6b commit e12ca41

File tree

2 files changed

+0
-3
lines changed

2 files changed

+0
-3
lines changed

docs/deploy-guide/extra-sites.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ place it where we've cloned understack. A complete file would look like:
2727
# this can remain as the literal value and will ensure it computes the right path
2828
UC_DEPLOY="$(cd "$(dirname ${BASH_SOURCE[0]})" && git rev-parse --show-toplevel)"
2929
DEPLOY_NAME="my-site"
30-
DNS_ZONE=home.lab
3130
UC_DEPLOY_EMAIL="my@email"
3231
NO_ARGOCD=yes
3332
```
@@ -69,7 +68,6 @@ metadata:
6968
uc_repo_ref: HEAD
7069
uc_deploy_git_url: "${UC_DEPLOY_GIT_URL}"
7170
uc_deploy_ref: HEAD
72-
dns_zone: "${DNS_ZONE}"
7371
labels:
7472
argocd.argoproj.io/secret-type: cluster
7573
type: Opaque

go/understackctl/cmd/argocd/templates/argoSecretDeployRepo.tmpl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,3 @@ metadata:
1616
uc_repo_ref: "HEAD"
1717
uc_deploy_git_url: "{{ .UC_DEPLOY_GIT_URL }}"
1818
uc_deploy_ref: "HEAD"
19-
dns_zone: "{{ .DNS_ZONE }}"

0 commit comments

Comments
 (0)