- Devel - libvirt List Archives

[libvirt] CSharp bindings patch
by achampion＠neuf.fr 14 Nov '10

14 Nov '10

?Hi, here is a 3rd patch that add a sample named "virDomainStats" Regards, Arnaud

1 0

[libvirt] CSharp bindings patch
by achampion＠neuf.fr 14 Nov '10

14 Nov '10

?Hi, here is a 2nd patch to correct marshaling around virError. Regards, Arnaud

1 0

[libvirt] CSharp Bindings Patch to correct some marshaling
by achampion＠neuf.fr 14 Nov '10

14 Nov '10

?Hi, Here is a new C# bindings patch to correct some marshaling mistakes around domain statistics. Regards, Arnaud

1 0

[libvirt] First Virsh command Reference Pages online
by Justin Clift 13 Nov '10

13 Nov '10

Hi all, Started work on a Virsh "Command Reference" a few days ago, as a way to "Officially Document" (!) the many virsh commands. My plan is to add to the commands here as I work on related sections. So, as I'm working on "Virtual Networking" at the moment, it's the net-* commands that have been added. This is a first go, and it's on the wiki: http://wiki.libvirt.org/page/VirshCmdRef The net-* commands there should all be accurate at this point: http://wiki.libvirt.org/page/VirshCmdNetAutostart http://wiki.libvirt.org/page/VirshCmdNetCreate http://wiki.libvirt.org/page/VirshCmdNetDefine http://wiki.libvirt.org/page/VirshCmdNetDestroy http://wiki.libvirt.org/page/VirshCmdNetDumpXML http://wiki.libvirt.org/page/VirshCmdNetEdit http://wiki.libvirt.org/page/VirshCmdNetList http://wiki.libvirt.org/page/VirshCmdNetName http://wiki.libvirt.org/page/VirshCmdNetStart http://wiki.libvirt.org/page/VirshCmdNetUndefine http://wiki.libvirt.org/page/VirshCmdNetUUID Does anyone have time to look over them, and make sure there aren't any obvious mistakes? If things are ok, I'll convert them to standard HTML tomorrow and move them to the main libvirt docs. Regards and best wishes, Justin Clift

5 17

[libvirt] [PATCH] docs: updated virsh command reference with libvirt.org links
by Justin Clift 13 Nov '10

13 Nov '10

--- docs/virshcmdref.html.in | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/docs/virshcmdref.html.in b/docs/virshcmdref.html.in index a5a7554..0a19ca7 100644 --- a/docs/virshcmdref.html.in +++ b/docs/virshcmdref.html.in @@ -31,10 +31,10 @@ <ul> <li> - <a href="http://justinclift.fedorapeople.org/virshcmdref/html/">Standard HTML format, multiple pages</a> + <a href="http://libvirt.org/sources/virshcmdref/html/">Standard HTML format, multiple pages</a> </li> <li> - <a href="http://justinclift.fedorapeople.org/virshcmdref/html-single/">HTML format, one long page</a> + <a href="http://libvirt.org/sources/virshcmdref/html-single/">HTML format, one long page</a> </li> </ul> @@ -49,21 +49,21 @@ <ul> <li> Standard HTML format, multiple pages - (<a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">.tar.gz</a>) - (<a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">.tar.bz2</a>) - (<a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">.zip</a>) + (<a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0-html…">.tar.gz</a>) + (<a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0-html…">.tar.bz2</a>) + (<a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0-html…">.zip</a>) </li> <li> HTML format, one long page - (<a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">.tar.gz</a>) - (<a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">.tar.bz2</a>) - (<a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">.zip</a>) + (<a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0-html…">.tar.gz</a>) + (<a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0-html…">.tar.bz2</a>) + (<a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0-html…">.zip</a>) </li> <li> - <a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">PDF format</a> + <a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0.pdf">PDF format</a> </li> <li> - <a href="http://justinclift.fedorapeople.org/virshcmdref/Virsh_Command_Reference-0.8…">ePub format</a> + <a href="http://libvirt.org/sources/virshcmdref/Virsh_Command_Reference-0.8.6-0.epub">ePub format</a> </li> </ul> -- 1.7.3

1 1

[libvirt] [PATCH] virsh: add command net-info
by Osier Yang 13 Nov '10

13 Nov '10

