Re: [PATCH] virsh-domain: Mark --live and --config mutually exclusive in vcpucount

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/07/2013 03:18 AM, Peter Krempa wrote:
> (cc'd Eric as a gentle ping)

Thanks.

>>>> I seem to recall specifically allowing '--live --config' as a way of
>>>> requesting multiple values in one command call (similar to how vcpucount
>>>> with no flags grabs every possible value).  I need to think about this
>>>> more, and double check historical behavior, to make sure we aren't
>>>> adding a regression.  Please hold off until after 1.1.4 for this one
>>>> (and ping me if I haven't responded with more details in a week, since
>>>> I'm currently busy on another patch).
>>
>> Any thoughts now that the release is out?

Let's look at the logic in an older version:

git show v0.9.0:tools/virsh.c

cmdVcpucount supported only --maximum, --current, --config, and --live,
with mutual exclusion for maximum/current, config/live, and then a check
that either no flags were present or else exactly 2 flags were present.
 (Weird, but that's what it was).  Thus, the valid call modes were:

vcpucount $dom => show four lines for all 4 combinations (with an error
message in place of a line for settings not possible, such as the config
lines on a transient domain)
vcpucount $dom --maximum --config => show one line
vcpucount $dom --maximum --live => show one line
vcpucount $dom --current --config => show one line
vcpucount $dom --current --live => show one line
all other combinations => error

0.9.4 changed things by adding --active and repurposing --config so that
it allowed back-compat use while also being more consistent with other
--live/--config/--current three-way choices.

>>
>>>
>>> If you look closely at the code, you'll see that there's no option to
>>> request multiple values other than not specifying any option when a
>>> table of the cpus is printed.

Indeed.  This is still true even in the 1.1.4 release, even where we
added --active and repurposed --current.

Okay, I've now played with your patch installed, and verified that the
pre-0.9.4 back-compat options that are supposed to work still do.  We no
longer reject attempts to use just one flag (vcpucount test --live is
now shorthand for vcpucount test --live --active); but that's not the
fault of this patch.  So it looks like all you really do is reject a
three-option combination, which matches our documentation that you
either omit all flags to get all lines printed, or specify at most 2
flags to print a specific line.

ACK.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]