jenningsloy318 / redfish_exporter

exporter to get metrics from redfish based hardware such as lenovo/dell/superc servers
Apache License 2.0
70 stars 61 forks source link

feat: add simple storage device #49

Closed mahuihuang closed 2 years ago

mahuihuang commented 2 years ago
  1. This PR adds two metrics system_simple_storage_device_state and system_simple_storage_device_health_state.
  2. Because there is no deviceID field of the Device, I upgraded gofish to use the Name as a unique identifier
  3. I used Supermicro BMC to do the test, information is as follows: Firmware Revision : 1.76.39 Firmware Build Time : 10/13/2021 BIOS Version : 3.0b BIOS Build Time : 03/04/2019 Redfish Version : 1.0.1 CPLD Version : 02.b4.aa
jenningsloy318 commented 2 years ago

Hi mahuihuang, Did you have any tests? and can you still add which device/server are supported?

mahuihuang commented 2 years ago

Hi mahuihuang, Did you have any tests? and can you still add which device/server are supported?

Thanks for your reply. I have tested on an OEM server named Enginetech(You may not be able to access this site due to the Great Firewall.), and the test results showed no problems. Its model is EG250-G20, and it uses Supermicro motherboard.At the same time, I also used a Huawei server for testing. It does not have SimpleStorage, but the program is still normal.

jenningsloy318 commented 2 years ago

Hi mahuihuang, thanks for your contribution, can you please modify the PR, and also update tested/support server ? it will help others to figure out it ?

mahuihuang commented 2 years ago

Hi jenningsloy318, Thanks for your reminding, I have updated the supported server.