To list basic information about the network. * tools/virsh.c --- tools/virsh.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 59 insertions(+), 0 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index e704799..54643f2 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -4088,6 +4088,64 @@ cmdNetworkUuid(vshControl *ctl, const vshCmd *cmd) return TRUE; } +/* + * "net-info" command + */ +static const vshCmdInfo info_network_info[] = { + {"help", N_("network information")}, + {"desc", "Returns basic information about the network"}, + {NULL, NULL} +}; + +static const vshCmdOptDef opts_network_info[] = { + {"network", VSH_OT_DATA, VSH_OFLAG_REQ, N_("network name")}, + {NULL, 0, 0, NULL} +}; + +static int +cmdNetworkInfo(vshControl *ctl, const vshCmd *cmd) +{ + virNetworkPtr network; + char uuid[VIR_UUID_STRING_BUFLEN]; + int autostart; + int persistent = -1; + int active = -1; + char *bridge = NULL; + + if (!vshConnectionUsability(ctl, ctl->conn)) + return FALSE; + + if (!(network = vshCommandOptNetworkBy(ctl, cmd, NULL, + VSH_BYNAME))) + return FALSE; + + vshPrint(ctl, "%-15s %s\n", _("Name"), virNetworkGetName(network)); + + if (virNetworkGetUUIDString(network, uuid) == 0) + vshPrint(ctl, "%-15s %s\n", _("UUID"), uuid); + + active = virNetworkIsActive(network); + if (active >= 0) + vshPrint(ctl, "%-15s %s\n", _("Active:"), active? _("yes") : _("no")); + + persistent = virNetworkIsPersistent(network); + if (persistent < 0) + vshPrint(ctl, "%-15s %s\n", _("Persistent:"), _("unknown")); + else + vshPrint(ctl, "%-15s %s\n", _("Persistent:"), persistent ? _("yes") : _("no")); + + if (virNetworkGetAutostart(network, &autostart) < 0) + vshPrint(ctl, "%-15s %s\n", _("Autostart:"), _("no autostart")); + else + vshPrint(ctl, "%-15s %s\n", _("Autostart:"), autostart ? _("yes") : _("no")); + + bridge = virNetworkGetBridgeName(network); + if (bridge) + vshPrint(ctl, "%-15s %s\n", _("Bridge:"), bridge); + + virNetworkFree(network); + return TRUE; +} /**************************************************************************/ /* @@ -9924,6 +9982,7 @@ static const vshCmdDef commands[] = { {"net-start", cmdNetworkStart, opts_network_start, info_network_start}, {"net-undefine", cmdNetworkUndefine, opts_network_undefine, info_network_undefine}, {"net-uuid", cmdNetworkUuid, opts_network_uuid, info_network_uuid}, + {"net-info", cmdNetworkInfo, opts_network_info, info_network_info}, {"iface-list", cmdInterfaceList, opts_interface_list, info_interface_list}, {"iface-name", cmdInterfaceName, opts_interface_name, info_interface_name}, -- 1.7.2.3

2 3

[libvirt] boostrap: gzip version check problem on FreeBSD
by Matthias Bolte 13 Nov '10

13 Nov '10

boostrap.conf lists gzip as build dependency. bootstrap then tries to get it's version number using a get_version() function that executes 'gzio --version' and tries to parse the result. The sed expression expects the version number to contain a dot as gzip outputs it on Linux: $ gzip --version gzip 1.3.12 But on FreeBSD it outputs this: # gzip --version FreeBSD gzip 20100407 Therefore, the sed expression should be relaxed, or bootstrap should not try to parse the version number when it doesn't need it because gzip is listed with '-' as version requirement. Matthias

4 4

[libvirt] [PATCH] qemu: fix typos in qemu_monitor_text.c
by Osier Yang 13 Nov '10

13 Nov '10

* src/qemu/qemu_monitor_text.c (qemuMonitorTextChangeMedia) --- src/qemu/qemu_monitor_text.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_monitor_text.c b/src/qemu/qemu_monitor_text.c index 2552111..64ec57b 100644 --- a/src/qemu/qemu_monitor_text.c +++ b/src/qemu/qemu_monitor_text.c @@ -906,7 +906,7 @@ int qemuMonitorTextChangeMedia(qemuMonitorPtr mon, if (qemuMonitorCommand(mon, cmd, &reply) < 0) { qemuReportError(VIR_ERR_OPERATION_FAILED, - _("could not eject media on %s"), devname); + _("could not change media on %s"), devname); goto cleanup; } @@ -915,7 +915,7 @@ int qemuMonitorTextChangeMedia(qemuMonitorPtr mon, * No message is printed on success it seems */ if (strstr(reply, "device ")) { qemuReportError(VIR_ERR_OPERATION_FAILED, - _("could not eject media on %s: %s"), devname, reply); + _("could not change media on %s: %s"), devname, reply); goto cleanup; } -- 1.7.3.2

1 1

[libvirt] [PATCH] configure: Remove bashism and replace 'test ==' with 'test ='
by Matthias Bolte 13 Nov '10

13 Nov '10

This also fixes configure problems on FreeBSD, as test doesn't understand '==' there. --- configure.ac | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index f691c27..66237e1 100644 --- a/configure.ac +++ b/configure.ac @@ -269,9 +269,9 @@ fi AC_MSG_CHECKING([where to write libvirtd PID file]) AC_ARG_WITH([remote-pid-file], [AC_HELP_STRING([--with-remote-pid-file=@<:@pidfile|none@:>@], [PID file for libvirtd])]) -if test "x$with_remote_pid_file" == "x" ; then +if test "x$with_remote_pid_file" = "x" ; then REMOTE_PID_FILE="$localstatedir/run/libvirtd.pid" -elif test "x$with_remote_pid_file" == "xnone" ; then +elif test "x$with_remote_pid_file" = "xnone" ; then REMOTE_PID_FILE="" else REMOTE_PID_FILE="$with_remote_pid_file" @@ -1776,7 +1776,7 @@ if test "$with_python" != "no" ; then fi fi - if test "$with_python" == "yes" ; then + if test "$with_python" = "yes" ; then AM_PATH_PYTHON(,, [:]) if test "$PYTHON" != : ; then -- 1.7.0.4

2 3

[libvirt] [PATCH] virsh: Fix compile error in VPATH build
by Matthias Bolte 13 Nov '10

13 Nov '10

tools/console.c depends on daemon/event.h. --- tools/Makefile.am | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tools/Makefile.am b/tools/Makefile.am index b686045..921d621 100644 --- a/tools/Makefile.am +++ b/tools/Makefile.am @@ -54,6 +54,7 @@ virsh_CFLAGS = \ -I../include -I$(top_srcdir)/include \ -I$(top_srcdir)/src \ -I$(top_srcdir)/src/util \ + -I$(top_srcdir) \ -DGETTEXT_PACKAGE=\"$(PACKAGE)\" \ -DLOCALEBASEDIR=\""$(datadir)/locale"\" \ $(WARN_CFLAGS) \ -- 1.7.0.4

2 2