-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PLUGIN_BUG(2): _parse_hpssacli_output(): Found duplicate line HPE E208i-a SR Gen10 #337
Comments
I think the exact command run by hpsa is:
|
Strange, we definitely have this running across various configurations of Gen10 controllers and haven't seen issues. I'll take a look as time allows and see what I can find. The message about needing to use a new version of ssacli is a new one to me, and may be worth trying out, just to see if that's causing the variation in output (needs a fix to deal with it either way, but could alleviate your problems in the short-term). Try going out and grabbing the newest version of ssacli (from wherever you're used to getting it from), and see if that helps. |
I updated ssacli to the version 3.25. |
Great. Do we need to support old ssacli also or just close this issue? |
My preference would be do document the minimum supported version of ssacli and close issue. |
@joehandzik Which version of ssacli would you suggested to put in the man page of hpsa plugin? |
I personnaly would prefer that LSM skips an unknown raid card, and continue to the other ones. In my case ssacli returns an error when I try to access the E208i-a, but as I use only the P408i-p, in command line, there is no issue.
|
Silently skip unknown hpsa might waste more time of user when they actually need that card managed by LSM. I guess suggest user to upgrade their ssacli in PLUGIN_BUG error message might be better. |
Suggest user to upgrade their ssacli tool when got parsing error. #337 Signed-off-by: Gris Ge <fge@redhat.com>
Included suggestion on ssacli tool upgrade when parsing error in commit 00e0048 |
Seems to be related to indentation on new Gen10 servers
below the output of ssacli:
The text was updated successfully, but these errors were encountered: