[ovs-dev] [PATCH 1/2] rhel: Don't delete bridges on boot.

[Ben Pfaff]

Yes, that's the right thing to do.  openvswitch.booted is only used by
the init script itself.

On Wed, Aug 31, 2011 at 05:21:22PM -0700, Ethan Jackson wrote:
> Summary of an offline discussion:
> 
> Did some research and we decided to just remove it.  I'm going to push
> this series now.
> 
> Ethan
> 
> On Wed, Aug 31, 2011 at 17:15, Ethan Jackson <ethan at nicira.com> wrote:
> > I'm not sure if that's some sort of rhel requirement so I wanted to be
> > safe. ?If you think it's ok I'll remove it.
> >
> > Ethan
> >
> > On Wed, Aug 31, 2011 at 17:13, Justin Pettit <jpettit at nicira.com> wrote:
> >> I think you can get rid of the "/var/run/openvswitch.booted" stuff surrounding it.
> >>
> >> --Justin
> >>
> >>
> >> On Aug 31, 2011, at 5:11 PM, Ethan Jackson wrote:
> >>
> >>> Before this patch, the RHEL init scripts would remove all bridges
> >>> configured on the system at boot.
> >>> ---
> >>> rhel/etc_init.d_openvswitch | ? ?1 -
> >>> 1 files changed, 0 insertions(+), 1 deletions(-)
> >>>
> >>> diff --git a/rhel/etc_init.d_openvswitch b/rhel/etc_init.d_openvswitch
> >>> index c7543cc..c4af816 100755
> >>> --- a/rhel/etc_init.d_openvswitch
> >>> +++ b/rhel/etc_init.d_openvswitch
> >>> @@ -50,7 +50,6 @@ start () {
> >>> ? ? fi
> >>> ? ? if test ! -e/var/run/openvswitch.booted; then
> >>> ? ? ? ? touch /var/run/openvswitch.booted
> >>> - ? ? ? ?set "$@" --delete-bridges
> >>> ? ? fi
> >>> ? ? "$@"
> >>>
> >>> --
> >>> 1.7.6
> >>>
> >>> _______________________________________________
> >>> dev mailing list
> >>> dev at openvswitch.org
> >>> http://openvswitch.org/mailman/listinfo/dev
> >>
> >>
> >
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev