Skip to content

Commit

Permalink
feat: Provide capability to configure DNS suffix for ingress controll…
Browse files Browse the repository at this point in the history
…er (#258)

Signed-off-by: Tom Kerkhove <[email protected]>
  • Loading branch information
tomkerkhove authored Oct 13, 2023
1 parent dd4690f commit 9b8614d
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 0 deletions.
5 changes: 5 additions & 0 deletions helm-charts/azure-api-management-gateway/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,11 @@ their default values.
| `ingress.tls` | Configuration for TLS on the ingress | None |
| `ingress.ingressClassName` | Configuration of the ingress class to be used | |
| `ingress.useHttpsBackend` | Indication whether or not the https port of the service should be used | `false` |
| `ingress.controller.enabled` | Indication whether or not Self-Hosted gateway should act as an Ingress controller. ([experimental feature])(https://github.com/Azure/api-management-self-hosted-gateway-ingress) | `false` |
| `ingress.controller.namespace` | Kubernetes namespace to watch. ([experimental feature])(https://github.com/Azure/api-management-self-hosted-gateway-ingress) | `default` |
| `ingress.controller.annotations` | Indication whether or not Self-Hosted gateway should act as an Ingress controller. ([experimental feature])(https://github.com/Azure/api-management-self-hosted-gateway-ingress) | `[]` |
| `ingress.controller.dns.suffix` | DNS suffix to use to build Kubernetes hostname for services inside the cluster. ([experimental feature])(https://github.com/Azure/api-management-self-hosted-gateway-ingress) | `svc.cluster.local` |
| `ingress.controller.ingressClass.annotations` | Annotationts to apply to ingress class ([experimental feature])(https://github.com/Azure/api-management-self-hosted-gateway-ingress) | `[]`|
| `serviceAccountName` | Configuration of the serviceAccountName used | default |
| `highAvailability.enabled` | Indication whether or not the gateway should be scheduled highly available in the cluster. | `false` |
| `highAvailability.disruption.maximumUnavailable` | Amount of pods that are allowed to be unavailable due to [voluntary disruptions](https://kubernetes.io/docs/concepts/workloads/pods/disruptions/#voluntary-and-involuntary-disruptions). | `25%` |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ data:
{{- if .Values.ingress.controller.enabled }}
k8s.ingress.enabled : {{ .Values.ingress.controller.enabled | quote }}
k8s.ingress.namespace : {{ .Values.ingress.controller.namespace | default ( .Release.Namespace ) | quote }}
k8s.ingress.dns.suffix: {{ .Values.ingress.controller.dns.suffix | quote }}
{{- end }}
telemetry.metrics.cloud: {{ .Values.observability.azureMonitor.metrics.enabled | quote }}
telemetry.logs.std: {{ .Values.observability.logs.std.format | quote }}
Expand Down
2 changes: 2 additions & 0 deletions helm-charts/azure-api-management-gateway/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,8 @@ ingress:
enabled: false
namespace: ""
annotations: []
dns:
suffix: "svc.cluster.local"
ingressClass:
controller: "azure-api-management/gateway"
annotations: []
Expand Down

0 comments on commit 9b8614d

Please sign in to comment.