Bug 532834

Summary: bridges created by libvirtd are not started unless the host has an IP
Product: Red Hat Enterprise Linux 5 Reporter: Charles Duffy <charles_duffy>
Component: libvirtAssignee: Laine Stump <laine>
Status: CLOSED ERRATA QA Contact: Virtualization Bugs <virt-bugs>
Severity: medium Docs Contact:
Priority: low    
Version: 5.4CC: gren, hbrock, llim, markmc, veillard, virt-maint, xen-maint, yoyzhang
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-03-30 08:11:04 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Upstream fix for this issue
none
patch rebased to RHEL-5 libvirt none

Description Charles Duffy 2009-11-04 00:30:11 UTC
Created attachment 367379 [details]
Upstream fix for this issue

Description of problem:

  The bridge associated with a network created and started with virsh net-define and virsh-start is not actually enabled ("ip link set DEVICE up") if the host has no IP on that bridge.

Version-Release number of selected component (if applicable):

  libvirt-0.6.3-20.1

How reproducible:

  Every time.

Steps to Reproduce:
1. virsh net-define <(echo "<network><name>test_noip</name><uuid>9533aa79-1064-449d-8b43-1b873e5e52d4</uuid><bridge name='test_noip_br'/></network>")
2. virsh net-start test_noip
3. ip -o link show test_noip_br | grep -q UP && echo 'Pass' || echo 'Fail'
  
Actual results:

  Above steps should emit 'Pass'.

Expected results:

  Given test emits 'Fail'; bridge does not pass packets.

Additional info:

  Fixed in upstream libvirt commit e978774ec67f4b062b1f65e5c76a13193a9430eb; patch (which applies cleanly except for the changelog entry) attached for reference.

Comment 1 Mark McLoughlin 2009-11-09 17:28:59 UTC
Charles supplied a pointer to the upstream fix, think this can move to POST

Comment 2 Laine Stump 2009-11-30 22:14:15 UTC
Created attachment 374897 [details]
patch rebased to RHEL-5 libvirt

This is the upstream patch newly rebased to the RHEL-5 libvirt-0.6.3. I built and tested, and it does now pass the test.

Comment 3 Daniel Veillard 2009-12-04 15:33:33 UTC
libvirt-0.6.3-23.el5 has been built in dist-5E-qu-candidate with
the patches,

Daniel

Comment 5 Gunannan Ren 2009-12-31 08:19:30 UTC
The bugs has been fixed in libvirt-0.6.3-28.el5

virsh net-dumpxml test_noip

<network>
  <name>test_noip</name>
  <uuid>9d92939c-208a-9cec-5ff9-86b6645ca2a5</uuid>
  <bridge name='test_noip_br' stp='on' forwardDelay='0' />
</network>

ip -o link show test_noip_br

7: test_noip_br: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue \    link/ether 00:00:00:00:00:00 brd ff:ff:ff:ff:ff:ff

Comment 10 errata-xmlrpc 2010-03-30 08:11:04 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2010-0205.html