From 790508a1fc8c5253f13f6cf04ede0b84bbf8e57c Mon Sep 17 00:00:00 2001 From: Oded Viner Date: Tue, 17 Dec 2024 19:02:16 +0200 Subject: [PATCH] operator: improve operator error logging for ok-to-stop failures this PR improves the error logging when ok-to-stop requests fail in the Rook operator. Instead of only showing a generic exit status Signed-off-by: Oded Viner (cherry picked from commit 3d7f5cff65be12cf016b20a15806abb817dafce6) --- pkg/daemon/ceph/client/upgrade.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/daemon/ceph/client/upgrade.go b/pkg/daemon/ceph/client/upgrade.go index 0b9ffcc21..8de07b5d1 100644 --- a/pkg/daemon/ceph/client/upgrade.go +++ b/pkg/daemon/ceph/client/upgrade.go @@ -180,7 +180,7 @@ func okToStopDaemon(context *clusterd.Context, clusterInfo *ClusterInfo, deploym args := []string{daemonType, "ok-to-stop", daemonName} buf, err := NewCephCommand(context, clusterInfo, args).Run() if err != nil { - return errors.Wrapf(err, "deployment %s cannot be stopped", deployment) + return errors.Wrapf(err, "deployment %s cannot be stopped. %s", deployment, string(buf)) } output := string(buf) logger.Debugf("deployment %s is ok to be updated. %s", deployment, output)