host-bmc: Update set host effecter PEL severity (ibm-openbmc#534)

riyadixitagra · web-flow · commit 6c8afb8546ba · 2024-05-17T08:47:11.000-05:00
This commit updates the severity of PEL
'xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed' from ERROR to
INFORMATIONAL.

Test: Triggered set numericEffecter at runtime and verified the severity of PEL logged as INFORMATIONAL.

Signed-off-by: Riya Dixit &lt;riyadixitagra@gmail.com&gt;
diff --git a/host-bmc/dbus_to_host_effecters.cpp b/host-bmc/dbus_to_host_effecters.cpp
@@ -290,7 +290,7 @@ int HostEffecterParser::sendSetStateEffecterStates(
                 "EFFECTER_ID", effecterId, "RC", rc);
             pldm::utils::reportError(
                 "xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",
-                pldm::PelSeverity::ERROR);
+                pldm::PelSeverity::INFORMATIONAL);
         }
         if (completionCode)
         {
@@ -300,7 +300,7 @@ int HostEffecterParser::sendSetStateEffecterStates(
                 static_cast<unsigned>(completionCode));
             pldm::utils::reportError(
                 "xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",
-                pldm::PelSeverity::ERROR);
+                pldm::PelSeverity::INFORMATIONAL);
         }
         else
         {
diff --git a/oem/ibm/libpldmresponder/oem_ibm_handler.cpp b/oem/ibm/libpldmresponder/oem_ibm_handler.cpp
@@ -1156,7 +1156,7 @@ void pldm::responder::oem_ibm_platform::Handler::setHostEffecterState(
                         "RESPONSE_CODE", rc);
                     pldm::utils::reportError(
                         "xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",
-                        pldm::PelSeverity::ERROR);
+                        pldm::PelSeverity::INFORMATIONAL);
                 }
                 if (completionCode)
                 {
@@ -1166,7 +1166,7 @@ void pldm::responder::oem_ibm_platform::Handler::setHostEffecterState(
                         completionCode);
                     pldm::utils::reportError(
                         "xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",
-                        pldm::PelSeverity::ERROR);
+                        pldm::PelSeverity::INFORMATIONAL);
                 }
             };
             rc = handler->registerRequest(

Original file line number	Diff line number	Diff line change
`@@ -290,7 +290,7 @@ int HostEffecterParser::sendSetStateEffecterStates(`
`290`	`290`	`"EFFECTER_ID", effecterId, "RC", rc);`
`291`	`291`	`pldm::utils::reportError(`
`292`	`292`	`"xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",`
`293`		`- pldm::PelSeverity::ERROR);`
	`293`	`+ pldm::PelSeverity::INFORMATIONAL);`
`294`	`294`	`}`
`295`	`295`	`if (completionCode)`
`296`	`296`	`{`
`@@ -300,7 +300,7 @@ int HostEffecterParser::sendSetStateEffecterStates(`
`300`	`300`	`static_cast<unsigned>(completionCode));`
`301`	`301`	`pldm::utils::reportError(`
`302`	`302`	`"xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",`
`303`		`- pldm::PelSeverity::ERROR);`
	`303`	`+ pldm::PelSeverity::INFORMATIONAL);`
`304`	`304`	`}`
`305`	`305`	`else`
`306`	`306`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1156,7 +1156,7 @@ void pldm::responder::oem_ibm_platform::Handler::setHostEffecterState(`
`1156`	`1156`	`"RESPONSE_CODE", rc);`
`1157`	`1157`	`pldm::utils::reportError(`
`1158`	`1158`	`"xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",`
`1159`		`- pldm::PelSeverity::ERROR);`
	`1159`	`+ pldm::PelSeverity::INFORMATIONAL);`
`1160`	`1160`	`}`
`1161`	`1161`	`if (completionCode)`
`1162`	`1162`	`{`
`@@ -1166,7 +1166,7 @@ void pldm::responder::oem_ibm_platform::Handler::setHostEffecterState(`
`1166`	`1166`	`completionCode);`
`1167`	`1167`	`pldm::utils::reportError(`
`1168`	`1168`	`"xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed",`
`1169`		`- pldm::PelSeverity::ERROR);`
	`1169`	`+ pldm::PelSeverity::INFORMATIONAL);`
`1170`	`1170`	`}`
`1171`	`1171`	`};`
`1172`	`1172`	`rc = handler->registerRequest(`