Bug 284771 - RFE: pvmove: give better diagnostic for invalid "-n LV" name
Summary: RFE: pvmove: give better diagnostic for invalid "-n LV" name
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: lvm2
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Milan Broz
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-09-10 16:30 UTC by Jim Meyering
Modified: 2013-03-13 20:41 UTC (History)
5 users (show)

Fixed In Version: 2.02.34
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-07-31 14:38:09 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
show pvmove's misleading diagnostic (1.28 KB, application/octet-stream)
2007-09-10 16:30 UTC, Jim Meyering
no flags Details
Proposed patch (2.82 KB, patch)
2008-04-03 15:50 UTC, Milan Broz
no flags Details | Diff

Description Jim Meyering 2007-09-10 16:30:52 UTC
Description of problem:
I was following an example from a non-RH (or old?) version of the pvmove man
page (http://linux.about.com/library/cmd/blcmdl8_pvmove.htm), that suggested
using pvmove's "-n vg_name:PE" option, e.g., "-n vg1:0".  However, using a PE
range there is not valid, and makes pvmove complain, just as if one had
specified any invalid VG name.  FWIW, the PE range must be appended to the
source or dest device name.

Version-Release number of selected component (if applicable):
LVM version:     2.02.27 (2007-07-17)

How reproducible:
see attached script

Steps to Reproduce:
1. run the script, and note the "No data to move for vg-pvmove-demo" diagnostic
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Jim Meyering 2007-09-10 16:30:52 UTC
Created attachment 191761 [details]
show pvmove's misleading diagnostic

Comment 2 Milan Broz 2008-04-03 15:50:39 UTC
Created attachment 300272 [details]
Proposed patch

Comment 3 Milan Broz 2008-04-04 12:00:34 UTC
Patch is in upstream cvs.

Comment 4 Jon Stanley 2008-04-23 20:30:03 UTC
Adding FutureFeature keyword to RFE's.

Comment 5 Milan Broz 2008-07-31 14:38:09 UTC
This is already fixed in rawhide since lvm2 2.02.34.


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