megaclisas-status shows duplicate info

[helix84@…]

First of all, thank you for providing this great site. It really made installing the RAID controller program painless on my Debian Squeeze.

On Dell R710 with H700 I installed megaclisas-status. It works correctly but it lists all my disks twice in "Disks informations":

# megaclisas-status
-- Controller informations --
-- ID | Model
c0 | PERC H700 Integrated

-- Arrays informations --
-- ID | Type | Size | Status | InProgress?
c0u0 | RAID1 | 136G | Optimal | None
c0u1 | RAID5 | 5456G | Optimal | Background Initialization: Completed 11%, Taken 127 min.

-- Disks informations
-- ID | Model | Status
c0u0p0 | SEAGATE ST3146855SS S5273LN507WK | Online
c0u1p1 | SEAGATE ST3146855SS S5273LN507P2 | Online
c0u1p0 | SEAGATE ST32000444SS KS659WM135ZP | Online
c0u1p1 | SEAGATE ST32000444SS KS659WM0Z5TR | Online
c0u1p2 | SEAGATE ST32000444SS KS659WM0Z5ZF | Online
c0u1p3 | SEAGATE ST32000444SS KS659WM0WPKD | Online
c0u0p0 | SEAGATE ST3146855SS S5273LN507WK | Online
c0u1p1 | SEAGATE ST3146855SS S5273LN507P2 | Online
c0u1p0 | SEAGATE ST32000444SS KS659WM135ZP | Online
c0u1p1 | SEAGATE ST32000444SS KS659WM0Z5TR | Online
c0u1p2 | SEAGATE ST32000444SS KS659WM0Z5ZF | Online
c0u1p3 | SEAGATE ST32000444SS KS659WM0WPKD | Online

I believe the reason is that on line 152:

cmd = 'megacli -LdPdInfo? -a'+str(controllerid)+' -NoLog?'

you need to pass the _adapter number_ as controllerid but you pass the _virtual disk number_ instead. This fix displays correct output on my machine:

--- /usr/sbin/megaclisas-status 2009-07-27 11:34:13.000000000 +0200
+++ megaclisas-status 2010-08-13 18:13:00.000000000 +0200
@@ -40,7 +40,7 @@

def returnArrayNumber(output):

i = 0
for line in output:

• if re.match(r'^{Virtual Disk.*$',line.strip()):}

+ if re.match(r'Adapter .*$',line.strip()):

i += 1

return i

You should also change 'informations' to 'information' - that's the correct English plural.

One more favour I'd like to ask you is to provide a key for your Debian archive if possible. Without it, I get warnings from apt:
# apt-get install megaclisas-status
...
WARNING: The following packages cannot be authenticated!

megaclisas-status

[gandalf]

Fixed with megaclisas-status 0.9.