summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatt Spinler <spinler@us.ibm.com>2022-04-08 00:23:47 +0300
committerJoel Stanley <joel@jms.id.au>2022-05-02 12:04:49 +0300
commit707c38c02777388cad53a37306361c72855446ce (patch)
treec95ce39b383adf1dadb8f8e9730bea88297a7040
parentc1c2370ca17d8d10c50e00b024da411f63f3b952 (diff)
downloadopenbmc-707c38c02777388cad53a37306361c72855446ce.tar.xz
meta-openpower: Use proper paths in service files
Using /usr/bin/env is an antipattern. Signed-off-by: Matt Spinler <spinler@us.ibm.com> Change-Id: I6b4f647a10fd8c261e2b42cfdae030749ded7c29
-rw-r--r--meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-checkstop@.service3
-rw-r--r--meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog-timeout@.service3
-rw-r--r--meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog@.service3
-rw-r--r--meta-openpower/recipes-phosphor/fsi/op-fsi/fsi-scan@.service3
-rw-r--r--meta-openpower/recipes-phosphor/host/op-host-control/cfam_override@.service3
-rw-r--r--meta-openpower/recipes-phosphor/host/op-host-control/start_host@.service3
-rw-r--r--meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-disable@.service3
-rw-r--r--meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-enable@.service3
-rw-r--r--meta-openpower/recipes-phosphor/occ/openpower-occ-control/org.open_power.OCC.Control.service3
-rw-r--r--meta-openpower/recipes-phosphor/vpd/openpower-fru-vpd/op-vpd-parser.service3
10 files changed, 10 insertions, 20 deletions
diff --git a/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-checkstop@.service b/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-checkstop@.service
index 5e8cf3dcdb..bac565bc11 100644
--- a/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-checkstop@.service
+++ b/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-checkstop@.service
@@ -7,7 +7,6 @@ Before=obmc-host-quiesce@%i.target
# before processing checkstop. OCC has to wait 5 seconds
# for our driver to stop using the SBEFIFO.
ExecStart={base_bindir}/sleep 10
-ExecStart=/usr/bin/env checkstop_app
-SyslogIdentifier=checkstop_app
+ExecStart=/usr/bin/checkstop_app
Type=oneshot
Restart=no
diff --git a/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog-timeout@.service b/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog-timeout@.service
index 55ba40351c..4f9870f38b 100644
--- a/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog-timeout@.service
+++ b/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog-timeout@.service
@@ -3,7 +3,6 @@ Description=OpenPOWER debug data collector for host %I watchdog timeout
Before=obmc-host-quiesce@%i.target
[Service]
-ExecStart=/usr/bin/env openpower-proc-control collectSBEHBData
-SyslogIdentifier=openpower-proc-control
+ExecStart=/usr/bin/openpower-proc-control collectSBEHBData
Type=oneshot
Restart=no
diff --git a/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog@.service b/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog@.service
index 6826764ff7..ca4e4af92f 100644
--- a/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog@.service
+++ b/meta-openpower/recipes-phosphor/debug/openpower-debug-collector/openpower-debug-collector-watchdog@.service
@@ -4,7 +4,6 @@ Before=obmc-host-quiesce@%i.target
After=op-stop-instructions@%i.service
[Service]
-ExecStart=/usr/bin/env watchdog_timeout
-SyslogIdentifier=watchdog_timeout
+ExecStart=/usr/bin/watchdog_timeout
Type=oneshot
Restart=no
diff --git a/meta-openpower/recipes-phosphor/fsi/op-fsi/fsi-scan@.service b/meta-openpower/recipes-phosphor/fsi/op-fsi/fsi-scan@.service
index 00676798db..1bf86aa785 100644
--- a/meta-openpower/recipes-phosphor/fsi/op-fsi/fsi-scan@.service
+++ b/meta-openpower/recipes-phosphor/fsi/op-fsi/fsi-scan@.service
@@ -11,8 +11,7 @@ Conflicts=obmc-host-stop@%i.target
ConditionPathExists=!/run/openbmc/mpreboot@%i
[Service]
-ExecStart=/usr/bin/env openpower-proc-control scanFSI
-SyslogIdentifier=openpower-proc-control
+ExecStart=/usr/bin/openpower-proc-control scanFSI
Type=oneshot
RemainAfterExit=yes
diff --git a/meta-openpower/recipes-phosphor/host/op-host-control/cfam_override@.service b/meta-openpower/recipes-phosphor/host/op-host-control/cfam_override@.service
index 8b773ec385..5f7cac7c05 100644
--- a/meta-openpower/recipes-phosphor/host/op-host-control/cfam_override@.service
+++ b/meta-openpower/recipes-phosphor/host/op-host-control/cfam_override@.service
@@ -12,8 +12,7 @@ ConditionPathExists=!/run/openbmc/chassis@%i-on
ConditionPathExists=/var/lib/obmc/cfam_overrides
[Service]
-ExecStart=/usr/bin/env openpower-proc-control CFAMOverride
-SyslogIdentifier=openpower-proc-control
+ExecStart=/usr/bin/openpower-proc-control CFAMOverride
Type=oneshot
RemainAfterExit=yes
diff --git a/meta-openpower/recipes-phosphor/host/op-host-control/start_host@.service b/meta-openpower/recipes-phosphor/host/op-host-control/start_host@.service
index aa12be3258..bfe8dbe5e5 100644
--- a/meta-openpower/recipes-phosphor/host/op-host-control/start_host@.service
+++ b/meta-openpower/recipes-phosphor/host/op-host-control/start_host@.service
@@ -12,8 +12,7 @@ ConditionPathExists=!/run/openbmc/host@%i-on
ConditionPathExists=!/run/openbmc/mpreboot@%i
[Service]
-ExecStart=/usr/bin/env openpower-proc-control startHost
-SyslogIdentifier=openpower-proc-control
+ExecStart=/usr/bin/openpower-proc-control startHost
Type=oneshot
RemainAfterExit=yes
diff --git a/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-disable@.service b/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-disable@.service
index 8a6f2bb61d..2988a4d22e 100644
--- a/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-disable@.service
+++ b/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-disable@.service
@@ -6,8 +6,7 @@ Before=op-enter-mpreboot@%i.service
Conflicts=op-occ-enable@%i.service
[Service]
-ExecStart=/usr/bin/env occ-active.sh disable
-SyslogIdentifier=occ-active.sh
+ExecStart=/usr/bin/occ-active.sh disable
Type=oneshot
[Install]
diff --git a/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-enable@.service b/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-enable@.service
index b97253cf07..54311019c2 100644
--- a/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-enable@.service
+++ b/meta-openpower/recipes-phosphor/occ/openpower-occ-control/op-occ-enable@.service
@@ -9,8 +9,7 @@ After=fsi-scan@%i.service
ConditionPathExists=/run/openbmc/host@%i-on
[Service]
-ExecStart=/usr/bin/env occ-active.sh enable
-SyslogIdentifier=occ-active.sh
+ExecStart=/usr/bin/occ-active.sh enable
Type=oneshot
[Install]
diff --git a/meta-openpower/recipes-phosphor/occ/openpower-occ-control/org.open_power.OCC.Control.service b/meta-openpower/recipes-phosphor/occ/openpower-occ-control/org.open_power.OCC.Control.service
index 77fb5d3402..ff4129a589 100644
--- a/meta-openpower/recipes-phosphor/occ/openpower-occ-control/org.open_power.OCC.Control.service
+++ b/meta-openpower/recipes-phosphor/occ/openpower-occ-control/org.open_power.OCC.Control.service
@@ -6,8 +6,7 @@ Wants=obmc-host-reset-running@0.target
After=obmc-host-reset-running@0.target
[Service]
-ExecStart=/usr/bin/env openpower-occ-control
-SyslogIdentifier=openpower-occ-control
+ExecStart=/usr/bin/openpower-occ-control
Restart=always
Type=dbus
BusName={BUSNAME}
diff --git a/meta-openpower/recipes-phosphor/vpd/openpower-fru-vpd/op-vpd-parser.service b/meta-openpower/recipes-phosphor/vpd/openpower-fru-vpd/op-vpd-parser.service
index 8238691096..87ac0da1a3 100644
--- a/meta-openpower/recipes-phosphor/vpd/openpower-fru-vpd/op-vpd-parser.service
+++ b/meta-openpower/recipes-phosphor/vpd/openpower-fru-vpd/op-vpd-parser.service
@@ -8,6 +8,5 @@ After=mapper-wait@-xyz-openbmc_project-inventory.service
Restart=no
Type=oneshot
EnvironmentFile={envfiledir}/obmc/inventory
-ExecStart=/usr/bin/env openpower-read-vpd --vpd $EEPROM --fru $FRUS --object $PATHS
-SyslogIdentifier=openpower-read-vpd
+ExecStart=/usr/bin/openpower-read-vpd --vpd $EEPROM --fru $FRUS --object $PATHS
RemainAfterExit=yes