[Merge] lp:~csurbhi/mdadm/mdadm-fix.728435 into lp:~ubuntu-core-dev/mdadm/ubuntu

Surbhi Palande surbhi.palande at canonical.com
Fri Mar 18 15:16:13 UTC 2011


Surbhi Palande has proposed merging lp:~csurbhi/mdadm/mdadm-fix.728435 into lp:~ubuntu-core-dev/mdadm/ubuntu.

Requested reviews:
  Ubuntu Core Development Team (ubuntu-core-dev)
Related bugs:
  Bug #728435 in mdadm (Ubuntu): "raid1 boot degraded mode fails"
  https://bugs.launchpad.net/ubuntu/+source/mdadm/+bug/728435

For more details, see:
https://code.launchpad.net/~csurbhi/mdadm/mdadm-fix.728435/+merge/54011

The udev rule for mdadm uses incremental mode to create an array and/or to add member disks to it. The correct way to start such a partially assembled array in degraded is to use the Incremental mode. So, adding the " mdadm --incremental --scan --run " to the mdadm/mountroot_failure().

Previously mdadm --assemble --scan --run worked because the arrays were stopped before scan assembling them.  Assemble scan mode is good for assembling arrays not created by Incremental mode, when member disks unassociated with any array are available for assembling a *new* array. So keeping the "mdadm --assemble --scan --run" for assembling arrays which were not partially created by Incremental mode. We should ideally not land in a situation where the arrays were not incrementally created. However, should we do (due to udevd failure say), then mdadm --assemble scan will never the less assemble a new array.

Please consider this change for natty.



-- 
https://code.launchpad.net/~csurbhi/mdadm/mdadm-fix.728435/+merge/54011
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~csurbhi/mdadm/mdadm-fix.728435 into lp:~ubuntu-core-dev/mdadm/ubuntu.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 3113 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20110318/96ad9dcc/attachment.diff>


More information about the Ubuntu-reviews mailing list