Ack, review only. /Thanks HansN

-----Original Message-----
From: Alex Jones [mailto:alex.jo...@genband.com] 
Sent: den 14 september 2017 16:14
To: mathi.np....@gmail.com; Anders Widell <anders.wid...@ericsson.com>
Cc: Alex Jones <alex.jo...@genband.com>; opensaf-devel@lists.sourceforge.net
Subject: [devel] [PATCH 1/1] plm: fix systemd files for plm and opensaf [#2581]

plmc does not send TERMINATED message to active controller during reboot.

This is because network-online.target is not pulled in explicitly by the PLM 
systemd service files, so networking may already be down when the TERMINATED is 
sent.

Update systemd service files to pull in network-online.target explicitly.
---
 src/nid/opensafd.service.in       | 3 ++-
 src/plm/plmcd/plmcboot.service.in | 3 ++-
 src/plm/plmcd/plmcd.service.in    | 2 +-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/nid/opensafd.service.in b/src/nid/opensafd.service.in index 
5be15fe..2698f79 100644
--- a/src/nid/opensafd.service.in
+++ b/src/nid/opensafd.service.in
@@ -1,6 +1,7 @@
 [Unit]
 Description=OpenSAF daemon
-After=remote-fs.target local-fs.target syslog.target network.target
+After=remote-fs.target local-fs.target syslog.target 
+network-online.target Wants=network-online.target
 
 [Service]
 ExecStart=@initscriptdir@/opensafd start diff --git 
a/src/plm/plmcd/plmcboot.service.in b/src/plm/plmcd/plmcboot.service.in
index df49907..be69de2 100644
--- a/src/plm/plmcd/plmcboot.service.in
+++ b/src/plm/plmcd/plmcboot.service.in
@@ -1,6 +1,7 @@
 [Unit]
 Description=Send a UDP datagram to OpenSAF PLM controllers indicating the OS 
state -After=remote-fs.target local-fs.target syslog.target network.target
+After=remote-fs.target local-fs.target syslog.target 
+network-online.target Wants=network-online.target
 
 [Service]
 ExecStart=@sbindir@/plmcd -c @sysconfdir@/plmcd.conf -s diff --git 
a/src/plm/plmcd/plmcd.service.in b/src/plm/plmcd/plmcd.service.in index 
b6013f1..4800f8d 100644
--- a/src/plm/plmcd/plmcd.service.in
+++ b/src/plm/plmcd/plmcd.service.in
@@ -1,6 +1,6 @@
 [Unit]
 Description=OpenSAF PLMC Daemon
-After=local-fs.target remote-fs.target syslog.target network.target 
plmcboot.service
+After=local-fs.target remote-fs.target syslog.target 
+network-online.target plmcboot.service
 Before=opensafd.service
 Wants=plmcboot.service
 
--
2.9.5


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to