commit 585d3a079d00382b1996740e78075867c03db3c7
author Thu Ba Nguyen <tbnguyen@amperecomputing.com> Tue Jul 20 20:25:15 2021 +0700
committer ThuBaNguyen <thu@os.amperecomputing.com> Tue Jul 20 21:36:55 2021 +0700
tree bb7b359a2d8a7e097ae7fde06937a7f6bedda115
parent 9f6d4fd6fad9a8d74e348317a13ee3c75e01b757

dbus-sensors: Change the host state match string

When the CurrentHostState dbus property value ends with "Running",
Dbus-sensors marks the host as running and starts reading the sensors
with "PowerState=On" configuration.
From commit "host state transitioning ...", phosphor-state-manager
package updates the transition state "TransitioningToOff" or
"TransitioningToRunning" to CurrentHostState property. This causes
dbus-sensors detects the host is running when it is transiting to
running state.
This commit fixs that issue by changing the match string from
"Running" to ".Running".

Tested:
	1. Power off the host.
	2. Power on the host.
	3. Make sure there is no threshold warnings/errors for
	the sensors with "PowerState=On" configuration.

Signed-off-by: ThuBaNguyen <thu@os.amperecomputing.com>
Change-Id: I3bde0f3d288614657869f54e286a2d690d92dcad

src/IpmbSensor.cpp

diff --git a/src/IpmbSensor.cpp b/src/IpmbSensor.cpp
index c7a9f33..edbe05f 100644
--- a/src/IpmbSensor.cpp
+++ b/src/IpmbSensor.cpp
@@ -581,7 +581,7 @@
     if (findStatus != values.end())
     {
         bool powerStatus = boost::ends_with(
-            std::get<std::string>(findStatus->second), "Running");
+            std::get<std::string>(findStatus->second), ".Running");
         if (powerStatus)
         {
             if (!initCmdTimer)

src/Utils.cpp

diff --git a/src/Utils.cpp b/src/Utils.cpp
index cf0ce9a..5a5064f 100644
--- a/src/Utils.cpp
+++ b/src/Utils.cpp
@@ -268,7 +268,7 @@
                 return;
             }
             powerStatusOn =
-                boost::ends_with(std::get<std::string>(state), "Running");
+                boost::ends_with(std::get<std::string>(state), ".Running");
         },
         power::busname, power::path, properties::interface, properties::get,
         power::interface, power::property);
@@ -330,7 +330,7 @@
             if (findState != values.end())
             {
                 bool on = boost::ends_with(
-                    std::get<std::string>(findState->second), "Running");
+                    std::get<std::string>(findState->second), ".Running");
                 if (!on)
                 {
                     timer.cancel();