Skip to content

Commit 15071d9

Browse files
authored
Merge pull request #1338 from andyzhangx/http-endpoint-1.23
[release-1.23] feat: use new restricted liveness probe endpoint
2 parents bbd45be + ea2a214 commit 15071d9

File tree

5 files changed

+10
-20
lines changed

5 files changed

+10
-20
lines changed
400 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
@@ -173,15 +173,12 @@ spec:
173173
- "--mount-permissions={{ .Values.node.mountPermissions }}"
174174
- "--allow-inline-volume-key-access-with-idenitity={{ .Values.node.allowInlineVolumeKeyAccessWithIdentity }}"
175175
- "--enable-aznfs-mount={{ .Values.node.enableAznfsMount }}"
176-
ports:
177-
- containerPort: {{ .Values.node.livenessProbe.healthPort }}
178-
name: healthz
179-
protocol: TCP
180176
livenessProbe:
181177
failureThreshold: 5
182178
httpGet:
179+
host: localhost
183180
path: /healthz
184-
port: healthz
181+
port: {{ .Values.node.livenessProbe.healthPort }}
185182
initialDelaySeconds: 30
186183
timeoutSeconds: 10
187184
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
@@ -129,15 +129,12 @@ spec:
129129
- "--blobfuse-proxy-endpoint=$(BLOBFUSE_PROXY_ENDPOINT)"
130130
- "--nodeid=$(KUBE_NODE_NAME)"
131131
- "--user-agent-suffix=OSS-kubectl"
132-
ports:
133-
- containerPort: 29633
134-
name: healthz
135-
protocol: TCP
136132
livenessProbe:
137133
failureThreshold: 5
138134
httpGet:
135+
host: localhost
139136
path: /healthz
140-
port: healthz
137+
port: 29633
141138
initialDelaySeconds: 30
142139
timeoutSeconds: 10
143140
periodSeconds: 30

0 commit comments

Comments
 (0)