Ticket #2 (closed defect: fixed)
megaclisas-status 0.3 Python TypeError
| Reported by: | mhjacks@… | Owned by: | gandalf |
|---|---|---|---|
| Priority: | major | Milestone: | 1.0.0 |
| Component: | wiki | Version: | |
| Keywords: | Cc: |
Description
When running on my PERC/5Di array, throws a type error converting to float. The error is reported on line 124, but the actual problem is on line 55. The strip expression is looking for "MB" to be the unit, while in my case the line is:
Size:136.25 GB
I suggest the attached patch, essentially trusting the output of the command line tool for units.
Attachments
Change History
Changed 3 years ago by mhjacks@…
-
attachment
size-patch.diff
added
comment:2 Changed 3 years ago by gandalf
Hello,
I'd rather make another fix because some people may rely on megaclisas-status output.
Here is a new patch. Could you please try it and tell me if it works ? I can't do it by myself because all megacli gives size as megabytes on all my cards.
Thanks in advance!
Regards, Adam.
PS: Patch attached to ticket #2: http://hwraid.le-vert.net/ticket/2
Changed 3 years ago by gandalf
-
attachment
megaclisas-status.diff
added
New patch that doesn't modify output style
comment:3 Changed 3 years ago by mhjacks@…
It works great for me. I tested it on 2 PERC5/Di systems, one with two arrays and one with one.
comment:4 Changed 3 years ago by anonymous
Okay, great !
I'll update the package with my patch in next days :)
Could you please check the megaraid wiki pages and create a new ticket with your "lspci -nn" if it's not already in the wiki ?
Thanks in advance,
Regards, Adam.
comment:5 follow-up: ↓ 6 Changed 3 years ago by mhjacks@…
Yes, I can do that. It will Monday night at least before I can post it.
Also, I should say: this site is an incredible resource. Thanks for creating and maintaining it!
comment:6 in reply to: ↑ 5 Changed 3 years ago by gandalf
Replying to mhjacks@…:
Yes, I can do that. It will Monday night at least before I can post it.
Also, I should say: this site is an incredible resource. Thanks for creating and maintaining it!
Hello,
I have updated megaclisas-status package to version 0.4.
Could you please update & upgrade and tell me if it works as expected ?
Thanks in advance,
Regards, Adam.
comment:7 Changed 3 years ago by mhjacks@…
Will check but I expect it will work fine.
I checked my cards and they're both standard Dell PERC 5/Di's - already on the wiki page.
comment:8 Changed 3 years ago by anonymous
BTW - you may be pleased to hear that your aacraid-status script works on FreeBSD 7 and 6, too, provided there are symlinks to provide /usr/bin/python. They have arcconf in their ports system.
comment:9 Changed 3 years ago by mhjacks@…
Version 0.4 of the megaclisas-status works fine. Tested on both 32 and 64 bit Jaunty.
comment:10 Changed 3 years ago by gandalf
- Status changed from accepted to closed
- Resolution set to fixed


patch to megaclisas-status