trunet / puppet-storcli

Puppet module to generate facts with types and providers to manage LSI MegaRAID controllers
Apache License 2.0
3 stars 2 forks source link

0.4.5 fails with perccli #17

Closed avk999 closed 1 year ago

avk999 commented 1 year ago

if perccli is used (vendor is Dell and perccli64 exists) facter/megaraid.rb tries to call it with nolog which perccli doesn't support, so facter fails.

avk999 commented 1 year ago

perccli:

Storage Command Line Tool  Ver 007.0318.0000.0000 Jan 03, 2018
jcpunk commented 1 year ago

Can you test the linked pull request and verify it is working as you'd expect?

jcpunk commented 1 year ago

Hmmm, I'm showing the current version is : PercCli SAS Customization Utility Ver 007.2313.0000.0000 Mar 07, 2023

from: https://www.dell.com/support/home/en-lt/drivers/driversdetails?driverid=tdghn

And it shows a nolog option is accepted...

jcpunk commented 1 year ago

I've published 1.0.0 with greater clarification on the minimum required versions of storcli and perccli.

avk999 commented 1 year ago

Thank you! Right now we are not using this module, I'll get back when we'll be re-enabling it.

avk999 commented 1 year ago

(closing)