[Libvir] Add 'console' and 'vncdisplay' commands to virsh

The attached patch adds two new commands to virsh: - console - connects to the guest's serial console - vncdisplay - outputs the ip address & port number of a guest vnc display The former is another stage in eliminating the need to run 'xm' - replacing the Xen specific 'xm console' code, with the added advantage of working with any libvirt backend driver. The vncdisplay is intended to make it easier for people to launch a VNC viewer process. It prints out a IP address & port number in a format suitable for passing to vncviewer on the command line, eg vncviewer `virsh vncdisplay myguest` Regards, Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|

On Fri, Jan 26, 2007 at 02:58:27AM +0000, Daniel P. Berrange wrote:
The attached patch adds two new commands to virsh:
- console - connects to the guest's serial console - vncdisplay - outputs the ip address & port number of a guest vnc display
The former is another stage in eliminating the need to run 'xm' - replacing the Xen specific 'xm console' code, with the added advantage of working with any libvirt backend driver.
Very cool. I wonder if we should not move out virsh specific code to a new separate subdir, like for the proxy to clean things up a bit in src/
The vncdisplay is intended to make it easier for people to launch a VNC viewer process. It prints out a IP address & port number in a format suitable for passing to vncviewer on the command line, eg
vncviewer `virsh vncdisplay myguest`
Okay, the other option would have been to fork vncviewer directly, but that's more generic. Seems however that would be incomplete if connecting to a remote xend, but that's something we should be able to do properly once we have remote support integrated. Daniel -- Red Hat Virtualization group http://redhat.com/virtualization/ Daniel Veillard | virtualization library http://libvirt.org/ veillard@redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/

On Fri, Jan 26, 2007 at 05:34:35AM -0500, Daniel Veillard wrote:
On Fri, Jan 26, 2007 at 02:58:27AM +0000, Daniel P. Berrange wrote:
The attached patch adds two new commands to virsh:
- console - connects to the guest's serial console - vncdisplay - outputs the ip address & port number of a guest vnc display
The former is another stage in eliminating the need to run 'xm' - replacing the Xen specific 'xm console' code, with the added advantage of working with any libvirt backend driver.
Very cool. I wonder if we should not move out virsh specific code to a new separate subdir, like for the proxy to clean things up a bit in src/
There's only three files for virsh in there soo far which isn't too many, so for now I think its fine having it in one place. It makes life easier with building/testing too because you can build libvirt & run virsh to test it without changing dirs.
The vncdisplay is intended to make it easier for people to launch a VNC viewer process. It prints out a IP address & port number in a format suitable for passing to vncviewer on the command line, eg
vncviewer `virsh vncdisplay myguest`
Okay, the other option would have been to fork vncviewer directly, but that's more generic. Seems however that would be incomplete if connecting to a remote xend, but that's something we should be able to do properly once we have remote support integrated.
I've commited the patches now. Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|

On Fri, Jan 26, 2007 at 05:34:35AM -0500, Daniel Veillard wrote:
On Fri, Jan 26, 2007 at 02:58:27AM +0000, Daniel P. Berrange wrote:
The attached patch adds two new commands to virsh:
- console - connects to the guest's serial console - vncdisplay - outputs the ip address & port number of a guest vnc display
Thanks!
The former is another stage in eliminating the need to run 'xm' - replacing the Xen specific 'xm console' code, with the added advantage of working with any libvirt backend driver.
Very cool. I wonder if we should not move out virsh specific code to a new separate subdir, like for the proxy to clean things up a bit in src/
This is old kzak's idea :-) Yes, it's one large file, but it doesn't meen that we cannot split it to more files. The virsh.c file will be biger and biger. Karel -- Karel Zak <kzak@redhat.com>
participants (3)
-
Daniel P. Berrange
-
Daniel Veillard
-
Karel Zak