Bug 1318425

Summary: [RFE] lvconvert --repair won't reuse physical volumes if the device isn't specified (part II)
Product: [Community] LVM and device-mapper Reporter: Corey Marthaler <cmarthal>
Component: lvm2Assignee: LVM Team <lvm-team>
lvm2 sub component: Mirroring and RAID QA Contact: cluster-qe <cluster-qe>
Status: NEW --- Docs Contact:
Severity: low    
Priority: unspecified CC: agk, heinzm, jbrassow, msnitzer, prajnoha, zkabelac
Version: unspecifiedKeywords: FutureFeature
Target Milestone: ---Flags: pm-rhel: lvm-technical-solution?
pm-rhel: lvm-test-coverage?
Target Release: ---   
Hardware: x86_64   
OS: All   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 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 2016-03-16 20:30:07 UTC 
Description of problem:
This is about the original intention of bug 1130329 and also what is mentioned in bug 877221. The device to be used in the repair when not specified should be the device that currently makes up part of the spanned raid image.

# Manual attempt with allocate warn fault policy
[root@host-115 ~]# lvs -a -o +devices
  /dev/sdd1: read failed after 0 of 4096 at 0: Input/output error
  /dev/sdd1: read failed after 0 of 4096 at 26838958080: Input/output error
  /dev/sdd1: read failed after 0 of 4096 at 26839048192: Input/output error
  /dev/sdd1: read failed after 0 of 4096 at 4096: Input/output error
  Couldn't find device with uuid xTPPbl-TQIN-o2xA-7qhV-rv1p-Vxb2-qAsVTE.
  LV                                              VG         Attr       LSize   Cpy%Sync Devices
  synced_spanned_primary_raid1_2legs_1            black_bird rwi-aor-p- 500.00m 100.00   synced_spanned_primary_raid1_2legs_1_rimage_0(0),synced_spanned_primary_raid1_2legs_1_rimage_1(0)
  [synced_spanned_primary_raid1_2legs_1_rimage_0] black_bird iwi-aor-p- 500.00m          /dev/sdc1(1)
  [synced_spanned_primary_raid1_2legs_1_rimage_0] black_bird iwi-aor-p- 500.00m          unknown device(0)
  [synced_spanned_primary_raid1_2legs_1_rimage_1] black_bird iwi-aor--- 500.00m          /dev/sdg1(1)
  [synced_spanned_primary_raid1_2legs_1_rimage_1] black_bird iwi-aor--- 500.00m          /dev/sda1(0)
  [synced_spanned_primary_raid1_2legs_1_rmeta_0]  black_bird ewi-aor-r-   4.00m          /dev/sdc1(0)
  [synced_spanned_primary_raid1_2legs_1_rmeta_1]  black_bird ewi-aor---   4.00m          /dev/sdg1(0)

[root@host-115 ~]# lvconvert --yes --repair black_bird/synced_spanned_primary_raid1_2legs_1
  /dev/sdd1: read failed after 0 of 4096 at 0: Input/output error
  /dev/sdd1: read failed after 0 of 4096 at 26838958080: Input/output error
  /dev/sdd1: read failed after 0 of 4096 at 26839048192: Input/output error
  /dev/sdd1: read failed after 0 of 4096 at 4096: Input/output error
  Couldn't find device with uuid xTPPbl-TQIN-o2xA-7qhV-rv1p-Vxb2-qAsVTE.
  Faulty devices in black_bird/synced_spanned_primary_raid1_2legs_1 successfully replaced.

### So above, the device that should have been used was /dev/sdc1. Instead /dev/sdb1 is used. /dev/sdc1 would have worked if it had been specified on the lvconvert cmd line, which makes this a low priority bug.

[root@host-115 ~]# lvs -a -o +devices
  /dev/sdd1: open failed: No such device or address
  Couldn't find device with uuid xTPPbl-TQIN-o2xA-7qhV-rv1p-Vxb2-qAsVTE.
  LV                                              VG         Attr       LSize   Cpy%Sync Devices
  synced_spanned_primary_raid1_2legs_1            black_bird rwi-aor--- 500.00m 100.00   synced_spanned_primary_raid1_2legs_1_rimage_0(0),synced_spanned_primary_raid1_2legs_1_rimage_1(0)
  [synced_spanned_primary_raid1_2legs_1_rimage_0] black_bird iwi-aor--- 500.00m          /dev/sdb1(1)
  [synced_spanned_primary_raid1_2legs_1_rimage_1] black_bird iwi-aor--- 500.00m          /dev/sdg1(1)
  [synced_spanned_primary_raid1_2legs_1_rimage_1] black_bird iwi-aor--- 500.00m          /dev/sda1(0)
  [synced_spanned_primary_raid1_2legs_1_rmeta_0]  black_bird ewi-aor---   4.00m          /dev/sdb1(0)
  [synced_spanned_primary_raid1_2legs_1_rmeta_1]  black_bird ewi-aor---   4.00m          /dev/sdg1(0)

Version-Release number of selected component (if applicable):
2.6.32-616.el6.x86_64

lvm2-2.02.143-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016
lvm2-libs-2.02.143-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016
lvm2-cluster-2.02.143-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016
udev-147-2.72.el6    BUILT: Tue Mar  1 06:14:05 CST 2016
device-mapper-1.02.117-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016
device-mapper-libs-1.02.117-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016
device-mapper-event-1.02.117-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016
device-mapper-event-libs-1.02.117-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016
device-mapper-persistent-data-0.6.2-0.1.rc5.el6    BUILT: Wed Feb 24 07:07:09 CST 2016
cmirror-2.02.143-2.el6    BUILT: Wed Mar 16 08:30:42 CDT 2016


How reproducible:
Everytime