TEMP: add debug logging for Healthy condition troubleshooting

Author: jacob-anders

internal/controller/metal3.io/baremetalhost_controller.go

@@ -248,6 +248,11 @@ func (r *BareMetalHostReconciler) Reconcile(ctx context.Context, request ctrl.Re
 	copy(conditionsBefore, existing)
 	computeConditions(ctx, host, prov)
 	conditionsChanged := !reflect.DeepEqual(conditionsBefore, host.GetConditions())
+	info.log.Info("conditions check",
+		"dirty", actResult.Dirty(),
+		"conditionsChanged", conditionsChanged,
+		"conditionsBefore", len(conditionsBefore),
+		"conditionsAfter", len(host.GetConditions()))
 
 	// Only save status when we're told to, otherwise we
 	// introduce an infinite loop reconciling the same object over and

pkg/provisioner/ironic/ironic.go

@@ -2002,12 +2002,16 @@ func (p *ironicProvisioner) HasPowerFailure(ctx context.Context) bool {
 // request to Ironic on every reconcile. Consider passing the node object through
 // computeConditions or caching it per reconcile cycle to avoid redundant calls.
 func (p *ironicProvisioner) GetHealth(ctx context.Context) string {
+	p.log.Info("GetHealth called", "nodeID", p.nodeID, "hasHealthAPI", p.availableFeatures.HasHealthAPI(), "maxVersion", p.availableFeatures.MaxVersion)
 	node, err := p.getNode(ctx)
 	if err != nil {
-		if !errors.Is(err, provisioner.ErrNeedsRegistration) {
-			p.log.Error(err, "ignored error while checking health status")
+		if errors.Is(err, provisioner.ErrNeedsRegistration) {
+			p.log.Info("GetHealth: node not registered yet")
+		} else {
+			p.log.Error(err, "GetHealth: getNode failed")
 		}
 		return ""
 	}
+	p.log.Info("GetHealth result", "health", node.Health, "nodeUUID", node.UUID, "provisionState", node.ProvisionState)
 	return node.Health
 }