Re: [PATCH v2] tools: Fix connect command

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

 



On Sat, Apr 23, 2016 at 01:18:48PM -0400, Cole Robinson wrote:
On 04/22/2016 02:28 AM, Martin Kletzander wrote:
The man page says: "(Re)-Connect to the hypervisor. When the shell is
first started, this is automatically run with the URI parameter
requested by the "-c" option on the command line."  However, if you run:

  virsh -c 'test://default' 'connect; uri'

the output will not be 'test://default'.  That's because the 'connect'
command does not care about any virsh-only related settings and if it is
run without parameters, it connects with @uri == NULL.  Not only that
doesn't comply to what the man page describes, but it also doesn't make
sense.  It also means you aren't able to reconnect to whatever you are
connected currently.

So let's fix that in both virsh and virt-admin add a test case for it.

Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>

There's something funky here:

$ sudo ./tools/virsh --connect qemu:///system
Welcome to lt-virsh, the virtualization interactive terminal.

Type:  'help' for help with commands
      'quit' to quit

virsh # connect test:///default
error: Reconnected to the hypervisor

virsh # list --all
Id    Name                           State
----------------------------------------------------
1     test                           running


It prints an error, but seems to succeed


I've never noticed that "Reconnected to hypervisor" is an error :D
I'll look into that.

- Cole

Attachment: signature.asc
Description: 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]