Bug 1043260 - virsh MAN page - typos and inconsistencies
Summary: virsh MAN page - typos and inconsistencies
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: libvirt
Version: 7.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Jiri Denemark
QA Contact: Virtualization Bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-12-15 13:56 UTC by Laura Novich
Modified: 2016-01-27 01:57 UTC (History)
7 users (show)

Fixed In Version: libvirt-1.1.1-16.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-06-13 09:32:00 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Laura Novich 2013-12-15 13:56:08 UTC
Description of problem:

Virsh man page has the following issues:
1. virsh connect command lists XEN as the default setting.
   as far as I know XEN is not supported in RHEL 7. Please indicate what the default setting is, and remove XEN from the MAN page.

2. the nodecpumap command displays the number of cpus correctly. I am not sure what the intended format for the display is supposed to be but I got this:
CPUs present: 1
CPUs online: 1
CPU map:  y

According to the MAN page it is supposed to give me a list of the online CPUs. I would assume it would tell me its id number or name. In this case, all I got was a number of online CPUs, not a list. If this is intentional, please change the language of the MAN page. If not, then this is a bug that needs to be fixed elsewhere.

I am sure there will be more issues - but I will open other BZs..


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 2 Jiri Denemark 2013-12-16 14:16:20 UTC
1. pushed upstream as

    commit dbfe8a089577ce7da5d66c0c53d9b5be5aa41a96
    Author: Jiri Denemark <jdenemar>
    Date:   Mon Dec 16 13:37:12 2013 +0100

        tools: Fix virsh connect man page
        
        The URI parameter is optional and xen:/// is not the default connection
        URI.
        
        Signed-off-by: Jiri Denemark <jdenemar>

2. "CPU map:  y" is the list (or rather map). It prints "y" for each CPU that is online and "-" offline CPUs. Four online CPUs out of 4 would be shown as "yyyy". If the third CPU (ID 2) is offline, "yy-y" would be reported.

Comment 5 Jincheng Miao 2013-12-20 08:37:50 UTC
1. This doc fix can be verified:
# man virsh
...
       connect [URI] [--readonly]
...
           xen:///
               this is used to connect to the local Xen hypervisor
...

Some typo was fixed, so I change the status to VERIFIED.

2. for nodecpumap
check cpus
# virsh nodecpumap
CPUs present:   8
CPUs online:    8
CPU map:        yyyyyyyy

shutdown cpu-1
# echo 0 >/sys/devices/system/cpu/cpu1/online 

recheck cpus
# virsh nodecpumap
CPUs present:   8
CPUs online:    7
CPU map:        y-yyyyyy

nodecpumap works.

So I change the status to VERIFIED.

Comment 6 Ludek Smid 2014-06-13 09:32:00 UTC
This request was resolved in Red Hat Enterprise Linux 7.0.

Contact your manager or support representative in case you have further questions about the request.


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