#Harvest getting shelf version

1 messages · Page 1 of 1 (latest)

gray yoke
#

I found this in harvest yaml :
harvest/conf/restperf/9.12.0/disk.yaml

- frus => psu: - ^^id => psu_id - ^installed => enabled # - ^location - ^part_number - ^serial_number => serial - ^psu.model => type - ^state => status - psu.power_drawn => power_drawn - psu.power_rating => power_rating

And from the API:
The response is :
"frus":[ { "type":"module", "id":0, "state":"ok", "part_number":"XXX", "serial_number":"XXX", "firmware_version":"0250", "installed":true } { "type":"psu", "id":1, "state":"ok", "part_number":"XXX", "serial_number":"XXX", "firmware_version":"0111", "installed":true, "psu":{ "model":"7D", "power_drawn":134, "power_rating":913, "crest_factor":89 } } ],

But from the metric,
I can only get the value when "type"="psu",
How can I get the type=module value?

lost steppe
#

@gray yoke The disk.yaml template for performance monitoring is designed to collect data specifically for PSU modules, as this information is required for power calculations. Are you interested in collecting only the type=module and psu data, along with their respective firmware versions?

gray yoke
#

I am interested in getting the firmware_version of the shelf,
Which is 0250 in the API response

lost steppe
#

Okay. I get 2 modules for my cluster. They look same. Could they be different?

image.png
gray yoke
#

I need both of them,
With also sn and part_number

lost steppe
#

Okay. Could you please open a feature request for this over github.

gray yoke
#

@lost steppe
I am curios that why is the default only showing psu type?
Is there any setting to result this?
Since from the API response,
There are PSU type and Module type

lost steppe
#

We only collect PSU type as they are the only ones needed for power calculation. We would need to probably make changes in custom plugin to enable other module type.

gray yoke
#

But from the disk.yaml
I don’t see any restriction setting only to collect PSU

lost steppe
gray yoke
#

Oh!!! Got it,
I’ll open a feature request for this

lost steppe
#

Thanks

gray yoke
gray yoke
#

@lost steppe If we change the plugin by deleting the conditional expressions:
&& obj.Get("type").ClonedString() != "psu"
Is this a recommendation?

lost steppe
#

We need to check. We may have to create a separate template.

gray yoke
#

@lost steppe I am thinking the workaround now is to
remove && obj.Get("type").ClonedString() != "psu",
and alter disk.yaml to get both module and psu type

lost steppe
#

Did you try this change and works for you?

gray yoke
lost steppe
#

Ok sure. Is this urgent requirement for your usecase?

gray yoke
#

Is there an estimated release time for the version that supported this feature?

lost steppe
#

It will be available in 25.02. Does that work?

image.png
gray yoke
#

Sure!
Is it with a new metric?

lost steppe
#

Thanks!

gray yoke
lost steppe
#

Should be somewhere around mid feburary. But We'll get it to you sooner via nightly builds.

gray yoke
#

Thanks so much @lost steppe

lost steppe
#

You are welcome!