providers/redfish: return ErrLoginFailed on login failure

bmc-toolbox · Jul 21, 2021 · 000f06f · 000f06f
1 parent 1bc9af0
commit 000f06f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/bmc/connection.go b/bmc/connection.go
@@ -8,6 +8,10 @@ import (
 	"github.com/hashicorp/go-multierror"
 )
 
+var (
+	ErrOpenConnection = errors.New("error opening connection")
+)
+
 // Opener interface for opening a connection to a BMC
 type Opener interface {
 	Open(ctx context.Context) error

diff --git a/providers/redfish/redfish.go b/providers/redfish/redfish.go
@@ -6,7 +6,9 @@ import (
 	"strings"
 	"time"
 
+	bmcErrs "github.com/bmc-toolbox/bmclib/errors"
 	"github.com/bmc-toolbox/bmclib/providers"
+
 	"github.com/go-logr/logr"
 	"github.com/jacobweinstock/registrar"
 	"github.com/pkg/errors"
@@ -68,7 +70,7 @@ func (c *Conn) Open(ctx context.Context) (err error) {
 
 	c.conn, err = gofish.ConnectContext(ctx, config)
 	if err != nil {
-		return err
+		return errors.Wrap(bmcErrs.ErrLoginFailed, err.Error())
 	}
 	return nil
 }