[lvm-devel] [PATCH] Relax exit codes for policy-driven lvconvert --repair.

[Petr Rockai]

Hi again,

Petr Rockai <prockai at redhat.com> writes:
> due to the interpretation of "replace" policies as "replace if possible,
> downconvert otherwise", we should treat successful downconversion as a
> success in these cases, even if we were unable to replace the missing
> devices with new ones.
>
> The attached patch also makes the "nothing to repair" condition a
> non-error (with and without --use-policies... it's arguably not really
> an error). This would hopefully address RHBZ 552723, if it is indeed the
> case (as it seems) that the problem there is that the missing devices
> have already returned by the time that lvconvert --repair kicks in (and
> therefore it does not detect anything to repair).

an amended patch that also ensures that MISSING_PV removal is done even
if device replacement fails is attached, and an interdiff with only the
new changes.

Yours,
   Petr.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: lvconvert-dont-skip-remove_missing_empty_pv.diff
Type: text/x-diff
Size: 1741 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/lvm-devel/attachments/20100107/40fc3df7/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lvconvert-relax-failures-with-policy-2.diff
Type: text/x-diff
Size: 6577 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/lvm-devel/attachments/20100107/40fc3df7/attachment-0001.bin>