Check power state before issuing power off for ipmi:

Some implementations of ipmi, Dell for example, will
error out if a machine is already off and an ipmitool
power off command is issued.
Error is "Command not supported in present state".
Checking the power state before issuing the power off
command is also done in other ipmitool commands.

Signed-off-by: Jacob Weinstock <[email protected]>

Author: jacobweinstock

internal/ipmi/ipmi.go

@@ -247,6 +247,9 @@ func (i *Ipmi) PowerOnForce(ctx context.Context) (status bool, err error) {
 
 // PowerOff power off the machine via bmc
 func (i *Ipmi) PowerOff(ctx context.Context) (status bool, err error) {
+	if on, err := i.IsOn(ctx); err == nil && !on {
+		return true, nil
+	}
 	output, err := i.run(ctx, []string{"chassis", "power", "off"})
 	if strings.Contains(output, "Chassis Power Control: Down/Off") {
 		return true, err