On 1/6/25 12:09, Roi Dayan via dev wrote: > This commit adds PIDFile to all systemd service file with Type=forking > in order to always have the correct PID to monitor. > > Related commit: > f385abded520 rhel: Use PIDFile on forking systemd service files > > Signed-off-by: Roi Dayan <[email protected]> > Acked-by: Eli Britstein <[email protected]> > --- > debian/openvswitch-switch.ovs-vswitchd.service | 1 + > debian/openvswitch-switch.ovsdb-server.service | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/debian/openvswitch-switch.ovs-vswitchd.service > b/debian/openvswitch-switch.ovs-vswitchd.service > index 519d80d8ede1..a4d445b953b6 100644 > --- a/debian/openvswitch-switch.ovs-vswitchd.service > +++ b/debian/openvswitch-switch.ovs-vswitchd.service > @@ -11,6 +11,7 @@ DefaultDependencies=no > [Service] > LimitNOFILE=1048576 > Type=forking > +PIDFile=/run/openvswitch/ovs-vswitchd.pid > Restart=on-failure > Environment=HOME=/var/run/openvswitch > EnvironmentFile=-/etc/default/openvswitch-switch > diff --git a/debian/openvswitch-switch.ovsdb-server.service > b/debian/openvswitch-switch.ovsdb-server.service > index 339665b25558..e0b0a727eede 100644 > --- a/debian/openvswitch-switch.ovsdb-server.service > +++ b/debian/openvswitch-switch.ovsdb-server.service > @@ -8,6 +8,7 @@ DefaultDependencies=no > [Service] > LimitNOFILE=1048576 > Type=forking > +PIDFile=/run/openvswitch/ovsdb-server.pid > Restart=on-failure > EnvironmentFile=-/etc/default/openvswitch-switch > ExecStart=/usr/share/openvswitch/scripts/ovs-ctl \
Thanks! Applied and backported down to 3.3. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
