Review Request 43: Patch16: Ignore MD RAIDs which are not active.

[Stephen Gallagher]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/43/
-----------------------------------------------------------

Review request for OpenLMI Developers.


Repository: openlmi-storage


Description
-------

Ignore MD RAIDs which are not active.

Now that Blivet does not start MD RAIDs on init, we can encounter some
MDRAIDArrayDevices with status=False, which tells that the RAID
is not running. We should not report these devices (at least now).


Diffs
-----

  src/openlmi/storage/LMI_MDRAIDBasedOn.py e94478af340ad234f7621a61d77182cca910a0d6 
  src/openlmi/storage/LMI_MDRAIDStorageExtent.py ce783e8eb97c9a56f2785d677eec7c28c733740c


Diff: http://reviewboard-openlmi.rhcloud.com/r/43/diff/


Testing
-------


Thanks,

Jan Safranek

[Stephen Gallagher]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/43/#review48
-----------------------------------------------------------

Ship it!


Ship It!

- Radek Novacek


On March 22, 2013, 5:41 p.m., Jan Safranek wrote:
>  
>  -----------------------------------------------------------
>  This is an automatically generated e-mail. To reply, visit:
>  http://reviewboard-openlmi.rhcloud.com/r/43/
>  -----------------------------------------------------------
>  
>  (Updated March 22, 2013, 5:41 p.m.)
>  
>  
>  Review request for OpenLMI Developers.
>  
>  
>  Repository: openlmi-storage
>  
>  
>  Description
>  -------
>  
>  Ignore MD RAIDs which are not active.
>  
>  Now that Blivet does not start MD RAIDs on init, we can encounter some
>  MDRAIDArrayDevices with status=False, which tells that the RAID
>  is not running. We should not report these devices (at least now).
>  
>  
>  Diffs
>  -----
>  
>    src/openlmi/storage/LMI_MDRAIDBasedOn.py e94478af340ad234f7621a61d77182cca910a0d6 
>    src/openlmi/storage/LMI_MDRAIDStorageExtent.py ce783e8eb97c9a56f2785d677eec7c28c733740c
>
>  
>  Diff: http://reviewboard-openlmi.rhcloud.com/r/43/diff/
>  
>  
>  Testing
>  -------
>  
>  
>  Thanks,
>  
>  Jan Safranek
>  
>

[Stephen Gallagher]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/43/
-----------------------------------------------------------

(Updated March 26, 2013, 3:28 p.m.)


Status
------

This change has been marked as submitted.


Review request for OpenLMI Developers.


Repository: openlmi-storage


Description
-------

Ignore MD RAIDs which are not active.

Now that Blivet does not start MD RAIDs on init, we can encounter some
MDRAIDArrayDevices with status=False, which tells that the RAID
is not running. We should not report these devices (at least now).


Diffs
-----

  src/openlmi/storage/LMI_MDRAIDBasedOn.py e94478af340ad234f7621a61d77182cca910a0d6 
  src/openlmi/storage/LMI_MDRAIDStorageExtent.py ce783e8eb97c9a56f2785d677eec7c28c733740c


Diff: http://reviewboard-openlmi.rhcloud.com/r/43/diff/


Testing
-------


Thanks,

Jan Safranek