Skip to content

Commit 6c01609

Browse files
PraveenrajmaniPraveenrajmani
andauthored
Remove unused healthz port from controller and fix port placement in DaemonSets (#1015)
- Remove port 9898 from controller container in Deployments (unused) - Move port 9898 definition to liveness-probe container in DaemonSets where it actually listens Co-authored-by: Praveenrajmani <[email protected]>
1 parent 4da1b5b commit 6c01609

File tree

4 files changed

+8
-12
lines changed

4 files changed

+8
-12
lines changed

resources/base/DaemonSet.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,9 +80,6 @@ spec:
8080
- containerPort: 30443
8181
name: readinessport
8282
protocol: TCP
83-
- containerPort: 9898
84-
name: healthz
85-
protocol: TCP
8683
- containerPort: 10443
8784
name: metrics
8885
protocol: TCP
@@ -176,6 +173,10 @@ spec:
176173
- --health-port=9898
177174
image: quay.io/minio/livenessprobe@sha256:8f3b1bec9c87a832a3fe6e8b7f165e0ff048aef7373f9764f40efee456a00321
178175
name: liveness-probe
176+
ports:
177+
- containerPort: 9898
178+
name: healthz
179+
protocol: TCP
179180
resources: {}
180181
terminationMessagePath: /var/log/driver-liveness-termination-log
181182
terminationMessagePolicy: FallbackToLogsOnError

resources/base/Deployment.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,6 @@ spec:
9191
- containerPort: 30443
9292
name: readinessport
9393
protocol: TCP
94-
- containerPort: 9898
95-
name: healthz
96-
protocol: TCP
9794
readinessProbe:
9895
failureThreshold: 5
9996
httpGet:

resources/legacy/DaemonSet.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,6 @@ spec:
7575
- containerPort: 30443
7676
name: readinessport
7777
protocol: TCP
78-
- containerPort: 9898
79-
name: healthz
80-
protocol: TCP
8178
- containerPort: 10443
8279
name: metrics
8380
protocol: TCP
@@ -127,6 +124,10 @@ spec:
127124
- --health-port=9898
128125
image: quay.io/minio/livenessprobe@sha256:8f3b1bec9c87a832a3fe6e8b7f165e0ff048aef7373f9764f40efee456a00321
129126
name: liveness-probe
127+
ports:
128+
- containerPort: 9898
129+
name: healthz
130+
protocol: TCP
130131
resources: {}
131132
terminationMessagePath: /var/log/driver-liveness-termination-log
132133
terminationMessagePolicy: FallbackToLogsOnError

resources/legacy/Deployment.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,6 @@ spec:
8888
- containerPort: 30443
8989
name: readinessport
9090
protocol: TCP
91-
- containerPort: 9898
92-
name: healthz
93-
protocol: TCP
9491
readinessProbe:
9592
failureThreshold: 5
9693
httpGet:

0 commit comments

Comments
 (0)