Skip to content

Commit acd616f

Browse files
authored
Merge pull request #1337 from andyzhangx/http-endpoint-1.24
[release-1.24] feat: use new restricted liveness probe endpoint
2 parents de0c085 + b68339d commit acd616f

File tree

5 files changed

+10
-20
lines changed

5 files changed

+10
-20
lines changed
14 Bytes
Binary file not shown.

charts/latest/blob-csi-driver/templates/csi-blob-controller.yaml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ spec:
9191
args:
9292
- --csi-address=/csi/csi.sock
9393
- --probe-timeout=3s
94-
- --health-port={{ .Values.controller.livenessProbe.healthPort }}
94+
- --http-endpoint=localhost:{{ .Values.controller.livenessProbe.healthPort }}
9595
imagePullPolicy: {{ .Values.image.livenessProbe.pullPolicy }}
9696
volumeMounts:
9797
- name: socket-dir
@@ -114,17 +114,15 @@ spec:
114114
- "--cloud-config-secret-namespace={{ .Values.controller.cloudConfigSecretNamespace }}"
115115
- "--allow-empty-cloud-config={{ .Values.controller.allowEmptyCloudConfig }}"
116116
ports:
117-
- containerPort: {{ .Values.controller.livenessProbe.healthPort }}
118-
name: healthz
119-
protocol: TCP
120117
- containerPort: {{ .Values.controller.metricsPort }}
121118
name: metrics
122119
protocol: TCP
123120
livenessProbe:
124121
failureThreshold: 5
125122
httpGet:
123+
host: localhost
126124
path: /healthz
127-
port: healthz
125+
port: {{ .Values.controller.livenessProbe.healthPort }}
128126
initialDelaySeconds: 30
129127
timeoutSeconds: 10
130128
periodSeconds: 30

charts/latest/blob-csi-driver/templates/csi-blob-node.yaml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -174,15 +174,12 @@ spec:
174174
- "--allow-inline-volume-key-access-with-idenitity={{ .Values.node.allowInlineVolumeKeyAccessWithIdentity }}"
175175
- "--enable-aznfs-mount={{ .Values.node.enableAznfsMount }}"
176176
- "--metrics-address=0.0.0.0:{{ .Values.node.metricsPort }}"
177-
ports:
178-
- containerPort: {{ .Values.node.livenessProbe.healthPort }}
179-
name: healthz
180-
protocol: TCP
181177
livenessProbe:
182178
failureThreshold: 5
183179
httpGet:
180+
host: localhost
184181
path: /healthz
185-
port: healthz
182+
port: {{ .Values.node.livenessProbe.healthPort }}
186183
initialDelaySeconds: 30
187184
timeoutSeconds: 10
188185
periodSeconds: 30

deploy/csi-blob-controller.yaml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ spec:
6262
args:
6363
- --csi-address=/csi/csi.sock
6464
- --probe-timeout=3s
65-
- --health-port=29632
65+
- --http-endpoint=localhost:29632
6666
volumeMounts:
6767
- name: socket-dir
6868
mountPath: /csi
@@ -81,17 +81,15 @@ spec:
8181
- "--metrics-address=0.0.0.0:29634"
8282
- "--user-agent-suffix=OSS-kubectl"
8383
ports:
84-
- containerPort: 29632
85-
name: healthz
86-
protocol: TCP
8784
- containerPort: 29634
8885
name: metrics
8986
protocol: TCP
9087
livenessProbe:
9188
failureThreshold: 5
9289
httpGet:
90+
host: localhost
9391
path: /healthz
94-
port: healthz
92+
port: 29632
9593
initialDelaySeconds: 30
9694
timeoutSeconds: 10
9795
periodSeconds: 30

deploy/csi-blob-node.yaml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -131,15 +131,12 @@ spec:
131131
- "--user-agent-suffix=OSS-kubectl"
132132
- "--metrics-address=0.0.0.0:29635"
133133
- "--enable-aznfs-mount=true"
134-
ports:
135-
- containerPort: 29633
136-
name: healthz
137-
protocol: TCP
138134
livenessProbe:
139135
failureThreshold: 5
140136
httpGet:
137+
host: localhost
141138
path: /healthz
142-
port: healthz
139+
port: 29633
143140
initialDelaySeconds: 30
144141
timeoutSeconds: 10
145142
periodSeconds: 30

0 commit comments

Comments
 (0)