From cdd9d51c5f9da41bab33d190099a003bb1f44701 Mon Sep 17 00:00:00 2001 From: Charles Rose Date: Tue, 9 Apr 2019 17:49:19 -0400 Subject: [PATCH] exchange-bmc-os-info: Remove dependency on ipmi.service Resolves ipmitool/ipmitool#46 Most modern systems do not require ipmi.service to load the kernel modules. Checking for /dev/ipmi* would be sufficient. v2: Use Assert in place of Condition to explicitly fail based on AlexanderAmelkin's feedback. Signed-off-by: Charles Rose --- contrib/exchange-bmc-os-info.service.redhat | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/contrib/exchange-bmc-os-info.service.redhat b/contrib/exchange-bmc-os-info.service.redhat index 100493b..eb28ffe 100644 --- a/contrib/exchange-bmc-os-info.service.redhat +++ b/contrib/exchange-bmc-os-info.service.redhat @@ -1,7 +1,8 @@ [Unit] Description=Exchange Information between BMC and OS -After=ipmi.service network.target -Requires=ipmi.service +After=network.target +AssertFileIsExecutable=/usr/bin/ipmitool +AssertPathExistsGlob=/dev/ipmi* [Service] Type=oneshot