Hi Gris,

Just saw one thing:

 megaraid.py

Line: 121
disk_status == Disk.STATUS_UNKNOWN

should be:
disk_status = Disk.STATUS_UNKNOWN

Support for this in SMI-S, guessing ugly to impossible?

Thanks!

Regards,
Tony


On 03/05/2015 08:07 AM, Gris Ge wrote:
>  * This patch set introduced new disk status -- STATUS_FREE to indicate
>    disk could be used for pool or assign as dedicated spare disk.
> 
>  * Patch set is based on:
>     [RESEND PATCH V5 00/15] New method: volume_raid_info()
> 
> Gris Ge (6):
>   MegaRAID: Fix incorrect system status query.
>   API: New disk status DISK.STATUS_FREE/LSM_DISK_STATUS_FREE
>   lsmcli: Add support of Disk.STATUS_FREE
>   Simulator Plugin: Support Disk.STATUS_FREE
>   ONTAP Plugin: Add support of Disk.STATUS_FREE
>   MegaRAID Plugin: Add support of Disk.STATUS_FREE
> 
>  .../include/libstoragemgmt/libstoragemgmt_types.h  | 10 +++
>  plugin/megaraid/megaraid.py                        | 90 
> ++++++++++++----------
>  plugin/ontap/ontap.py                              | 10 +++
>  plugin/sim/simarray.py                             |  9 ++-
>  python_binding/lsm/_data.py                        |  8 ++
>  tools/lsmcli/data_display.py                       |  1 +
>  6 files changed, 87 insertions(+), 41 deletions(-)
> 


------------------------------------------------------------------------------
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