commit 013487e5d56e4e7964e85a38d6c3044f081a753b
author raviteja-b <raviteja28031990@gmail.com> Tue Mar 03 03:20:48 2020 -0600
committer Ravi Teja <raviteja28031990@gmail.com> Thu May 28 00:33:01 2020 +0000
tree fc14a3f33cff56861a0dd3a288f23917443c98a7
parent 0657843a00da62c3a076dacbf6203867a4fb5eb5

Redfish: ClearLog action support for system dump log entries

Tested By:
POST https://${IP}/redfish/v1/Systems/system/LogServices/SystemDump/Actions/LogService.ClearLog

Change-Id: I28af3ccc1d7bd54c521e79d93e6ccb1436eefc4f
Signed-off-by: Ravi Teja <raviteja28031990@gmail.com>

redfish-core/include/redfish.hpp

diff --git a/redfish-core/include/redfish.hpp b/redfish-core/include/redfish.hpp
index 2fa5e00..48cca29 100644
--- a/redfish-core/include/redfish.hpp
+++ b/redfish-core/include/redfish.hpp
@@ -105,6 +105,7 @@
         nodes.emplace_back(std::make_unique<SystemDumpEntryCollection>(app));
         nodes.emplace_back(std::make_unique<SystemDumpEntry>(app));
         nodes.emplace_back(std::make_unique<SystemDumpEntryDownload>(app));
+        nodes.emplace_back(std::make_unique<SystemDumpClear>(app));
 #endif
 
 #ifndef BMCWEB_ENABLE_REDFISH_DBUS_LOG_ENTRIES

redfish-core/lib/log_services.hpp

diff --git a/redfish-core/lib/log_services.hpp b/redfish-core/lib/log_services.hpp
index 8590373..df6e210 100644
--- a/redfish-core/lib/log_services.hpp
+++ b/redfish-core/lib/log_services.hpp
@@ -156,8 +156,6 @@
                 boost::beast::http::status::internal_server_error);
             return;
         }
-
-        asyncResp->res.result(boost::beast::http::status::ok);
     };
     crow::connections::systemBus->async_method_call(
         respHandler, "xyz.openbmc_project.Dump.Manager",
@@ -1880,6 +1878,54 @@
     }
 };
 
+class SystemDumpClear : public Node
+{
+  public:
+    SystemDumpClear(CrowApp &app) :
+        Node(app, "/redfish/v1/Systems/system/LogServices/System/"
+                  "Actions/"
+                  "LogService.ClearLog/")
+    {
+        entityPrivileges = {
+            {boost::beast::http::verb::get, {{"Login"}}},
+            {boost::beast::http::verb::head, {{"Login"}}},
+            {boost::beast::http::verb::post, {{"ConfigureManager"}}}};
+    }
+
+  private:
+    void doPost(crow::Response &res, const crow::Request &req,
+                const std::vector<std::string> &params) override
+    {
+
+        auto asyncResp = std::make_shared<AsyncResp>(res);
+        crow::connections::systemBus->async_method_call(
+            [asyncResp](const boost::system::error_code ec,
+                        const std::vector<std::string> &dumpList) {
+                if (ec)
+                {
+                    messages::internalError(asyncResp->res);
+                    return;
+                }
+
+                for (const std::string &objectPath : dumpList)
+                {
+                    std::size_t pos = objectPath.rfind("/");
+                    if (pos != std::string::npos)
+                    {
+                        std::string logID = objectPath.substr(pos + 1);
+                        deleteSystemDumpEntry(asyncResp->res, logID);
+                    }
+                }
+            },
+            "xyz.openbmc_project.ObjectMapper",
+            "/xyz/openbmc_project/object_mapper",
+            "xyz.openbmc_project.ObjectMapper", "GetSubTreePaths",
+            "/xyz/openbmc_project/dump", 0,
+            std::array<const char *, 1>{
+                "xyz.openbmc_project.Dump.Entry.System"});
+    }
+};
+
 class CrashdumpService : public Node
 {
   public: