Skip to content

RDKB-59420: Minor update to Previous Logs lookup feature #64

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
shibu-kv merged 1 commit into from
Apr 10, 2025
Merged

RDKB-59420: Minor update to Previous Logs lookup feature #64

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions source/bulkdata/profile.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1379,6 +1379,15 @@ static void loadReportProfilesFromDisk(bool checkPreviousSeek)
}
else
{

#ifdef PERSIST_LOG_MON_REF
if(profile->checkPreviousSeek)
{
T2Info("Previous Seek is enabled so generate the report for %s\n", profile->name);
// Trigger the report generation for the profile if previous seek is valid
NotifyTimeout(profile->name, true);
}
#endif
// Load the cached messages from previous boot, if any
populateCachedReportList(profile->name, profile->cachedReportList);
}
Expand Down
9 changes: 9 additions & 0 deletions source/bulkdata/profilexconf.c
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/*
* If not stated otherwise in this file or this component's LICENSE file the
* following copyright and licenses apply:

Check failure on line 3 in source/bulkdata/profilexconf.c

View workflow job for this annotation

GitHub Actions / call-fossid-workflow / Fossid Annotate PR

FossID License Issue Detected 

Source code with 'Apache-2.0' license found in local file 'source/bulkdata/profilexconf.c' (Match: rdk/components/generic/telemetry/rdk/components/generic/telemetry/1, 973 lines, url: https://code.rdkcentral.com/r/plugins/gitiles/rdk/components/generic/telemetry/+archive/RDKB-RELEASE-TEST-DUNFELL-1.tar.gz, file: source/bulkdata/profilexconf.c)

Check failure on line 3 in source/bulkdata/profilexconf.c

View workflow job for this annotation

GitHub Actions / call-fossid-workflow / Fossid Annotate PR

FossID License Issue Detected 

Source code with 'Apache-2.0' license found in local file 'source/bulkdata/profilexconf.c' (Match: rdkcmf/rdk-telemetry/2010, 973 lines, url: https://github.com/rdkcmf/rdk-telemetry/archive/RDK-DEV-2010.zip, file: source/bulkdata/profilexconf.c)
*
* Copyright 2019 RDK Management
*
Expand Down Expand Up @@ -512,6 +512,15 @@
if(T2ERROR_SUCCESS == ProfileXConf_set(profile))
{
T2Info("Successfully set new profile: %s\n", profile->name);

#ifdef PERSIST_LOG_MON_REF
if(profile->checkPreviousSeek)
{
T2Info("Previous Seek is enabled so generate the Xconf report \n");
// Trigger a xconf report if previous seek is enabled and valid.
ProfileXConf_notifyTimeout(true, true);
}
#endif
populateCachedReportList(profile->name, profile->cachedReportList);
}
else
Expand Down
14 changes: 2 additions & 12 deletions source/bulkdata/reportprofiles.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -615,18 +615,6 @@ T2ERROR initReportProfiles()
}
}

#ifdef PERSIST_LOG_MON_REF

if(previousLogCheck)
{
//generate previous logs report
generateDcaReport(false, false);
T2Info("Previous Log check is enabled sending Interrupt to scheduler \n");
sendLogUploadInterruptToScheduler();
}
#endif


if(ProfileXConf_isSet() || getProfileCount() > 0)
{

Expand Down Expand Up @@ -1386,6 +1374,8 @@ int __ReportProfiles_ProcessReportProfilesMsgPackBlob(void *msgpack, bool checkP
{
T2Info("Previous seek is enabled for profile %s \n", profile->name);
profile->checkPreviousSeek = true;
T2Info("Previous Seek is enabled so generate the report for %s\n", profile->name);
NotifyTimeout(profile->name, true);
}
else
{
Expand Down
2 changes: 1 addition & 1 deletion source/ccspinterface/rbusInterface.c
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/*
* If not stated otherwise in this file or this component's LICENSE file the
* following copyright and licenses apply:

Check failure on line 3 in source/ccspinterface/rbusInterface.c

View workflow job for this annotation

GitHub Actions / call-fossid-workflow / Fossid Annotate PR

FossID License Issue Detected 

Source code with 'Apache-2.0' license found in local file 'source/ccspinterface/rbusInterface.c' (Match: rdkcmf/rdk-telemetry/2011, 1533 lines, url: https://github.com/rdkcmf/rdk-telemetry/archive/RDK-DEV-2011.zip, file: source/ccspinterface/rbusInterface.c)
*
* Copyright 2019 RDK Management
*
Expand Down Expand Up @@ -1356,7 +1356,7 @@

if(!compTr181ParamMap)
{
T2Info("No data elements present to unregister");
T2Info("No data elements present to unregister\n");
T2Debug("%s --out\n", __FUNCTION__);
pthread_mutex_unlock(&compParamMap);
return;
Expand Down
12 changes: 6 additions & 6 deletions source/dcautil/legacyutils.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -580,8 +580,8 @@ char *getLogLine(hash_map_t *logSeekMap, char *buf, int buflen, char *name, int
rotatedLog = strdup(currentLogFile);
if(NULL != rotatedLog)
{
rotatedLog[-1] = '1';
//T2Debug("Log file name seems to be having .0 extension hence Rotated log file name is %s\n", rotatedLog);
rotatedLog[name_len - 1] = '1';
T2Debug("Log file name seems to be having .0 extension hence Rotated log file name is %s\n", rotatedLog);
}
}
else
Expand All @@ -590,7 +590,7 @@ char *getLogLine(hash_map_t *logSeekMap, char *buf, int buflen, char *name, int
if(NULL != rotatedLog)
{
snprintf(rotatedLog, fileExtn_len, "%s%s%s", logpath, name, fileExtn);
//T2Debug("Rotated log file name is %s\n", rotatedLog);
T2Debug("Rotated log file name is %s\n", rotatedLog);
}
}
if(NULL != rotatedLog)
Expand Down Expand Up @@ -659,8 +659,8 @@ char *getLogLine(hash_map_t *logSeekMap, char *buf, int buflen, char *name, int
rotatedLog = strdup(currentLogFile);
if(NULL != rotatedLog)
{
rotatedLog[-1] = '1';
//T2Debug("Log file name seems to be having .0 extension hence Rotated log file name is %s\n", rotatedLog);
rotatedLog[name_len - 1] = '1';
T2Debug("Log file name seems to be having .0 extension hence Rotated log file name is %s\n", rotatedLog);
}
}
else
Expand All @@ -669,7 +669,7 @@ char *getLogLine(hash_map_t *logSeekMap, char *buf, int buflen, char *name, int
if(NULL != rotatedLog)
{
snprintf(rotatedLog, fileExtn_len, "%s%s%s", logpath, name, fileExtn);
//T2Debug("Rotated log file name is %s\n", rotatedLog);
T2Debug("Rotated log file name is %s\n", rotatedLog);
}
}
if(currentLogFile != NULL)
Expand Down
Loading