Bug 1043260

Summary:	virsh MAN page - typos and inconsistencies
Product:	Red Hat Enterprise Linux 7	Reporter:	Laura Novich <lnovich>
Component:	libvirt	Assignee:	Jiri Denemark <jdenemar>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Virtualization Bugs <virt-bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	7.0	CC:	acathrow, dyuan, gsun, honzhang, jhradile, jmiao, mzhan
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	libvirt-1.1.1-16.el7	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-06-13 09:32:00 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:

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.