From ca08fec1570bfdb3111d0303fb52002117a1da76 Mon Sep 17 00:00:00 2001
From: Dmitry Ukov <dukov@mirantis.com>
Date: Sat, 16 Nov 2019 10:50:03 -0600
Subject: [PATCH] Fix redfish reboot logic

Power On commnad was not sent due to wrong logic around
ScreenRedfishError function.

Change-Id: Iec39c49ddf7c4df8fed16fdfe06828513692abad
---
 pkg/remote/redfish/utils.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pkg/remote/redfish/utils.go b/pkg/remote/redfish/utils.go
index 73d04bfb5..e3c36393f 100644
--- a/pkg/remote/redfish/utils.go
+++ b/pkg/remote/redfish/utils.go
@@ -89,8 +89,8 @@ func RebootSystem(ctx context.Context, api redfishApi.RedfishAPI, systemId strin
 	resetReq := redfishClient.ResetRequestBody{}
 	resetReq.ResetType = redfishClient.RESETTYPE_FORCE_OFF
 	_, httpResp, err := api.ResetSystem(ctx, systemId, resetReq)
-	if err != nil {
-		return ScreenRedfishError(httpResp, err)
+	if err = ScreenRedfishError(httpResp, err); err != nil {
+		return err
 	}
 
 	time.Sleep(SystemRebootDelay)