Skip to content

Commit 0a90280

Browse files
committed
Add documentation on testing external-dns integration. Update external dns chart. Propagate hostname on contour-istio blueprint gateways
1 parent fcf1327 commit 0a90280

File tree

5 files changed

+40
-6
lines changed

5 files changed

+40
-6
lines changed

Makefile.local

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ deploy-multitool:
104104
#################
105105
.PHONY: deploy-external-dns
106106
deploy-external-dns:
107-
helm upgrade -i --repo https://kubernetes-sigs.github.io/external-dns external-dns external-dns --version 1.12.0 --values test-data/external-dns-values.yaml
107+
helm upgrade -i --repo https://kubernetes-sigs.github.io/external-dns external-dns external-dns --version 1.12.2 --values test-data/external-dns-values.yaml
108108

109109
#################
110110
# https://kind.sigs.k8s.io/docs/user/loadbalancer/
@@ -208,7 +208,7 @@ undeploy-controller:
208208
helm uninstall -n bifrost-gateway-controller-system bifrost-gateway-controller-helm
209209

210210
#################
211-
BIFROST_BLUEPRINTS_VERSION ?= 0.0.18
211+
BIFROST_BLUEPRINTS_VERSION ?= 0.0.19
212212

213213
.PHONY: deploy-controller-blueprint
214214
setup-getting-started-controller-blueprint:

blueprints/contour-istio/gatewayclassblueprint-contour-istio-cert.yaml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,18 @@ spec:
2525
spec:
2626
gatewayClassName: istio
2727
listeners:
28-
{{- toYaml .Gateway.spec.listeners | nindent 6 }}
28+
{{- range .Gateway.spec.listeners }}
29+
- name: {{ .name }}
30+
port: 80
31+
protocol: HTTP
32+
{{ if get . "hostname" }}
33+
hostname: {{ .hostname | quote }}
34+
{{ end -}}
35+
{{ if get . "allowedRoutes" }}
36+
allowedRoutes:
37+
{{ toYaml .allowedRoutes | nindent 6 }}
38+
{{ end -}}
39+
{{- end }}
2940
loadBalancer: |
3041
apiVersion: networking.k8s.io/v1
3142
kind: Ingress

blueprints/contour-istio/gatewayclassblueprint-contour-istio.yaml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,18 @@ spec:
1818
spec:
1919
gatewayClassName: istio
2020
listeners:
21-
{{- toYaml .Gateway.spec.listeners | nindent 6 }}
21+
{{- range .Gateway.spec.listeners }}
22+
- name: {{ .name }}
23+
port: 80
24+
protocol: HTTP
25+
{{ if get . "hostname" }}
26+
hostname: {{ .hostname | quote }}
27+
{{ end -}}
28+
{{ if get . "allowedRoutes" }}
29+
allowedRoutes:
30+
{{ toYaml .allowedRoutes | nindent 6 }}
31+
{{ end -}}
32+
{{- end }}
2233
loadBalancer: |
2334
apiVersion: networking.k8s.io/v1
2435
kind: Ingress

doc/getting-started.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -182,6 +182,20 @@ curl --cacert foo-example-com.crt --resolve foo.example.com:443:127.0.0.1 https:
182182
curl --cacert foo-example-com.crt --resolve foo.example.com:443:127.0.0.1 https://foo.example.com/store
183183
```
184184

185+
### Testing Integration with DNS
186+
187+
The KIND cluster for the getting-started example includes a deployment
188+
of [external-dns](https://github.com/kubernetes-sigs/external-dns),
189+
configured to update a test-only CoreDNS instance. This CoreDNS
190+
instance can be quieried using the tooling container also deployed as
191+
part of the KIND cluster setup:
192+
193+
```
194+
kubectl exec -it deploy/multitool -- dig @coredns-test-only-coredns example.com +short
195+
```
196+
197+
The output should be the same IP address as reported from `kubectl get gateway -n foo-infra` in the `ADDRESS` column.
198+
185199
## Cleanup
186200

187201
```

test-data/coredns-test-values.yaml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,7 @@ servers:
1414
- name: ready
1515
- name: reload
1616
- name: loadbalance
17-
# Addition for local test
1817
- name: etcd
19-
parameters: example-foo4567.com
2018
configBlock: |-
2119
stubzones
2220
path /skydns

0 commit comments

Comments
 (0)