From 1bb44a8e78e4b313e3aefbf4b7060b8217918de7 Mon Sep 17 00:00:00 2001 From: Dave Chen Date: Thu, 1 Sep 2022 16:59:11 +0800 Subject: [PATCH] kubeadm: too early return for reset with dry-run or force mode Signed-off-by: Dave Chen --- cmd/kubeadm/app/cmd/phases/reset/preflight.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/cmd/kubeadm/app/cmd/phases/reset/preflight.go b/cmd/kubeadm/app/cmd/phases/reset/preflight.go index 6354db4f03c..041664bfc81 100644 --- a/cmd/kubeadm/app/cmd/phases/reset/preflight.go +++ b/cmd/kubeadm/app/cmd/phases/reset/preflight.go @@ -53,7 +53,9 @@ func runPreflight(c workflow.RunData) error { if !r.ForceReset() && !r.DryRun() { klog.Warning("[reset] WARNING: Changes made to this host by 'kubeadm init' or 'kubeadm join' will be reverted.") - return util.InteractivelyConfirmAction("reset", "Are you sure you want to proceed?", r.InputReader()) + if err := util.InteractivelyConfirmAction("reset", "Are you sure you want to proceed?", r.InputReader()); err != nil { + return err + } } fmt.Println("[preflight] Running pre-flight checks")