Re: [ovs-dev] [PATCH V3] hotplug: add openvswitch script

2013-04-23 Thread Bastian Blank
On Tue, Apr 23, 2013 at 10:42:15AM +0100, Ian Campbell wrote: > On Tue, 2013-04-23 at 09:56 +0100, Roger Pau Monne wrote: > > Also, it might be interesting to add something like: > > if ! type ovs-vsctl > /dev/null 2>&1; then > I think type is a bash-ism but I don't know the portable alternativ

[ovs-dev] Bug#684056: openvswitch

2012-08-06 Thread Bastian Blank
Package: openvswitch-switch Version: 1.4.2+git20120612-8 Severity: grave Subject: openvswitch-switch - Does not start with upstream openvswitch Package: openvswitch-switch Version: 1.4.2+git20120612-8 Severity: grave -- System Information: Debian Release: wheezy/sid APT prefers testing APT

[ovs-dev] Bug#684057: openvswitch-switch - Does not start with upstream openvswitch module

2012-08-06 Thread Bastian Blank
Package: openvswitch-switch Version: 1.4.2+git20120612-8 Severity: grave The openvswitch-switch init script fails if openvswitch included in Linux should be used. | # invoke-rc.d openvswitch-switch start | FATAL: Module openvswitch_mod not found. | Inserting openvswitch module ... failed! | Modul

[ovs-dev] Bug#680536: openvswitch - Fails to start without /etc/protocols

2012-07-06 Thread Bastian Blank
Package: openvswitch Version: 1.4.2+git20120612-2 Severity: serious The openvswitch-switch init script fails to start stuff if /etc/protocols is missing. This file is provided by the non-essential netbase, so it needs a dependency. Bastian -- System Information: Debian Release: wheezy/sid APT

[ovs-dev] Bug#680537: openvswitch - Plays with iptables without consent

