Bug 818020

Summary: fail to rename the thin LV
Product: [Fedora] Fedora Reporter: Xiaowei Li <xiaoli>
Component: lvm2Assignee: Zdenek Kabelac <zkabelac>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: 18CC: agk, bmarzins, bmr, dwysocha, heinzm, jonathan, lvm-team, mbroz, msnitzer, prajnoha, prockai, qcai, zkabelac
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: lvm2-2.02.97-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 820229 (view as bug list) Environment:
Last Closed: 2012-12-07 05:59:47 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:
Bug Depends On:    
Bug Blocks: 820229    

Description Xiaowei Li 2012-05-02 03:05:13 UTC 
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.# lvs tsvg
lvs  LV   VG   Attr     LSize   Pool Origin Data%  Move Log Copy%  Convert
lvs  lv1  tsvg Vwi-a-tz 100.00m pool          0.00                        
lvs  lv2  tsvg Vwi-a-tz 100.00m pool          0.00                        
lvs  pool tsvg twi-a-tz   4.00m               0.00     
              
# lvrename tsvg lv1 lv3
lvrename  Cannot rename "lv1": name format not recognized for internal LV "pool1"

2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Alasdair Kergon 2012-05-02 12:45:50 UTC 
What about the other form of the command - does that work or fail?

lvrename tsvg/lv1 lv3
Comment 2 Zdenek Kabelac 2012-05-02 12:56:18 UTC 
This is bug in for_each_sub_lv() logic in lv_manip code  for thin pool rename  

While for renaming pool - multiple LV must be renamed,  for  thin volume this is not the case.
Comment 3 Xiaowei Li 2012-05-03 01:15:49 UTC 
(In reply to comment #1)
> What about the other form of the command - does that work or fail?
> 
> lvrename tsvg/lv1 lv3

Still failed.
[root@laker T1]# lvrename tsvg/lv1 lv2
lvrename  Cannot rename "lv1": name format not recognized for internal LV "pool"
Comment 4 Zdenek Kabelac 2012-05-09 12:37:24 UTC 
Regression fixed upstream with this commit:

https://www.redhat.com/archives/lvm-devel/2012-May/msg00003.html
Comment 5 Alasdair Kergon 2012-07-31 19:24:45 UTC 
Fixed in 2.02.96
Comment 6 Xiaowei Li 2012-09-03 10:00:16 UTC 
verified with lvm2-2.02.97-1.el7.x86_64
Comment 7 Xiaowei Li 2012-11-14 03:55:34 UTC 
change to verified according to the comment 6