Fix: EventService stops sending events
When redfish log file(/var/log/redfish) is recreated
by rsyslog daemon, EventService stops sending events
to subscribers.
The rsyslog service take redfish log files to back up
as per design. EventService has inotify on redfish log
file modifications. It needs to be removed from inotify
watcher list and added with new File descriptor.
Corrected code to handle remove & add of new descriptor.
Also corrected code to properly handle last event timestamp.
Tested:
- Ran stress test on redfish log file to create multiple
backup files and events sends to clients properly, in
all switching scenarios.
Change-Id: Ic0378edfc2cae86d1f69f6df4c429d07c2744bdf
Signed-off-by: AppaRao Puli <apparao.puli@linux.intel.com>
diff --git a/redfish-core/include/event_service_manager.hpp b/redfish-core/include/event_service_manager.hpp
index 9220a28..9c42e06 100644
--- a/redfish-core/include/event_service_manager.hpp
+++ b/redfish-core/include/event_service_manager.hpp
@@ -1103,6 +1103,7 @@
#ifndef BMCWEB_ENABLE_REDFISH_DBUS_LOG_ENTRIES
void cacheLastEventTimestamp()
{
+ lastEventTStr.clear();
std::ifstream logStream(redfishEventLogFile);
if (!logStream.good())
{
@@ -1146,7 +1147,7 @@
std::string logEntry;
while (std::getline(logStream, logEntry))
{
- if (!startLogCollection)
+ if (!startLogCollection && !lastEventTStr.empty())
{
if (boost::starts_with(logEntry, lastEventTStr))
{
@@ -1244,9 +1245,12 @@
if (fileWatchDesc != -1)
{
BMCWEB_LOG_DEBUG
- << "Redfish log file is already on "
- "inotify_add_watch.";
- return;
+ << "Remove and Add inotify watcher on "
+ "redfish event log file";
+ // Remove existing inotify watcher and add
+ // with new redfish event log file.
+ inotify_rm_watch(inotifyFd, fileWatchDesc);
+ fileWatchDesc = -1;
}
fileWatchDesc = inotify_add_watch(