Skip to content

Commit 0c561f3

Browse files
authored
Fix NPD in Deploy reconcile (#1071)
Signed-off-by: Rob Hundley <[email protected]>
1 parent 372b64f commit 0c561f3

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

internal/controller/operator/deployment.go

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -727,18 +727,24 @@ func preserveObservedFields(observed, generated *appsv1.Deployment) {
727727
if observedContainer.Name != generatedContainer.Name {
728728
continue
729729
}
730+
730731
if generatedContainer.LivenessProbe == nil {
731732
generated.Spec.Template.Spec.Containers[i].LivenessProbe = &corev1.Probe{}
732733
}
733-
generated.Spec.Template.Spec.Containers[i].LivenessProbe.FailureThreshold = observedContainer.LivenessProbe.FailureThreshold
734-
generated.Spec.Template.Spec.Containers[i].LivenessProbe.PeriodSeconds = observedContainer.LivenessProbe.PeriodSeconds
735-
generated.Spec.Template.Spec.Containers[i].LivenessProbe.SuccessThreshold = observedContainer.LivenessProbe.SuccessThreshold
734+
if observedContainer.LivenessProbe != nil {
735+
generated.Spec.Template.Spec.Containers[i].LivenessProbe.FailureThreshold = observedContainer.LivenessProbe.FailureThreshold
736+
generated.Spec.Template.Spec.Containers[i].LivenessProbe.PeriodSeconds = observedContainer.LivenessProbe.PeriodSeconds
737+
generated.Spec.Template.Spec.Containers[i].LivenessProbe.SuccessThreshold = observedContainer.LivenessProbe.SuccessThreshold
738+
}
739+
736740
if generatedContainer.ReadinessProbe == nil {
737741
generated.Spec.Template.Spec.Containers[i].ReadinessProbe = &corev1.Probe{}
738742
}
739-
generated.Spec.Template.Spec.Containers[i].ReadinessProbe.SuccessThreshold = observedContainer.ReadinessProbe.SuccessThreshold
740-
generated.Spec.Template.Spec.Containers[i].TerminationMessagePath = observedContainer.TerminationMessagePath
741-
generated.Spec.Template.Spec.Containers[i].TerminationMessagePolicy = observedContainer.TerminationMessagePolicy
743+
if observedContainer.ReadinessProbe != nil {
744+
generated.Spec.Template.Spec.Containers[i].ReadinessProbe.SuccessThreshold = observedContainer.ReadinessProbe.SuccessThreshold
745+
generated.Spec.Template.Spec.Containers[i].TerminationMessagePath = observedContainer.TerminationMessagePath
746+
generated.Spec.Template.Spec.Containers[i].TerminationMessagePolicy = observedContainer.TerminationMessagePolicy
747+
}
742748
}
743749
}
744750
for _, observedContainer := range observed.Spec.Template.Spec.InitContainers {

0 commit comments

Comments
 (0)