I added a bit more text and pushed the change to master. I'm going to cherry-pick this change into branch-1.2 and branch-1.3, too.
--Justin On Oct 25, 2011, at 10:48 AM, Ben Pfaff wrote: > That's great, thanks. Can you add some of that to the commit message? > > On Mon, Oct 24, 2011 at 11:35:13PM -0700, Justin Pettit wrote: >> The init scripts end up having the same start number, but I did a bit more >> research and according to what I've read, it should be correct. This page >> gets into specifics: >> >> http://wiki.debian.org/LSBInitScripts/DependencyBasedBoot >> >> I produced dependency graphs with both the old and new way, and it looks >> like this change corrects the behavior. >> >> --Justin >> >> >> On Oct 24, 2011, at 11:21 PM, Ben Pfaff wrote: >> >>> I thought you said earlier today that this didn't have the desired >>> effect? (Or maybe you just hadn't been able to test it yet.) >>> >>> On Mon, Oct 24, 2011 at 11:19:46PM -0700, Justin Pettit wrote: >>>> ovs-monitor-ipsec uses the OVS database to get configuration, so don't >>>> bother starting the daemon until it's up. >>>> --- >>>> debian/openvswitch-ipsec.init | 2 +- >>>> 1 files changed, 1 insertions(+), 1 deletions(-) >>>> >>>> diff --git a/debian/openvswitch-ipsec.init b/debian/openvswitch-ipsec.init >>>> index 17835a5..125b50f 100755 >>>> --- a/debian/openvswitch-ipsec.init >>>> +++ b/debian/openvswitch-ipsec.init >>>> @@ -19,7 +19,7 @@ >>>> # >>>> ### BEGIN INIT INFO >>>> # Provides: openvswitch-ipsec >>>> -# Required-Start: $network $local_fs $remote_fs >>>> +# Required-Start: $network $local_fs $remote_fs openvswitch-switch >>>> # Required-Stop: $remote_fs >>>> # Default-Start: 2 3 4 5 >>>> # Default-Stop: 0 1 6 >>>> -- >>>> 1.7.1 >>>> >>>> _______________________________________________ >>>> dev mailing list >>>> [email protected] >>>> http://openvswitch.org/mailman/listinfo/dev >> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
