Closed aj-gh closed 10 months ago
Hi,
That's interesting. Would you mind creating a mock-up and sending it over? Then I could implement way quicker.
Hi @aj-gh,
I would be really interested in adding support for this.
But It's quite impossible without a mockup. Also would need to somehow filter out the duplicate entries.
Hi @aj-gh,
I just pushed a change to the next-release
branch. Would you be able to check it out and test it?
closed issue due to no response
The current version reads
/SmartStorage
on HPE servers which usually works fine but seems to be problematic when NVMe are present in a server as these are not "behind" the Smart Array controller, thus not present in/SmartStorage
. When patching it to read/Storage
instead of/SmartStorage
- similar to e4e47ea where this is triggered when no Smart Array is found/no results are returned - all devices are properly detected but it looks like there are some additional health checks done for/SmartStorage
such as battery health (which is read from chassis status and not/SmartStorage
) and enclosures that are then no longer present. I can make it read both paths but then I have some duplicates. It's also interesting that both NVMe seem count as a single controller; maybe because of the same name?Maybe it would make sense to read both paths and only process those devices present in
/Storage
that were not previously handled in/SmartStorage
? Or as a very quick workaround a switch to force it to read/Storage
even when/SmartStorage
returned results.Thanks for this very handy tool!
Current version:
Patched to read
/Storage
:Full inventory with
/Storage
: