Merge pull request #379 from bmc-toolbox/dell-bmc-reset

joelrebel · web-flow · commit 4d960aa3d240 · 2023-12-06T13:55:15.000+01:00
providers/dell: implement the BMCResetter interface
diff --git a/providers/dell/idrac.go b/providers/dell/idrac.go
@@ -42,7 +42,10 @@ var (
 		providers.FeatureFirmwareUploadInitiateInstall,
 		providers.FeatureFirmwareTaskStatus,
 		providers.FeatureInventoryRead,
+		providers.FeatureBmcReset,
 	}
+
+	errManufacturerUnknown = errors.New("error identifying device manufacturer")
 )
 
 type Config struct {
@@ -211,7 +214,10 @@ func (c *Conn) Inventory(ctx context.Context) (device *common.Device, err error)
 	return c.redfishwrapper.Inventory(ctx, false)
 }
 
-var errManufacturerUnknown = errors.New("error identifying device manufacturer")
+// BmcReset power cycles the BMC
+func (c *Conn) BmcReset(ctx context.Context, resetType string) (ok bool, err error) {
+	return c.redfishwrapper.BMCReset(ctx, resetType)
+}
 
 // deviceManufacturer returns the device manufacturer and model attributes
 func (c *Conn) deviceManufacturer(ctx context.Context) (vendor string, err error) {

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,10 @@ var (`
`42`	`42`	`providers.FeatureFirmwareUploadInitiateInstall,`
`43`	`43`	`providers.FeatureFirmwareTaskStatus,`
`44`	`44`	`providers.FeatureInventoryRead,`
	`45`	`+ providers.FeatureBmcReset,`
`45`	`46`	`}`
	`47`	`+`
	`48`	`+ errManufacturerUnknown = errors.New("error identifying device manufacturer")`
`46`	`49`	`)`
`47`	`50`
`48`	`51`	`type Config struct {`
`@@ -211,7 +214,10 @@ func (c Conn) Inventory(ctx context.Context) (device common.Device, err error)`
`211`	`214`	`return c.redfishwrapper.Inventory(ctx, false)`
`212`	`215`	`}`
`213`	`216`
`214`		`-var errManufacturerUnknown = errors.New("error identifying device manufacturer")`
	`217`	`+// BmcReset power cycles the BMC`
	`218`	`+func (c *Conn) BmcReset(ctx context.Context, resetType string) (ok bool, err error) {`
	`219`	`+ return c.redfishwrapper.BMCReset(ctx, resetType)`
	`220`	`+}`
`215`	`221`
`216`	`222`	`// deviceManufacturer returns the device manufacturer and model attributes`
`217`	`223`	`func (c *Conn) deviceManufacturer(ctx context.Context) (vendor string, err error) {`