On 6 October 2017 at 15:27, Ben Pfaff <b...@ovn.org> wrote: > On Fri, Oct 06, 2017 at 04:08:23AM -0700, Gurucharan Shetty wrote: > > We still use SysV scripts for RHEL. Currently, invoking > > /etc/init.d/openvswitch will redirect the calls to > > dynamically generated systemd scripts. In the above case when you call > > "/etc/init.d/openvswitch-switch start", it inturn calls > > "/bin/systemctl start openvswitch-switch.service" and > > that inturn again calls "/etc/init.d/openvswitch-switch start" > > > > This patch avoids it. This is similar to what was done to > > Debian in commit 873d85653d8 (debian: Skip systemctl redirect.) > > > > Signed-off-by: Gurucharan Shetty <g...@ovn.org> > > --- > > rhel/etc_init.d_openvswitch | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/rhel/etc_init.d_openvswitch b/rhel/etc_init.d_openvswitch > > index 3921aab..caf24cc 100755 > > --- a/rhel/etc_init.d_openvswitch > > +++ b/rhel/etc_init.d_openvswitch > > @@ -27,6 +27,8 @@ > > # Short-Description: Open vSwitch switch > > ### END INIT INFO > > > > +SYSTEMCTL_SKIP_REDIRECT=yes > > Thank you for finding this and fixing it. > > The Debian scripts set this variable and another one with an extra > leading _. I don't know the reason. Should the RHEL script do the > same? > The Debian did both because there were some systems were systemd would only accept one with "_". I haven't tested this across multiple RHEL systems. My idea is that if we find one, we will add another commit.
> > Acked-by: Ben Pfaff <b...@ovn.org> > Thanks. I applied this to master and will backport to relevent branches. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev