Skip to content

Commit 3b9787e

Browse files
committed
Improve logging
1 parent a70eef4 commit 3b9787e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

controllers/karpentermachinepool_controller.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ func (r *KarpenterMachinePoolReconciler) Reconcile(ctx context.Context, req reco
179179
karpenterMachinePool.Status.Ready = true
180180

181181
if !karpenterMachinePoolCopy.Status.Ready || karpenterMachinePoolCopy.Status.Replicas != numberOfNodeClaims {
182-
logger.Info("Found NodeClaims in workload cluster, patching KarpenterMachinePool.status field", "numberOfNodeClaims", numberOfNodeClaims)
182+
logger.Info("Found NodeClaims in workload cluster, patching KarpenterMachinePool.status field", "numberOfNodeClaims", numberOfNodeClaims, "currentNumberOfReplicas", karpenterMachinePoolCopy.Status.Replicas)
183183
if err := r.client.Status().Patch(ctx, karpenterMachinePool, client.MergeFrom(karpenterMachinePoolCopy), client.FieldOwner("karpentermachinepool-controller")); err != nil {
184184
logger.Error(err, "failed to patch karpenterMachinePool.status.Replicas")
185185
return reconcile.Result{}, err
@@ -245,6 +245,7 @@ func (r *KarpenterMachinePoolReconciler) reconcileDelete(ctx context.Context, lo
245245
karpenterMachinePoolCopy := karpenterMachinePool.DeepCopy()
246246

247247
controllerutil.RemoveFinalizer(karpenterMachinePool, KarpenterFinalizer)
248+
logger.Info("Removing finalizer", "finalizer", KarpenterFinalizer)
248249
if err := r.client.Patch(ctx, karpenterMachinePool, client.MergeFrom(karpenterMachinePoolCopy)); err != nil {
249250
logger.Error(err, "failed to remove finalizer", "finalizer", KarpenterFinalizer)
250251
return reconcile.Result{}, err

0 commit comments

Comments
 (0)