From d4f939ae05ae64591d562a2990fab648e8abe4c8 Mon Sep 17 00:00:00 2001 From: Michael Gugino Date: Thu, 18 Apr 2019 08:07:19 -0400 Subject: [PATCH] Clarify log message for machine-controller isDeleteAllowed Users have noted that this log message is confusing and does not lead to a clear path forward when deleting this machine. This commit makes it clear that the machine will not be deleted. --- pkg/controller/machine/controller.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/controller/machine/controller.go b/pkg/controller/machine/controller.go index da77d2686f13..d8fba73f8aaa 100644 --- a/pkg/controller/machine/controller.go +++ b/pkg/controller/machine/controller.go @@ -163,7 +163,7 @@ func (r *ReconcileMachine) Reconcile(request reconcile.Request) (reconcile.Resul } if !r.isDeleteAllowed(m) { - klog.Infof("Skipping reconciling of machine %q", name) + klog.Infof("Deleting machine hosting this controller is not allowed. Skipping reconciliation of machine %q", name) return reconcile.Result{}, nil }