Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Sensors data #238

Open
wants to merge 9 commits into
base: v1
Choose a base branch
from
Open

[WIP] Sensors data #238

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
d0f1e3c
devices/sensors: define structs for sensor data
joelrebel Jul 21, 2021
ba6a2cd
bmc/sensors: define methods and interface getters
joelrebel Jul 21, 2021
2b5a0ee
providers/redfish: declare sensor features supported, consts
joelrebel Jul 21, 2021
71c8e9c
redfish/sensors: Add sensors interface implementations
joelrebel Jul 21, 2021
443515c
client: define sensor passthrough methods
joelrebel Jul 21, 2021
c136613
examples/sensors: add example for power draw, temperature, fan speeds…
joelrebel Jul 21, 2021
1bc9af0
devices/sensors: include Name field
joelrebel Jul 21, 2021
000f06f
providers/redfish: return ErrLoginFailed on login failure
joelrebel Jul 21, 2021
3c3eec6
providers/redfish: fix features typo
joelrebel Jul 27, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions providers/providers.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,4 +29,12 @@ const (
FeatureBmcFirmwareUpdate registrar.Feature = "bmcfirwareupdate"
// FeatureBiosFirmwareUpdate means an implementation that updates the BIOS firmware
FeatureBiosFirmwareUpdate registrar.Feature = "biosfirwareupdate"
// FeaturePowersensors indicates an implementation that returns Power sensor information
FeaturePowersensors registrar.Feature = "powersensors"
// FeatureTemperatureSensors indicates an implementation that returns Temperature sensor information
FeatureTemperatureSensors registrar.Feature = "temperaturesensors"
// FeatureFanSensors indicates an implementation that returns Fan sensor information
FeatureFanSensors registrar.Feature = "fansensors"
// FeatureChassisHealth indicates an implementation that returns ChassisHealth information
FeatureChassisHealth registrar.Feature = "chassishealth"
)
30 changes: 24 additions & 6 deletions providers/redfish/redfish.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,17 +26,35 @@ var (
Features = registrar.Features{
providers.FeaturePowerSet,
providers.FeaturePowerState,
providers.FeaturePowersensors,
joelrebel marked this conversation as resolved.
Show resolved Hide resolved
providers.FeatureTemperatureSensors,
providers.FeatureFanSensors,
providers.FeatureChassisHealth,
}

// Supported Chassis Odata IDs
chassisOdataIdURLs = []string{
// Dells
"/redfish/v1/Chassis/System.Embedded.1",
// Supermicro
"/redfish/v1/Chassis/1",
// MegaRAC
"/redfish/v1/Chassis/Self",
}

ErrRedfishChassisOdataID = errors.New("no compatible chassis Odata IDs identified")
ErrRedfishServiceNil = errors.New("redfish connection returned a nil redfish Service object")
)

// Conn details for redfish client
type Conn struct {
Host string
Port string
User string
Pass string
conn *gofish.APIClient
Log logr.Logger
Host string
Port string
User string
Pass string
Vendor string
conn *gofish.APIClient
Log logr.Logger
}

// Open a connection to a BMC via redfish
Expand Down