2014-05-01 04:31:13 |
Hiroaki Kawai |
bug |
|
|
added bug |
2014-05-03 14:40:21 |
Launchpad Janitor |
openvswitch (Ubuntu): status |
New |
Confirmed |
|
2014-05-14 12:43:17 |
MGS |
bug |
|
|
added subscriber Martin GS |
2014-08-12 11:49:03 |
James Page |
openvswitch (Ubuntu): importance |
Undecided |
Medium |
|
2014-08-12 11:49:08 |
James Page |
openvswitch (Ubuntu): status |
Confirmed |
Triaged |
|
2014-08-19 21:16:51 |
Mike McCallister |
bug |
|
|
added subscriber Mike McCallister |
2015-06-14 15:28:24 |
Launchpad Janitor |
branch linked |
|
lp:~brian-turek/ubuntu/trusty/openvswitch/fix-for-1314887 |
|
2015-10-01 17:09:45 |
Serge Hallyn |
nominated for series |
|
Ubuntu Trusty |
|
2015-10-01 17:09:45 |
Serge Hallyn |
bug task added |
|
openvswitch (Ubuntu Trusty) |
|
2015-10-01 17:14:12 |
Serge Hallyn |
attachment added |
|
debdiff https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/1314887/+attachment/4481153/+files/debdiff |
|
2015-10-12 12:48:27 |
Brian Turek |
bug |
|
|
added subscriber Brian Turek |
2015-10-12 13:45:49 |
Brian Turek |
attachment added |
|
amended-fix-for-1314887.patch https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/1314887/+attachment/4492519/+files/amended-fix-for-1314887.patch |
|
2015-10-12 16:19:56 |
Ubuntu Foundations Team Bug Bot |
tags |
|
patch |
|
2015-10-12 16:20:06 |
Ubuntu Foundations Team Bug Bot |
bug |
|
|
added subscriber Ubuntu Review Team |
2015-10-15 00:48:39 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/wily-proposed/openvswitch |
|
2015-10-15 02:58:54 |
Launchpad Janitor |
openvswitch (Ubuntu): status |
Triaged |
Fix Released |
|
2015-10-15 19:47:36 |
Launchpad Janitor |
openvswitch (Ubuntu Trusty): status |
New |
Confirmed |
|
2015-10-15 19:56:02 |
Arata Notsu |
attachment added |
|
patch1314887-2.txt https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/1314887/+attachment/4496040/+files/patch1314887-2.txt |
|
2015-10-15 20:11:10 |
Arata Notsu |
bug |
|
|
added subscriber Arata Notsu |
2015-10-16 08:40:55 |
James Page |
openvswitch (Ubuntu): status |
Fix Released |
In Progress |
|
2015-10-16 08:40:57 |
James Page |
openvswitch (Ubuntu): importance |
Medium |
High |
|
2015-10-16 08:41:01 |
James Page |
openvswitch (Ubuntu Trusty): importance |
Undecided |
Medium |
|
2015-10-18 10:59:42 |
Launchpad Janitor |
openvswitch (Ubuntu): status |
In Progress |
Fix Released |
|
2015-11-10 21:42:31 |
Mathew Hodson |
branch linked |
|
lp:~brian-turek/ubuntu/trusty/openvswitch/fix-for-1314887 |
|
2015-11-10 21:47:25 |
Mathew Hodson |
tags |
patch |
patch trusty |
|
2015-12-07 11:48:40 |
Iain Lane |
openvswitch (Ubuntu Trusty): status |
Confirmed |
In Progress |
|
2015-12-09 14:06:34 |
Chris J Arges |
openvswitch (Ubuntu Trusty): status |
In Progress |
Fix Committed |
|
2015-12-09 14:06:37 |
Chris J Arges |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
2015-12-09 14:06:42 |
Chris J Arges |
bug |
|
|
added subscriber SRU Verification |
2015-12-09 14:06:47 |
Chris J Arges |
tags |
patch trusty |
patch trusty verification-needed |
|
2015-12-10 02:02:22 |
Mathew Hodson |
description |
ifupdown hook is present in /etc/init.d/openvswitch-switch, but not in /etc/init/openvswitch-switch.conf which is the default.
=== modified file 'debian/openvswitch-switch.upstart'
--- debian/openvswitch-switch.upstart 2013-08-23 11:18:52 +0000
+++ debian/openvswitch-switch.upstart 2014-05-01 04:26:00 +0000
@@ -32,11 +32,13 @@
fi
set "$@" $OVS_CTL_OPTS
"$@"
+ ifup --allow ovs `ifquery --allow ovs --list`
end script
post-stop script
. /usr/share/openvswitch/scripts/ovs-lib
test -e /etc/default/openvswitch-switch && . /etc/default/openvswitch-switch
+ ifdown --allow ovs `ifquery --allow ovs --list`
ovs_ctl stop
end script |
[Impact]
The documentation included in both upstream Open vSwitch (ovs) 2.0.2 and the version distributed by Ubuntu states that it supports configuring ovs using /etc/network/interfaces (see /usr/share/doc/openvswitch-switch/README.Debian). Moreover, /etc/init.d/openvswitch-switch contains the necessary logic to support parsing /etc/network/interfaces but that logic is completely missing in the Upstart /etc/init/openvswitch-switch.conf script.
The result of this issue is a user can properly configure their network in /etc/network/interfaces, test it with ifup/ifdown, reboot, and have no network configured with the machine comes back up. If the user is unaware that Upstart is being used and not classic init.d scripts, a manual call using /etc/init.d/openvswitch-switch works as expected. Without boot integration, users can't easily use Open vSwitch to configure their "main" interface and have a system work on boot.
The fix in this thread basically adds support for using the normal config files (/etc/network/interfaces and /etc/network/interfaces.d/*) to configure the network on boot.
[Test Case]
After running `apt-get install -y openvswitch-switch`, add a stanza to /etc/network/interfaces like the following:
allow-ovs br0
iface br0 inet static
ovs_type OVSBridge
address 172.16.0.1
network 255.255.255.0
Try `ifup br0` and `ifdown br0` and notice how the interface is correctly controlled. Try `/etc/init.d/openvswitch-switch restart` and note how the interface comes up. Reboot the machine and note how br0 is not correctly configured.
[Regression Potential]
This problem is recognized by the user community (see http://www.opencloudblog.com/?p=240, http://blog.scottlowe.org/2012/08/17/installing-kvm-and-open-vswitch-on-ubuntu/, etc) and people have been manually patching their systems for years to get the desired capability. It's conceivable people's odd ways of fixing this problem could conflict with the potential patch. Double ifup'ing or ifdown'ing an interface shouldn't break any systems but error messages may be emitted.
The other unlikely issue is that users have interfaces correctly configured in /etc/network/interfaces and are deliberately depending on the interfaces *not* coming up on boot. This patch would cause these boot-dormant interfaces to come up unexpectedly but, again, I think this problem unlikely.
[Other Info]
This patch aligns the boot behavior both with what's expected and what the documentation states. |
|
2015-12-10 22:13:05 |
Brian Turek |
tags |
patch trusty verification-needed |
patch trusty verification-done |
|
2015-12-16 14:45:04 |
Launchpad Janitor |
openvswitch (Ubuntu Trusty): status |
Fix Committed |
Fix Released |
|
2015-12-16 14:45:11 |
Chris J Arges |
removed subscriber Ubuntu Stable Release Updates Team |
|
|
|