Bug 749487 - Minor typo on page 23, section 4.2.1.2:
Summary: Minor typo on page 23, section 4.2.1.2:
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: doc-Logical_Volume_Manager
Version: 6.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Steven J. Levine
QA Contact: ecs-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-27 08:16 UTC by Damiano Bianchi
Modified: 2011-11-15 17:38 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-11-15 17:38:33 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Damiano Bianchi 2011-10-27 08:16:20 UTC
Description of problem:
Minor typo on page 23, section 4.2.1.2:

pvcreate /dev/sdd1 /dev/sde1 /dev/sdf1

should be

pvcreate /dev/sdd /dev/sde /dev/sdf

Because the paragraph explain how to initialize the whole disk, not a specified partition.


Version-Release number of selected component (if applicable):
Logical_Volume_Manager_Administration(EN)-6 (2011-09-19-15:20)

Comment 2 Steven J. Levine 2011-10-27 21:57:36 UTC
Thanks for pointing this out -- it uncovered something. As a matter of practice and policy, the LVM manual examples in this document all use partitions (sdd1 rather than sdd, for example). This section originally, in an early draft, showed how to intialize a whole disk, then how to intialize a partition. But then we went through and updated the examples so that they used partitions -- and that left this particular paragraph unclear.

The problem here is that both examples show the same thing: initializing partitions for use as LVM physical volumes.

So what I've done is removed the second example. Now the section says:

--------------------
Use the pvcreate command to
initialize a block device to be used as a physical volume.
Initialization is analogous to formatting a file system.

The following command initializes the partitions
/dev/sdd1, /dev/sde1, and /dev/sdf1 as LVM physical
volumes for later use as part of LVM logical volumes.

pvcreate /dev/sdd1 /dev/sde1 /dev/sdf1
-----------

And I've removed the example intializing /dev/hdb1, which just showed the same thing. This makes it clear that you can initialize partitions as PVs, which is what we want to show.

I have made these changes to my draft and checked them in to the SVN repository.

Comment 5 Steven J. Levine 2011-11-07 17:46:03 UTC
Re-opening this based on this feedback from Tom Coughlan:

Steven, our preference is for customers to use whole disks, with no
partition table. If it is possible that the disk may be used in a shared
environment, and software in that environment may scan for a partition
table in order to determine whether the disk is in use, then the
customer may optionally place a partition table on the disk, and create
PV(s) on the partitions(s). 

Maybe you can leave the one example in the document, showing creation of
a PV on the whole disk, and then add a note with the information above. 

------

So I'm returning to the original 2-example description, with the correction noted in the bug.

Comment 6 Steven J. Levine 2011-11-07 19:18:56 UTC
I have fixed the content of the document as per the original bug, restoring the paragraph about creating pvs on whole disks and using the whole disks in the first example, followed by a paragraph about creating pvs on partitions and restoring the original example that creates pvs on partitions.

This is build:
Red_Hat_Enterprise_Linux-Logical_Volume_Manager_Administration-6-Beta-web-en-US-1-44.el6eng

Section 4.2.1.2

I'm moving this bug to MODIFIED. I will move it to ON_QA when the build completes.


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