All of lore.kernel.org
 help / color / mirror / Atom feed
From: Brassow Jonathan <jbrassow@redhat.com>
To: lvm-devel@redhat.com
Subject: master - RAID: Make "raid1" the default mirror segment type
Date: Wed, 7 Aug 2013 11:42:44 -0500	[thread overview]
Message-ID: <82E23FFA-F01A-4859-BDDD-88587A24E81B@redhat.com> (raw)
In-Reply-To: <52025B3C.9060309@redhat.com>


On Aug 7, 2013, at 9:35 AM, Peter Rajnoha wrote:

> On 08/07/2013 04:03 PM, Brassow Jonathan wrote:
>> 
>> On Aug 7, 2013, at 4:21 AM, Zdenek Kabelac wrote:
>> 
>>> 
>>> We need fallback here - since if the user doesn't have dm-raid -  then he needs to manually configure defaults.
>>> 
>>> As a result automated tests on systems without dm raid fails.
>>> 
>>> Also we need to update all tests which were supposed to test mirrors,
>>> to really test mirrors - otherwise we are checking raid code twice
>>> (and even without checking  dm-raid version presence)
>> 
>> I'll fix-up all the mirror tests today.
>> 
>> Not sure about what to do regarding a user on an old system with fresh LVM.  The commands should complain about not having RAID kernel support.  Do we overwrite their current lvm.conf when installing?  I don't think so...
>> 
> 
> We do have a patch attached to RHEL5 already that removes some RAID stuff from lvm.conf
> (the mirror_segtype_default setting):
> 
>  http://pkgs.devel.redhat.com/cgit/rpms/lvm2/tree/lvm2-rhel5.patch?h=rhel-5.9
> 
> We can change that if needed...

We will definitely need to do that - and for RHEL6 now too.  RAID1/10 should only become the default upstream and in RHEL7.

I think Zdenek was talking mostly about users who compile their own LVM on machines with older kernels though.

 brassow




      reply	other threads:[~2013-08-07 16:42 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-06 19:14 master - RAID: Make "raid1" the default mirror segment type Jonathan Brassow
2013-08-07  9:21 ` Zdenek Kabelac
2013-08-07 14:03   ` Brassow Jonathan
2013-08-07 14:35     ` Peter Rajnoha
2013-08-07 16:42       ` Brassow Jonathan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=82E23FFA-F01A-4859-BDDD-88587A24E81B@redhat.com \
    --to=jbrassow@redhat.com \
    --cc=lvm-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.