Re: [ovs-dev] [PATCH] debian: Add an upstart job file

2014-08-01 Thread Ben Pfaff
On Wed, Jul 30, 2014 at 03:36:27PM -0600, Andrew James wrote: > Following the guidelines also involves short-circuiting the sysv > init script with upstart_is_init. This makes it difficult or > impossible to use the sysv init script from the upstart job. Given > that, would a v2 patch be welcome th

Re: [ovs-dev] [PATCH] debian: Add an upstart job file

2014-07-30 Thread Gurucharan Shetty
>> If the primary goal is to just emit events for other upstart jobs, >> then it would be nice if there is some sort of >> integration between the upstart job and /etc/init.d/openvswitch-switch. >> (I do not know how upstart jobs work very well, so I may need more >> details while you respond.) > >

Re: [ovs-dev] [PATCH] debian: Add an upstart job file

2014-07-30 Thread Andrew James
On 07/30/2014 12:45 PM, Gurucharan Shetty wrote: On Wed, Jul 30, 2014 at 11:19 AM, Andrew James wrote: My goal here was to support using OVS with upstart on Debian. I'm certainly open to the possibility that this belongs in the Debian package more than here. Upstream debian package for OVS is

Re: [ovs-dev] [PATCH] debian: Add an upstart job file

2014-07-30 Thread Gurucharan Shetty
On Wed, Jul 30, 2014 at 11:19 AM, Andrew James wrote: > My goal here was to support using OVS with upstart on Debian. I'm certainly > open to the possibility that this belongs in the Debian package more than > here. Upstream debian package for OVS is maintained by Ben Pfaff and I think he simply u

Re: [ovs-dev] [PATCH] debian: Add an upstart job file

2014-07-30 Thread Andrew James
My goal here was to support using OVS with upstart on Debian. I'm certainly open to the possibility that this belongs in the Debian package more than here. I did test this on Debian (without the noted issues). Obviously, Ubuntu needed testing too (issues confirmed on 12.04, btw). Please let me k

Re: [ovs-dev] [PATCH] debian: Add an upstart job file

2014-07-25 Thread Gurucharan Shetty
On Thu, Jul 24, 2014 at 2:14 PM, Andrew James wrote: > This adds the upstart job configuration file written by James Page from > Ubuntu's openvswitch-2.0.1+git20140120 package plus the changes proposed in > https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/1314888 > > Signed-off-by: Andre

Re: [ovs-dev] [PATCH] debian: Add an upstart job file

2014-07-25 Thread Ben Pfaff
On Thu, Jul 24, 2014 at 03:14:02PM -0600, Andrew James wrote: > This adds the upstart job configuration file written by James Page from > Ubuntu's openvswitch-2.0.1+git20140120 package plus the changes proposed in > https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/1314888 > > Signed-off-

[ovs-dev] [PATCH] debian: Add an upstart job file

2014-07-24 Thread Andrew James
This adds the upstart job configuration file written by James Page from Ubuntu's openvswitch-2.0.1+git20140120 package plus the changes proposed in https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/1314888 Signed-off-by: Andrew James --- debian/openvswitch-switch.upstart | 47