Bug 1439937

Summary:	RAID TAKEOVER: confusing "Replaced LV type raid5_n with possible type raid5_n" warning
Product:	Red Hat Enterprise Linux 7	Reporter:	Corey Marthaler <cmarthal>
Component:	lvm2	Assignee:	LVM and device-mapper development team <lvm-team>
lvm2 sub component:	Mirroring and RAID	QA Contact:	Heinz Mauelshagen <heinzm>
Status:	CLOSED ERRATA	Docs Contact:
Severity:	low
Priority:	unspecified	CC:	agk, heinzm, jbrassow, msnitzer, prajnoha, prockai, zkabelac
Version:	7.4	Keywords:	Reopened
Target Milestone:	rc
Target Release:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:	lvm2-2.02.170-2.el7	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-08-01 21:52:19 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Corey Marthaler 2017-04-06 22:27:34 UTC

Description of problem:

[root@host-073 ~]# lvcreate -i 2 --type raid4 -L 100M -n LV1 VG
  Using default stripesize 64.00 KiB.
  Rounding size 100.00 MiB (25 extents) up to stripe boundary size 104.00 MiB(26 extents).
  Logical volume "LV1" created.

# Replaced LV type raid5_n? but that's exactly the type i specified.
[root@host-073 ~]# lvconvert --type raid5_n VG/LV1
  Using default stripesize 64.00 KiB.
  Replaced LV type raid5_n with possible type raid5_n.
Do you want to convert raid5_n LV VG/LV1 to raid5_n? [y/n]: y
  Logical volume VG/LV1 successfully converted.

[root@host-073 ~]# lvs -a -o +devices,segtype
  LV             VG  Attr       LSize   Cpy%Sync Devices                                         Type   
  LV1            VG  rwi-a-r--- 104.00m 100.00   LV1_rimage_0(0),LV1_rimage_1(0),LV1_rimage_2(0) raid5_n
  [LV1_rimage_0] VG  iwi-aor---  52.00m          /dev/sdb1(1)                                    linear 
  [LV1_rimage_1] VG  iwi-aor---  52.00m          /dev/sdc1(1)                                    linear 
  [LV1_rimage_2] VG  iwi-aor---  52.00m          /dev/sda1(1)                                    linear 
  [LV1_rmeta_0]  VG  ewi-aor---   4.00m          /dev/sdb1(0)                                    linear 
  [LV1_rmeta_1]  VG  ewi-aor---   4.00m          /dev/sdc1(0)                                    linear 
  [LV1_rmeta_2]  VG  ewi-aor---   4.00m          /dev/sda1(0)                                    linear 

# Here it makes sense as raid6 is not possible, however raid6_n_6 is.
[root@host-073 ~]# lvconvert --type raid6 VG/LV1
  Using default stripesize 64.00 KiB.
  Replaced LV type raid6 with possible type raid6_n_6.
Do you want to convert raid6 LV VG/LV1 to raid6_n_6? [y/n]: y


Version-Release number of selected component (if applicable):
lvm2-2.02.169-3.el7    BUILT: Wed Mar 29 09:17:46 CDT 2017
lvm2-libs-2.02.169-3.el7    BUILT: Wed Mar 29 09:17:46 CDT 2017
lvm2-cluster-2.02.169-3.el7    BUILT: Wed Mar 29 09:17:46 CDT 2017
device-mapper-1.02.138-3.el7    BUILT: Wed Mar 29 09:17:46 CDT 2017
device-mapper-libs-1.02.138-3.el7    BUILT: Wed Mar 29 09:17:46 CDT 2017
device-mapper-event-1.02.138-3.el7    BUILT: Wed Mar 29 09:17:46 CDT 2017
device-mapper-event-libs-1.02.138-3.el7    BUILT: Wed Mar 29 09:17:46 CDT 2017
device-mapper-persistent-data-0.7.0-0.1.rc6.el7    BUILT: Mon Mar 27 10:15:46 CDT 2017

Comment 2 Heinz Mauelshagen 2017-04-12 12:43:06 UTC

Fixed in upstream commit eb6302c8cb6bfd1d9831e5ff3d76ecbb49613178 which'll be contained in lvm2 version 170.

Comment 3 Heinz Mauelshagen 2017-04-12 14:13:16 UTC

Correcting to POST for further errate processing

Comment 4 Corey Marthaler 2017-04-12 21:14:58 UTC

Here's at least one more:

lvconvert --yes  --type raid5_rs centipede2/takeover
  Replaced LV type raid5_rs with possible type raid5_rs.

Comment 6 Corey Marthaler 2017-04-13 22:41:49 UTC

Fix verified in the latest rpms.

[root@host-116 ~]# lvcreate -i 2 --type raid4 -L 100M -n LV1 VG
  Using default stripesize 64.00 KiB.
  Rounding size 100.00 MiB (25 extents) up to stripe boundary size 104.00 MiB(26 extents).
  Logical volume "LV1" created.

[root@host-116 ~]# lvconvert --type raid5_n VG/LV1
  Using default stripesize 64.00 KiB.
Are you sure you want to convert raid4 LV VG/LV1 to raid5_n type? [y/n]: y
  Logical volume VG/LV1 successfully converted.

[root@host-116 ~]# lvconvert --yes  --type raid5_rs VG/LV1
  Using default stripesize 64.00 KiB.
  Converting raid5_n LV VG/LV1 to raid5_rs.
  Logical volume VG/LV1 successfully converted.



3.10.0-651.el7.x86_64

lvm2-2.02.170-2.el7    BUILT: Thu Apr 13 14:37:43 CDT 2017
lvm2-libs-2.02.170-2.el7    BUILT: Thu Apr 13 14:37:43 CDT 2017
lvm2-cluster-2.02.170-2.el7    BUILT: Thu Apr 13 14:37:43 CDT 2017
device-mapper-1.02.139-2.el7    BUILT: Thu Apr 13 14:37:43 CDT 2017
device-mapper-libs-1.02.139-2.el7    BUILT: Thu Apr 13 14:37:43 CDT 2017
device-mapper-event-1.02.139-2.el7    BUILT: Thu Apr 13 14:37:43 CDT 2017
device-mapper-event-libs-1.02.139-2.el7    BUILT: Thu Apr 13 14:37:43 CDT 2017
device-mapper-persistent-data-0.7.0-0.1.rc6.el7    BUILT: Mon Mar 27 10:15:46 CDT 2017

Comment 7 errata-xmlrpc 2017-08-01 21:52:19 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:2222