2012-07-06 Thread Bastian Blank
Package: openvswitch Version: 1.4.2+git20120612-2 Severity: serious The openvswitch-switch init script modifies filter rules using iptables without consent by the admin. Bastian -- System Information: Debian Release: wheezy/sid APT prefers testing APT policy: (990, 'testing'), (500, 'unstabl

[ovs-dev] Bug#680538: openvswitch - brcompat unable to overtake bridge configured by ifupdown

2012-07-06 Thread Bastian Blank
Package: openvswitch-brcompat Version: 1.4.2+git20120612-2 Severity: normal The standard way to configure bridges in Debian is using /etc/network/interfaces. This is set up much earlier than the openvswitch-switch init script is run, so this won't work. Bastian -- System Information: Debian Rele

[ovs-dev] Bug#680537: closed by Ben Pfaff (Bug#680537: fixed in openvswitch 1.4.2+git20120612-3)

2012-07-10 Thread Bastian Blank
reopen 680537 thanks On Mon, Jul 09, 2012 at 07:36:09PM +, Debian Bug Tracking System wrote: > #680537: openvswitch - Plays with iptables without consent >* debian/control: Add "netbase" dependency to ensure that /etc/protocols > is available. Closes: #680537.

[ovs-dev] Bug#680537: openvswitch - Plays with iptables without consent

2012-07-10 Thread Bastian Blank
On Fri, Jul 06, 2012 at 03:16:03PM -0700, Ben Pfaff wrote: > On Fri, Jul 06, 2012 at 06:13:25PM +0200, Bastian Blank wrote: > > The openvswitch-switch init script modifies filter rules using iptables > > without consent by the admin. > Where are the requirements spelled out?

[ovs-dev] Bug#681880: openvswitch-switch - Automatic changed file in /etc/

2012-07-17 Thread Bastian Blank
Package: openvswitch-switch Version: 1.4.2+git20120612-2 Severity: serious openvswitch uses a db called /etc/openvswitch/conf.db. This file is programmatic modified and not user editable. This violates §10.7 of the policy. /var/lib is the correct location for state data. Bastian -- System Infor

[ovs-dev] Bug#681954: openvswitch-brcompat - Init script switch and different package

2012-07-18 Thread Bastian Blank
Package: openvswitch-brcompat Version: 1.4.2+git20120612-2 Severity: normal Installing openvswitch-brcompat is not enough to actually enable it. There is an additional switch in /etc/defaults/openvswitch-switch to enable it. Bastian -- System Information: Debian Release: wheezy/sid APT prefers

[ovs-dev] Bug#681955: openvswitch-switch - Enabling brcompat breaks normal operation

2012-07-18 Thread Bastian Blank
Package: openvswitch-brcompat Version: 1.4.2+git20120612-2 Severity: serious Enabling brcompat without openvswitch-brcompat installed or the module available breaks the init script and normal operation without brcompat. Bastian -- System Information: Debian Release: wheezy/sid APT prefers test

[ovs-dev] Bug#681880: openvswitch-switch - Automatic changed file in /etc/

2012-07-18 Thread Bastian Blank
On Tue, Jul 17, 2012 at 09:31:44AM -0700, Ben Pfaff wrote: > On Tue, Jul 17, 2012 at 03:20:40PM +0200, Bastian Blank wrote: > > openvswitch uses a db called /etc/openvswitch/conf.db. This file is > > programmatic modified and not user editable. This violates §10.7 of the > >

[ovs-dev] Bug#682384: odd s390 build failure for openvswitch

2012-07-22 Thread Bastian Blank
Package: openvswitch Version: 1.4.2+git20120612-5 Severity: serious On Sat, Jul 21, 2012 at 09:36:07PM -0700, Ben Pfaff wrote: > The s390 build of openvswitch 1.4.2+git20120612-5 failed for a reason > that I don't understand: > > https://buildd.debian.org/status/fetch.php?pkg=openvswitch&

[ovs-dev] Bug#681955: openvswitch-switch - Enabling brcompat breaks normal operation

2012-07-26 Thread Bastian Blank
On Wed, Jul 18, 2012 at 06:50:10AM -0700, Ben Pfaff wrote: > On Wed, Jul 18, 2012 at 09:57:13AM +0200, Bastian Blank wrote: > > Enabling brcompat without openvswitch-brcompat installed or the module > > available breaks the init script and normal operation without brcompat. > I&#

[ovs-dev] Bug#681880: openvswitch-switch - Automatic changed file in /etc/

2012-07-26 Thread Bastian Blank
On Wed, Jul 18, 2012 at 07:01:42AM -0700, Ben Pfaff wrote: > On Wed, Jul 18, 2012 at 10:00:49AM +0200, Bastian Blank wrote: > > No. It is no configuration file if it is not static. > The FHS defines "static" as: > "Static" files include binaries, libraries,

[ovs-dev] Bug#681954: openvswitch-brcompat - Init script switch and different package

2012-07-26 Thread Bastian Blank
On Wed, Jul 18, 2012 at 06:47:24AM -0700, Ben Pfaff wrote: > On Wed, Jul 18, 2012 at 09:55:56AM +0200, Bastian Blank wrote: > > Installing openvswitch-brcompat is not enough to actually enable it. > > There is an additional switch in /etc/defaults/openvswitch-switch to > &

Re: [ovs-dev] [bug 681880 1/3] lockfile: Fix hang locking through a dangling symlink.

2012-07-26 Thread Bastian Blank
On Thu, Jul 26, 2012 at 02:48:51PM -0700, Ben Pfaff wrote: > open() with O_CREAT|O_EXCL yields EEXIST if the file being opened is a > symlink. lockfile_try_lock() interpreted that error code to mean that > some other process had created the lock file in the meantime, so it went > around its loop a

Re: [ovs-dev] [bug 681880 1/3] lockfile: Fix hang locking through a dangling symlink.

2012-07-27 Thread Bastian Blank
On Thu, Jul 26, 2012 at 11:53:25PM -0700, Ben Pfaff wrote: > On Fri, Jul 27, 2012 at 08:39:42AM +0200, Bastian Blank wrote: > > I would always use the lock file at the target location of the database > > and not follow symlinks there. Otherwise more than one lock file can > >

Re: [ovs-dev] [bug 681880 1/3] lockfile: Fix hang locking through a dangling symlink.

2012-07-27 Thread Bastian Blank
On Fri, Jul 27, 2012 at 10:21:10AM -0700, Ben Pfaff wrote: > On Fri, Jul 27, 2012 at 10:28:08AM +0200, Bastian Blank wrote: > > I was unclear. You should first use realpath(3) on the database filename > > and calculate the lock file from there. Otherwise there may be several > &g