/etc/qemu-ifup /etc/qemu-ifdown assume the wrong location for brctl

Bug #833475 reported by David Medberry
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qemu-kvm (Ubuntu)
Fix Released
High
Serge Hallyn
Oneiric
Fix Released
Undecided
Unassigned

Bug Description

Launching kvm with -net nic -net tap triggers the /etc/qemu-ifup /etc/qemu-ifdown scripts. They are (in this version) hardcoding brctl as /usr/sbin/brctl but actually installed as /sbin/brctl.

/etc/qemu-ifup /etc/qmeu-ifdown are provided by qemu-common with source package of qemu-kvm

ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: qemu-common 0.14.1+noroms-0ubuntu2
ProcVersionSignature: Ubuntu 3.0.0-9.14-server 3.0.3
Uname: Linux 3.0.0-9-server x86_64
Architecture: amd64
CurrentDmesg:
 [ 18.522844] nf_conntrack version 0.5.0 (16384 buckets, 65536 max)
 [ 18.550476] ADDRCONF(NETDEV_UP): virbr0: link is not ready
 [ 18.872333] ip6_tables: (C) 2000-2006 Netfilter Core Team
 [ 25.430014] br0: no IPv6 routers present
Date: Wed Aug 24 19:32:00 2011
Dependencies:
 seabios 0.6.2-0ubuntu1
 vgabios 0.6c-2ubuntu3
InstallationMedia: Ubuntu-Server 11.10 "Oneiric Ocelot" - Alpha amd64 (20110802.1)
KvmCmdLine: Error: command ['ps', '-C', 'kvm', '-F'] failed with exit code 1: UID PID PPID C SZ RSS PSR STIME TTY TIME CMD
MachineType: ECS IC780M-A
PackageArchitecture: all
ProcEnviron:
 LANGUAGE=en_US:
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.0.0-9-server root=UUID=f2e68c5d-89b6-4c8f-b83a-e59f307cbf47 ro
SourcePackage: qemu-kvm
UpgradeStatus: No upgrade log present (probably fresh install)
dmi.bios.date: 04/19/2010
dmi.bios.vendor: American Megatrends Inc.
dmi.bios.version: 080015
dmi.board.name: IC780M-A
dmi.board.vendor: ECS
dmi.board.version: 1.0
dmi.chassis.type: 3
dmi.chassis.vendor: ECS
dmi.modalias: dmi:bvnAmericanMegatrendsInc.:bvr080015:bd04/19/2010:svnECS:pnIC780M-A:pvr1.0:rvnECS:rnIC780M-A:rvr1.0:cvnECS:ct3:cvr:
dmi.product.name: IC780M-A
dmi.product.version: 1.0
dmi.sys.vendor: ECS

Related branches

Revision history for this message
David Medberry (med) wrote :
Revision history for this message
David Medberry (med) wrote :

I suspect adding a dependency on bridge-utils may help in preventing more bugs like this.

I also see that /etc/qemu-ifup (and down) are symlinks to /usr/bin/qemu-ifup (and down).

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Looks like this was caused by the merge from Debian, where brctl was moved to /sbin at version 1.5-1. Marking Triaged, importance High. should be an easy enough fix.

Changed in qemu-kvm (Ubuntu):
status: New → Triaged
importance: Undecided → High
milestone: none → ubuntu-11.10-beta-2
Changed in qemu-kvm (Ubuntu):
assignee: nobody → Serge Hallyn (serge-hallyn)
status: Triaged → In Progress
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qemu-kvm - 0.14.1+noroms-0ubuntu3

---------------
qemu-kvm (0.14.1+noroms-0ubuntu3) oneiric; urgency=low

  * debian/patches/etc-qemuifscripts-fix-paths.patch: don't hardcode a path
    to brctl in qemu-ifup. (LP: #833475)
  * debian/control: move Depends: on bridge-utils from qemu-kvm to
    qemu-common. (LP: #835355)
  * debian/patches/debian/patches/fix-pa-configure.patch: fix FTBFS. Fix
    comes from upstream and will be in 0.15 when merged. (LP: #829492)
 -- Serge Hallyn <email address hidden> Mon, 29 Aug 2011 12:23:12 -0500

Changed in qemu-kvm (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Christophe Besson (christophe-besson) wrote :

Hi,

in oneiric, the version 0.14.1+noroms-0ubuntu6 of qemu-common still have the bug for qemu-ifdown (the path of brctl is wrong).

Moreover, it seems there is maybe a regression in qemu-kvm :
=> when using "script=no" when setting my NICs, the qemu scripts are not disabled... that's why I can see this bug :)

Regards

Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

The patch debian/patches/etc-qemuifscripts-fix-paths.patch doesn't appear to be in the source tree.

Changed in qemu-kvm (Ubuntu):
status: Fix Released → In Progress
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

Ah, sorry, it is there, its name however is different than listed in the changelog.

Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

@Christophe

the script does not specify a path for brctl any more. Would you mind filing a new bug for your issue?

Changed in qemu-kvm (Ubuntu):
status: In Progress → Fix Released
Changed in qemu-kvm (Ubuntu Oneiric):
status: New → Fix Released
Revision history for this message
Christophe Besson (christophe-besson) wrote :

I created a new bug here: #898234

Regards,
Christophe

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.