Skip to content

Commit c26609e

Browse files
committed
Fix: Update deployment template annotations and labels in Reconcile method
1 parent 8f4ac6d commit c26609e

1 file changed

Lines changed: 20 additions & 0 deletions

File tree

internal/controller/overcommit/overcommit_controller.go

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,16 @@ func (r *OvercommitReconciler) Reconcile(ctx context.Context, req ctrl.Request)
164164
overcommitClassDeployment.Spec.Template.Spec.Containers[0].Env = updatedDeployment.Spec.Template.Spec.Containers[0].Env
165165
updated = true
166166
}
167+
// Update template annotations if they changed
168+
if !annotationsEqual(updatedDeployment.Spec.Template.Annotations, overcommitClassDeployment.Spec.Template.Annotations) {
169+
overcommitClassDeployment.Spec.Template.Annotations = updatedDeployment.Spec.Template.Annotations
170+
updated = true
171+
}
172+
// Update template labels if they changed
173+
if !labelsEqual(updatedDeployment.Spec.Template.Labels, overcommitClassDeployment.Spec.Template.Labels) {
174+
overcommitClassDeployment.Spec.Template.Labels = updatedDeployment.Spec.Template.Labels
175+
updated = true
176+
}
167177
// Only set controller reference if we actually updated something
168178
if updated {
169179
return ctrl.SetControllerReference(overcommit, overcommitClassDeployment, r.Scheme)
@@ -250,6 +260,16 @@ func (r *OvercommitReconciler) Reconcile(ctx context.Context, req ctrl.Request)
250260
validatingPodDeployment.Spec.Template.Spec.Containers[0].Env = updatedDeployment.Spec.Template.Spec.Containers[0].Env
251261
updated = true
252262
}
263+
// Update template annotations if they changed
264+
if !annotationsEqual(updatedDeployment.Spec.Template.Annotations, validatingPodDeployment.Spec.Template.Annotations) {
265+
validatingPodDeployment.Spec.Template.Annotations = updatedDeployment.Spec.Template.Annotations
266+
updated = true
267+
}
268+
// Update template labels if they changed
269+
if !labelsEqual(updatedDeployment.Spec.Template.Labels, validatingPodDeployment.Spec.Template.Labels) {
270+
validatingPodDeployment.Spec.Template.Labels = updatedDeployment.Spec.Template.Labels
271+
updated = true
272+
}
253273
// Only set controller reference if we actually updated something
254274
if updated {
255275
return ctrl.SetControllerReference(overcommit, validatingPodDeployment, r.Scheme)

0 commit comments

Comments
 (0)