On Tue, Jul 31, 2018 at 10:27:59AM +0200, Jakub Sitnicki wrote:
> On Mon, 30 Jul 2018 09:47:45 -0400
> Mark Michelson <mmich...@redhat.com> wrote:
> 
> > The --restart flag allows for uninterrupted packet flowage when exiting
> > ovn-controller. This patch modifies the restart_controller argument to
> > ovn-ctl to use --restart.
> > 
> > Signed-off-by: Mark Michelson <mmich...@redhat.com>
> > ---
> >  ovn/utilities/ovn-ctl | 4 ++--
> >  utilities/ovs-lib.in  | 2 +-
> >  2 files changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl
> > index 2fce47714..05feed2b6 100755
> > --- a/ovn/utilities/ovn-ctl
> > +++ b/ovn/utilities/ovn-ctl
> > @@ -350,7 +350,7 @@ stop_northd () {
> >  }
> >  
> >  stop_controller () {
> > -    OVS_RUNDIR=${OVN_RUNDIR} stop_daemon ovn-controller
> > +    OVS_RUNDIR=${OVN_RUNDIR} stop_daemon ovn-controller $@
> >  }
> 
> It needs to be "$@" or the arguments containing whitespace (or
> characters from IFS) will get split up :-C

I made that change and pushed this series to master and branch-2.10.
Thanks Mark and Jakub!
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to