Five PRs in one: Update to Python 3, rely on PATH, rename script, fix a typo, add gerdriesselmann's fix from issue #2

[hacklschorsch]

Thank you for your script!

I especially like that it comes w/o heavy dependencies and is easy to follow.

For our setup I had to change a couple of things; Please see this PR if you find anything of interest.

Merge at will or please drop me a line if you'd like me to turn this into distinct PRs.

Cheers!

[hacklschorsch]

Added a fix for a metric missing from our MegaCli64 output.

[hbokh]

The initial script was not working, but this PR did! 👍🏻
Only changed the shebang into #!/usr/bin/env python3 and MegaCli64 into megacli (since that is in $PATH).

[stefangweichinger]

nice PR. Although I get no separate temperatures per slot, it seems (I can't select slots to display per disk):

# TYPE megacli_pd_temperature gauge
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="celsius"} 21
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="barbarians"} 69.80
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="celsius"} 21
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="barbarians"} 69.80
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="celsius"} 21
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="barbarians"} 69.80
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="celsius"} 21
megacli_pd_temperature{adapter="None", enclosure="None", slot="None", type="barbarians"} 69.80

Any idea? I can provide raw data, if helpful.

megacli_controller{adapter="None", product_name="PRAID EP420i"} 1

[hacklschorsch]

nice PR.

Thanks @stefangweichinger !

I am afraid nothing in this PR has anything to do with the issues you are seeing.
Maybe open a new issue or new PR?

We have a couple of different MegaRAID controllers, and annoyingly MegaCli64 outputs different stats depending on the HW - so this same script throws exceptions on one controller and runs fine on another.

[stefangweichinger]

@hacklschorsch I am getting lost a bit between megacli, storcli, smartctl ... basically I want to read and monitor the drive temperatures. megaclisas-status shows them for example, but so far I couldn't find a working exporter doing that.

Yes, I might create a new PR out of #3 (comment)
Thanks.

[hacklschorsch]

I agree this is much harder than it should be.

[stefangweichinger]

Getting the drive-temperatures via smartmon.py now.
Controller temperature is not returned by storcli.py for me.