Skip to content

Commit 2d5add1

Browse files
authored
Merge pull request #195986 from feiskyer/patch-2
AKS: switch to /healthz for nginx ingress controller health probes
2 parents bc4f6ce + e61e7ef commit 2d5add1

File tree

7 files changed

+21
-2
lines changed

7 files changed

+21
-2
lines changed

articles/aks/csi-secrets-store-nginx-tls.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ helm install ingress-nginx/ingress-nginx --generate-name \
120120
--namespace $NAMESPACE \
121121
--set controller.replicaCount=2 \
122122
--set controller.nodeSelector."beta\.kubernetes\.io/os"=linux \
123+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
123124
--set defaultBackend.nodeSelector."beta\.kubernetes\.io/os"=linux
124125
```
125126

@@ -136,6 +137,7 @@ helm install ingress-nginx/ingress-nginx --generate-name \
136137
--set controller.replicaCount=2 \
137138
--set controller.nodeSelector."beta\.kubernetes\.io/os"=linux \
138139
--set defaultBackend.nodeSelector."beta\.kubernetes\.io/os"=linux \
140+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
139141
--set controller.podLabels.aadpodidbinding=$AAD_POD_IDENTITY_NAME \
140142
-f - <<EOF
141143
controller:

articles/aks/ingress-basic.md

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,10 @@ NAMESPACE=ingress-basic
5454
helm repo add ingress-nginx https://kubernetes.github.io/ingress-nginx
5555
helm repo update
5656

57-
helm install ingress-nginx ingress-nginx/ingress-nginx --create-namespace --namespace $NAMESPACE
57+
helm install ingress-nginx ingress-nginx/ingress-nginx \
58+
--create-namespace \
59+
--namespace $NAMESPACE \
60+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz
5861
```
5962

6063
### [Azure PowerShell](#tab/azure-powershell)
@@ -65,7 +68,10 @@ $Namespace = 'ingress-basic'
6568
helm repo add ingress-nginx https://kubernetes.github.io/ingress-nginx
6669
helm repo update
6770
68-
helm install ingress-nginx ingress-nginx/ingress-nginx --create-namespace --namespace $Namespace
71+
helm install ingress-nginx ingress-nginx/ingress-nginx `
72+
--create-namespace `
73+
--namespace $Namespace `
74+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz
6975
```
7076

7177
---
@@ -153,6 +159,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx \
153159
--set controller.image.tag=$CONTROLLER_TAG \
154160
--set controller.image.digest="" \
155161
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux \
162+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
156163
--set controller.admissionWebhooks.patch.image.registry=$ACR_URL \
157164
--set controller.admissionWebhooks.patch.image.image=$PATCH_IMAGE \
158165
--set controller.admissionWebhooks.patch.image.tag=$PATCH_TAG \
@@ -183,6 +190,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx `
183190
--set controller.image.tag=$ControllerTag `
184191
--set controller.image.digest="" `
185192
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux `
193+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz `
186194
--set controller.admissionWebhooks.patch.image.registry=$AcrUrl `
187195
--set controller.admissionWebhooks.patch.image.image=$PatchImage `
188196
--set controller.admissionWebhooks.patch.image.tag=$PatchTag `

articles/aks/ingress-internal-ip.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx \
138138
--set controller.admissionWebhooks.patch.image.tag=$PATCH_TAG \
139139
--set controller.admissionWebhooks.patch.image.digest="" \
140140
--set defaultBackend.nodeSelector."kubernetes\.io/os"=linux \
141+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
141142
--set defaultBackend.image.registry=$ACR_URL \
142143
--set defaultBackend.image.image=$DEFAULTBACKEND_IMAGE \
143144
--set defaultBackend.image.tag=$DEFAULTBACKEND_TAG \
@@ -163,6 +164,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx `
163164
--set controller.image.tag=$ControllerTag `
164165
--set controller.image.digest="" `
165166
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux `
167+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz `
166168
--set controller.admissionWebhooks.patch.image.registry=$AcrUrl `
167169
--set controller.admissionWebhooks.patch.image.image=$PatchImage `
168170
--set controller.admissionWebhooks.patch.image.tag=$PatchTag `

articles/aks/ingress-own-tls.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx \
121121
--set controller.image.tag=$CONTROLLER_TAG \
122122
--set controller.image.digest="" \
123123
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux \
124+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
124125
--set controller.admissionWebhooks.patch.image.registry=$ACR_URL \
125126
--set controller.admissionWebhooks.patch.image.image=$PATCH_IMAGE \
126127
--set controller.admissionWebhooks.patch.image.tag=$PATCH_TAG \
@@ -154,6 +155,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx `
154155
--set controller.image.tag=$ControllerTag `
155156
--set controller.image.digest="" `
156157
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux `
158+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz `
157159
--set controller.admissionWebhooks.patch.image.registry=$AcrUrl `
158160
--set controller.admissionWebhooks.patch.image.image=$PatchImage `
159161
--set controller.admissionWebhooks.patch.image.tag=$PatchTag `

articles/aks/ingress-static-ip.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx \
192192
--set controller.image.tag=$CONTROLLER_TAG \
193193
--set controller.image.digest="" \
194194
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux \
195+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
195196
--set controller.admissionWebhooks.patch.image.registry=$ACR_URL \
196197
--set controller.admissionWebhooks.patch.image.image=$PATCH_IMAGE \
197198
--set controller.admissionWebhooks.patch.image.tag=$PATCH_TAG \
@@ -225,6 +226,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx `
225226
--set controller.image.tag=$ControllerTag `
226227
--set controller.image.digest="" `
227228
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux `
229+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz `
228230
--set controller.admissionWebhooks.patch.image.registry=$AcrUrl `
229231
--set controller.admissionWebhooks.patch.image.image=$PatchImage `
230232
--set controller.admissionWebhooks.patch.image.tag=$PatchTag `

articles/aks/ingress-tls.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx \
147147
--set controller.image.tag=$CONTROLLER_TAG \
148148
--set controller.image.digest="" \
149149
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux \
150+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
150151
--set controller.admissionWebhooks.patch.image.registry=$ACR_URL \
151152
--set controller.admissionWebhooks.patch.image.image=$PATCH_IMAGE \
152153
--set controller.admissionWebhooks.patch.image.tag=$PATCH_TAG \
@@ -180,6 +181,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx `
180181
--set controller.image.tag=$ControllerTag `
181182
--set controller.image.digest="" `
182183
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux `
184+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz `
183185
--set controller.admissionWebhooks.patch.image.registry=$AcrUrl `
184186
--set controller.admissionWebhooks.patch.image.image=$PatchImage `
185187
--set controller.admissionWebhooks.patch.image.tag=$PatchTag `

articles/aks/kubernetes-helm.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,7 @@ helm install nginx-ingress ingress-nginx/ingress-nginx \
134134
--set controller.image.tag=$CONTROLLER_TAG \
135135
--set controller.image.digest="" \
136136
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux \
137+
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-health-probe-request-path"=/healthz \
137138
--set controller.admissionWebhooks.patch.image.registry=$ACR_URL \
138139
--set controller.admissionWebhooks.patch.image.image=$PATCH_IMAGE \
139140
--set controller.admissionWebhooks.patch.image.tag=$PATCH_TAG \

0 commit comments

Comments
 (0)