more logging

Author: mweibel

azure/scope/strategies/machinepool_deployments/machinepool_deployment_strategy.go

@@ -128,7 +128,7 @@ func (rollingUpdateStrategy rollingUpdateStrategy) SelectMachinesToDelete(ctx co
 				return orderRandom
 			}
 		}()
-		log                        = ctrl.LoggerFrom(ctx).V(4)
+		log                        = ctrl.LoggerFrom(ctx).V(2)
 		deleteAnnotatedMachines    = order(getDeleteAnnotatedMachines(machinesByProviderID))
 		failedMachines             = order(getFailedMachines(machinesByProviderID))
 		deletingMachines           = order(getDeletingMachines(machinesByProviderID))

azure/services/scalesets/spec.go

@@ -92,6 +92,9 @@ func (s *ScaleSetSpec) OwnerResourceName() string {
 }
 
 func (s *ScaleSetSpec) existingParameters(ctx context.Context, existing interface{}) (parameters interface{}, err error) {
+	ctx, log, done := tele.StartSpanWithLogger(ctx, "scalesets.ScaleSetSpec.existingParameters")
+	defer done()
+
 	existingVMSS, ok := existing.(armcompute.VirtualMachineScaleSet)
 	if !ok {
 		return nil, errors.Errorf("%T is not an armcompute.VirtualMachineScaleSet", existing)
@@ -131,6 +134,14 @@ func (s *ScaleSetSpec) existingParameters(ctx context.Context, existing interfac
 		return nil, nil
 	}
 
+	log.Info("updating VMSS",
+		"name", s.Name,
+		"capacity", vmss.SKU.Capacity,
+		"existingCapacity", existingInfraVMSS.Capacity,
+		"hasModelChanges", hasModelChanges,
+		"shouldPatchCustomData", s.ShouldPatchCustomData,
+	)
+
 	return vmss, nil
 }