On Thu, Mar 05, 2015 at 02:54:46PM -0600, Tony Asleson wrote:
> Hi Gris,
Hi Tony,
> 
> Just saw one thing:
> 
>  megaraid.py
> 
> Line: 121
> disk_status == Disk.STATUS_UNKNOWN
> 
> should be:
> disk_status = Disk.STATUS_UNKNOWN
Thanks for the detailed review.
> 
> Support for this in SMI-S, guessing ugly to impossible?
The official way is ugly by using 'Extent Composition Subprofile'
profile to walk through all StorageExtents recursively.

But we do have very good vendor specific way:
 * EMC, they have 'EMCInUse' property in CIM_DiskDrive.
 * LSI MegaRAID, they have 'StoragePool_InstanceID' in primordial
   CIM_StorageExtent(directly associated with CIM_DiskDrive).

I emailed to SNIA for this issue on 14 May 2014, only EMC replied with
their way, seems other vendor just ignored this request.

Let me add support for these two products in V2 patch.

Best regards.
> 
> Thanks!
> 
> Regards,
> Tony


-- 
Gris Ge

Attachment: signature.asc
Description: Digital signature

------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the 
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Libstoragemgmt-devel mailing list
Libstoragemgmt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libstoragemgmt-devel

Reply via email to