render / rpms / libvirt

Forked from rpms/libvirt 5 months ago
Clone
c401cc
From 40dc061be172fcd64be29054782254c5c0d3786f Mon Sep 17 00:00:00 2001
c401cc
Message-Id: <40dc061be172fcd64be29054782254c5c0d3786f.1385135432.git.jdenemar@redhat.com>
c401cc
From: Peter Krempa <pkrempa@redhat.com>
c401cc
Date: Mon, 11 Nov 2013 11:30:02 +0100
c401cc
Subject: [PATCH] virsh-domain: Mark --live and --config mutually exclusive in
c401cc
 vcpucount
c401cc
c401cc
The 'vcpucount' command is a getter command for the vCPUu count. When
c401cc
one or more of the filtering flags are specified the command returns the
c401cc
value only for the selected combination. In this case the --live and
c401cc
--config combination isn't valid. This however didn't cause errors as
c401cc
the combination of flags was rejected by the libvirt API but then the
c401cc
fallback code kicked in and requested the count in a way where the clash
c401cc
of the flags didn't matter.
c401cc
c401cc
Mark the flag combination mutually exclusive so that users aren't
c401cc
confused.
c401cc
c401cc
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1024245
c401cc
(cherry picked from commit bf45db6041c207e78e6a3185ceaf278b09e21561)
c401cc
c401cc
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
c401cc
---
c401cc
 tools/virsh-domain.c | 1 +
c401cc
 1 file changed, 1 insertion(+)
c401cc
c401cc
diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
c401cc
index b4faa69..b692299 100644
c401cc
--- a/tools/virsh-domain.c
c401cc
+++ b/tools/virsh-domain.c
c401cc
@@ -5345,6 +5345,7 @@ cmdVcpucount(vshControl *ctl, const vshCmd *cmd)
c401cc
     if (!maximum && !active && current)
c401cc
         current = false;
c401cc
 
c401cc
+    VSH_EXCLUSIVE_OPTIONS_VAR(live, config)
c401cc
     VSH_EXCLUSIVE_OPTIONS_VAR(current, live);
c401cc
     VSH_EXCLUSIVE_OPTIONS_VAR(current, config);
c401cc
     VSH_EXCLUSIVE_OPTIONS_VAR(active, maximum);
c401cc
-- 
c401cc
1.8.4.4
c401cc