]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commit
tools: fix the wrong check when use virsh setvcpus --maximum
authorLuyao Huang <lhuang@redhat.com>
Fri, 20 Mar 2015 14:39:03 +0000 (15:39 +0100)
committerPavel Hrdina <phrdina@redhat.com>
Mon, 4 May 2015 08:26:57 +0000 (10:26 +0200)
commit3dae162db7f53cce57a21ad15799ace27768a8a0
treea019cebbed4e6ea39251d1e853f7e9110e0c3dc5
parent170fb72c704bdf8fd2eb4510938a58d559bba28f
tools: fix the wrong check when use virsh setvcpus --maximum

The --maximum option wasn't properly parsed and the equivalent flag
wasn't set.  Fix this bug and also rewrite the way we check this option
by using new macro.  The new approach is that --maximum requires
--config, no other combination is allowed, because they don't make sense.

The new error will be:

 # virsh setvcpus test --maximum 10
 error: Option --config is required by option --maximum

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1204033

Signed-off-by: Luyao Huang <lhuang@redhat.com>
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
tools/virsh-domain.c
tools/virsh.pod