Bug 491680 - Documentation update is needed - LVM2 mirror: failed log device isn't replaced.
Summary: Documentation update is needed - LVM2 mirror: failed log device isn't replaced.
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: lvm2
Version: 4.8
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: Milan Broz
QA Contact: Cluster QE
URL:
Whiteboard:
Depends On: 186437
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-03-23 15:53 UTC by Tom Coughlan
Modified: 2018-10-20 00:48 UTC (History)
17 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of: 186437
Environment:
Last Closed: 2009-05-18 20:10:49 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2009:0967 0 normal SHIPPED_LIVE lvm2 bug-fix and enhancement update 2009-05-18 13:33:39 UTC

Comment 3 Alasdair Kergon 2009-03-24 12:45:06 UTC
Tom writes:
> Okay, we are in agreement that the code in 4.8 will not change. The remaining
> task is to change the comments in lvm.conf so that they are not misleading. 

The current text is:

==============

    # In the event of a failure, the specified policy will be used to
    # determine what happens:
    #
    # "remove" - Simply remove the faulty device and run without it.  If
    #            the log device fails, the mirror would convert to using
    #            an in-memory log.  This means the mirror will not
    #            remember its sync status across crashes/reboots and
    #            the entire mirror will be re-synced.  If a
    #            mirror image fails, the mirror will convert to a
    #            non-mirrored device if there is only one remaining good
    #            copy.
    #
    # "allocate" - Remove the faulty device and try to allocate space on
    #            a new device to be a replacement for the failed device.
    #            Using this policy for the log is fast and maintains the
    #            ability to remember sync state through crashes/reboots.
    #            Using this policy for a mirror device is slow, as it
    #            requires the mirror to resynchronize the devices, but it
    #            will preserve the mirror characteristic of the device.
    #            This policy acts like "remove" if no suitable device and
    #            space can be allocated for the replacement.
    #            Currently this is not implemented properly and behaves
    #            similarly to:
    #
    # "allocate_anywhere" - Operates like "allocate", but it does not
    #            require that the new space being allocated be on a
    #            device is not part of the mirror.  For a log device
    #            failure, this could mean that the log is allocated on
    #            the same device as a mirror device.  For a mirror
    #            device, this could mean that the mirror device is
    #            allocated on the same device as another mirror device.
    #            This policy would not be wise for mirror devices
    #            because it would break the redundant nature of the
    #            mirror.  This policy acts like "remove" if no suitable
    #            device and space can be allocated for the replacement.

    mirror_log_fault_policy = "allocate"
    mirror_device_fault_policy = "remove"
}

==================

We propose to change:

     #            This policy acts like "remove" if no suitable device and
     #            space can be allocated for the replacement.
-    #            Currently this is not implemented properly and behaves
-    #            similarly to:
+    #            Currently (version 2.02.NN) this is not implemented properly
+    #            and behaves in an identical way to "remove" above.
     #
     # "allocate_anywhere" - Operates like "allocate", but it does not
     #            require that the new space being allocated be on a


Nick: is this enough?

Comment 6 Milan Broz 2009-03-24 21:19:57 UTC
Fixed in lvm2-2.02.42-5.el4

Comment 9 Jun'ichi NOMURA 2009-03-26 02:32:22 UTC
Thanks for working on to fix this.

(In reply to comment #3)
> We propose to change:
> 
>      #            This policy acts like "remove" if no suitable device and
>      #            space can be allocated for the replacement.
> -    #            Currently this is not implemented properly and behaves
> -    #            similarly to:
> +    #            Currently (version 2.02.NN) this is not implemented properly
> +    #            and behaves in an identical way to "remove" above.
>      #
>      # "allocate_anywhere" - Operates like "allocate", but it does not
>      #            require that the new space being allocated be on a
> 
> Nick: is this enough?  

I think either the same note be added also in "allocate_anywhere" section
or the note be added in the general part of the mirror policy description
(i.e. in the first paragraph of the comment block).

Also, given that the old config file may likely live after updates,
it might be better to add such description in 'lvm.conf(5)' man page
rather than just in the default lvm.conf.

Comment 11 errata-xmlrpc 2009-05-18 20:10:49 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2009-0967.html


Note You need to log in before you can comment on or make changes to this bug.