| commit | a5eefd0f3efc08f68333bea3343b0aa759036769 | [log] [tgz] |
|---|---|---|
| author | Andrew Geissler <geissonator@yahoo.com> | Fri Jan 20 13:16:40 2023 -0700 |
| committer | Andrew Geissler <geissonator@yahoo.com> | Fri Jan 20 13:16:40 2023 -0700 |
| tree | 85b9a9919c29957901550072678f769455cafa21 | |
| parent | 960017a4dce1bbc8cdc730c8552af2eb83798100 [diff] |
service-deps: remove logging as dependency With the introduction of the following best practice: https://github.com/openbmc/docs/commit/43421bdd06dece37b2acc4c656d7839bfedcc681 It was determined that phosphor-logging should be a d-bus activated service. That is, this service will be activated when called so there is no reason to specifically list it as a dependency in service files. Signed-off-by: Andrew Geissler <geissonator@yahoo.com> Change-Id: I8d871768bd3e515741825287bed4646a51c9b996
diff --git a/service_files/phosphor-systemd-target-monitor.service b/service_files/phosphor-systemd-target-monitor.service index f43331e..1a9fc06 100644 --- a/service_files/phosphor-systemd-target-monitor.service +++ b/service_files/phosphor-systemd-target-monitor.service
@@ -1,8 +1,6 @@ [Unit] Description=phosphor systemd target monitor After=dbus.service -Wants=xyz.openbmc_project.Logging.service -After=xyz.openbmc_project.Logging.service [Service] Restart=always
diff --git a/service_files/xyz.openbmc_project.State.BMC.service b/service_files/xyz.openbmc_project.State.BMC.service index 45ba6f0..c9520b1 100644 --- a/service_files/xyz.openbmc_project.State.BMC.service +++ b/service_files/xyz.openbmc_project.State.BMC.service
@@ -4,8 +4,6 @@ After=xyz.openbmc_project.State.Chassis@0.service Wants=obmc-mapper.target After=obmc-mapper.target -Wants=xyz.openbmc_project.Logging.service -After=xyz.openbmc_project.Logging.service [Service]
diff --git a/service_files/xyz.openbmc_project.State.Chassis@.service b/service_files/xyz.openbmc_project.State.Chassis@.service index 5f4796c..4f9b98d 100644 --- a/service_files/xyz.openbmc_project.State.Chassis@.service +++ b/service_files/xyz.openbmc_project.State.Chassis@.service
@@ -4,8 +4,6 @@ Wants=obmc-mapper.target After=obmc-mapper.target After=org.openbmc.control.Power@%i.service -Wants=xyz.openbmc_project.Logging.service -After=xyz.openbmc_project.Logging.service [Service] ExecStart=/usr/bin/phosphor-chassis-state-manager --chassis %i
diff --git a/service_files/xyz.openbmc_project.State.ScheduledHostTransition@.service b/service_files/xyz.openbmc_project.State.ScheduledHostTransition@.service index 7c7ab5d..13c617b 100644 --- a/service_files/xyz.openbmc_project.State.ScheduledHostTransition@.service +++ b/service_files/xyz.openbmc_project.State.ScheduledHostTransition@.service
@@ -4,8 +4,6 @@ After=obmc-mapper.target Wants=xyz.openbmc_project.State.Host%i.service After=xyz.openbmc_project.State.Host%i.service -Wants=xyz.openbmc_project.Logging.service -After=xyz.openbmc_project.Logging.service [Service] ExecStart=/usr/bin/phosphor-scheduled-host-transition --host %i