Skip to content

Commit 0c0a644

Browse files
Merge pull request #291 from mrogers950/46deploy
Fix deployment on OCP 4.6
2 parents 19c7965 + fb3d150 commit 0c0a644

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

cmd/manager/operator.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -250,6 +250,14 @@ func ensureMetricsServiceAndSecret(ctx context.Context, kClient *kubernetes.Clie
250250
if !reflect.DeepEqual(curService.Spec, newService.Spec) {
251251
serviceCopy := curService.DeepCopy()
252252
serviceCopy.Spec = newService.Spec
253+
254+
// OCP-4.6 only - Retain ClusterIP from the current service in case we overwrite it when copying the updated
255+
// service. Avoids "Error creating metrics service/secret","error":"Service \"metrics\" is invalid: spec.clusterIP:
256+
// Invalid value: \"\": field is immutable","stacktrace"...
257+
if len(serviceCopy.Spec.ClusterIP) == 0 {
258+
serviceCopy.Spec.ClusterIP = curService.Spec.ClusterIP
259+
}
260+
253261
updatedService, updateErr := kClient.CoreV1().Services(ns).Update(ctx, serviceCopy, metav1.UpdateOptions{})
254262
if updateErr != nil {
255263
return nil, updateErr

0 commit comments

Comments
 (0)