commit dfe839fddb2967e76dd4233c5de168c9ad3613d8
author Jagpal Singh Gill <paligill@gmail.com> Fri Feb 16 09:54:02 2024 -0800
committer Jagpal Singh Gill <paligill@gmail.com> Fri Feb 16 14:53:51 2024 -0800
tree 62e7b01ff637247b15c8599fd98ba53189f70e14
parent 7f3fd6ea94794c9573ed31c8a1df81e788d4ebe6

health_metric: add tmp storage support

Add the tmp storage support in health config parser and health metric.

Change-Id: Ibb265c28b22d786031ed152d2cc52f40814be984
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

health_metric.cpp

diff --git a/health_metric.cpp b/health_metric.cpp
index ed41fd1..b37698e 100644
--- a/health_metric.cpp
+++ b/health_metric.cpp
@@ -54,6 +54,10 @@
         {
             return std::string(BmcPath) + "/" + PathIntf::read_write_storage;
         }
+        case SubType::storageTmp:
+        {
+            return std::string(BmcPath) + "/" + PathIntf::tmp_storage;
+        }
         default:
         {
             error("Invalid Memory metric {TYPE}", "TYPE",

health_metric_config.cpp

diff --git a/health_metric_config.cpp b/health_metric_config.cpp
index be8ebb7..231371a 100644
--- a/health_metric_config.cpp
+++ b/health_metric_config.cpp
@@ -44,7 +44,7 @@
     {"Memory_Shared", SubType::memoryShared},
     {"Memory_Buffered_And_Cached", SubType::memoryBufferedAndCached},
     {"Storage_RW", SubType::storageReadWrite},
-    {"Storage_TMP", SubType::storageReadWrite}};
+    {"Storage_TMP", SubType::storageTmp}};
 
 /** Deserialize a Threshold from JSON. */
 void from_json(const json& j, Threshold& self)

health_metric_config.hpp

diff --git a/health_metric_config.hpp b/health_metric_config.hpp
index 2b0a578..49f39ad 100644
--- a/health_metric_config.hpp
+++ b/health_metric_config.hpp
@@ -37,6 +37,7 @@
     memoryTotal,
     // Storage subtypes
     storageReadWrite,
+    storageTmp,
     NA
 };
 

test/test_health_metric_config.cpp

diff --git a/test/test_health_metric_config.cpp b/test/test_health_metric_config.cpp
index 471bf2b..c13e819 100644
--- a/test/test_health_metric_config.cpp
+++ b/test/test_health_metric_config.cpp
@@ -43,7 +43,8 @@
                 .contains(subType);
 
         case metric::Type::storage:
-            return set_t{metric::SubType::storageReadWrite, metric::SubType::NA}
+            return set_t{metric::SubType::storageReadWrite,
+                         metric::SubType::storageTmp}
                 .contains(subType);
 
         case metric::Type::inode: