PEL: On remove() call, return the removed LogID
Have Repository::remove(const LogID& id), which is used to remove
a PEL, return the full LogID of the removed PEL instead of not
returning anything.
That input id only has to have a filled in PEL ID or OpenBMC event log
ID field, but not both, so returning the full LogID of the removed PEL
ensure the calling code will have both values.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: If869c59acd59563db137d7c9523210539b51a040
diff --git a/extensions/openpower-pels/repository.cpp b/extensions/openpower-pels/repository.cpp
index 59b3eaa..7fd509b 100644
--- a/extensions/openpower-pels/repository.cpp
+++ b/extensions/openpower-pels/repository.cpp
@@ -207,11 +207,14 @@
}
}
-void Repository::remove(const LogID& id)
+std::optional<Repository::LogID> Repository::remove(const LogID& id)
{
+ std::optional<LogID> actualID;
+
auto pel = findPEL(id);
if (pel != _pelAttributes.end())
{
+ actualID = pel->first;
updateRepoStats(pel->second, false);
log<level::DEBUG>("Removing PEL from repository",
@@ -228,6 +231,8 @@
entry("PEL_ID=0x%X", id.pelID.id),
entry("OBMC_LOG_ID=%d", id.obmcID.id));
}
+
+ return actualID;
}
std::optional<std::vector<uint8_t>> Repository::getPELData(const LogID& id)