On Fri, Jan 10, 2025 at 6:46 PM Ilya Maximets <[email protected]> wrote: > > On 1/6/25 12:09, Roi Dayan via dev wrote: > > From: Han Zhou <[email protected]> > > > > This causes the /var/run/openvswitch directory being recreated when > > service "stop" encounters failures, which causes container mount > > failure after service restart of openvswitch-switch on Ubuntu. > > > > The same was done in rhel service. See [1]. > > > > [1] 7a65e5a9252a rhel: let *-ctl handle runtime directory > > > > Signed-off-by: Han Zhou <[email protected]> > > Acked-by: Roi Dayan <[email protected]> > > --- > > debian/openvswitch-switch.ovsdb-server.service | 3 --- > > 1 file changed, 3 deletions(-) > > > > diff --git a/debian/openvswitch-switch.ovsdb-server.service > > b/debian/openvswitch-switch.ovsdb-server.service > > index e0b0a727eede..35654d7059d1 100644 > > --- a/debian/openvswitch-switch.ovsdb-server.service > > +++ b/debian/openvswitch-switch.ovsdb-server.service > > @@ -19,6 +19,3 @@ ExecStop=/usr/share/openvswitch/scripts/ovs-ctl > > --no-ovs-vswitchd stop > > ExecReload=/usr/share/openvswitch/scripts/ovs-ctl --no-ovs-vswitchd \ > > --no-record-hostname \ > > --no-monitor restart $OVS_CTL_OPTS > > -RuntimeDirectory=openvswitch > > -RuntimeDirectoryMode=0755 > > -RuntimeDirectoryPreserve=yes > > This seems reasonable to me. > > Frode, what do you think?
LGTM. Acked-by: Frode Nordahl <[email protected]> -- Frode Nordahl > Best regards, Ilya Maximets. > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
