From: Marius Bakke <marius@gnu.org>
To: Lars-Dominik Braun <lars@6xq.net>, 57272-done@debbugs.gnu.org
Subject: bug#57272: libvirt 8.6 fails to start network
Date: Mon, 29 Aug 2022 21:17:31 +0200 [thread overview]
Message-ID: <87k06qdflg.fsf@gnu.org> (raw)
In-Reply-To: <Yv30OJt42lzGRydF@noor.fritz.box>
[-- Attachment #1: Type: text/plain, Size: 2656 bytes --]
Hi, thanks for the bug report, and sorry for the breakage.
Lars-Dominik Braun <lars@6xq.net> skriver:
> Hi,
>
> after the update to libvirt 8.6.0 in
> 3a76c2bfd94557c9776aa11240fec14580aec1b0 networks don’t start any more:
>
> > LANG=C virsh net-start default
> error: Failed to start network default
> error: Unable to find 'dnsmasq' binary in $PATH: No such file or directory
>
> I tried to patch dnsmasq’s path like follows, but then the testcase
> networkxml2conftest fails and cannot find dnsmasq either.
I don't understand why that test is failing with your patch. It has a
hard coded "/usr/sbin/dnsmasq", but patching that does not make a
difference. It seems the lookup via virFindFileInPath, which ultimately
calls out to GLib's g_find_program_in_path, fails; but ostensibly using
an absolute file name should not make a difference[0].
[0]: https://docs.gtk.org/glib/func.find_program_in_path.html
Anyway, I was able to reproduce the failure in the system test:
--8<---------------cut here---------------start------------->8---
diff --git a/gnu/tests/virtualization.scm b/gnu/tests/virtualization.scm
index 4bd56e5d9d..557f30db4f 100644
--- a/gnu/tests/virtualization.scm
+++ b/gnu/tests/virtualization.scm
@@ -106,6 +106,26 @@ (define marionette
"-c" "qemu:///system" "connect"))
marionette))
+ (test-eq "create default network"
+ 0
+ (marionette-eval
+ '(begin
+ (chdir "/tmp")
+ (system* #$(file-append libvirt "/bin/virsh")
+ "-c" "qemu:///system" "net-define"
+ #$(file-append libvirt
+ "/etc/libvirt/qemu/networks/default.xml")))
+ marionette))
+
+ (test-eq "start default network"
+ 0
+ (marionette-eval
+ '(begin
+ (chdir "/tmp")
+ (system* #$(file-append libvirt "/bin/virsh")
+ "-c" "qemu:///system" "net-start" "default"))
+ marionette))
+
(test-end))))
(gexp->derivation "libvirt-test" test))
--8<---------------cut here---------------end--------------->8---
And can confirm your patch fixes that, which is arguably more important.
So I went ahead and committed your patch (and disabled the failing
test), and the system test to avoid future regressions in this area.
Fixed in:
acbf2f9def gnu: libvirt: Use absolute dnsmasq.
3e0abde17b tests: libvirt: Ensure the default network can be started.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]
prev parent reply other threads:[~2022-08-29 19:18 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-18 8:11 bug#57272: libvirt 8.6 fails to start network Lars-Dominik Braun
2022-08-29 19:17 ` Marius Bakke [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k06qdflg.fsf@gnu.org \
--to=marius@gnu.org \
--cc=57272-done@debbugs.gnu.org \
--cc=lars@6xq.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.