Spike

.github/workflows/L2-tests.yml

@@ -39,7 +39,7 @@ jobs:
 
       - name: Enter Inside Platform native container and run L2 Test
         run: |
-          docker exec -i native-platform /bin/bash -c "cd /mnt/L2_CONTAINER_SHARED_VOLUME/ && sh cov_build.sh && export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/lib/x86_64-linux-gnu:/lib/aarch64-linux-gnu:/usr/local/lib: && sh test/run_l2.sh && sh test/run_uploadstblogs_l2.sh"
+          docker exec -i native-platform /bin/bash -c "cd /mnt/L2_CONTAINER_SHARED_VOLUME/ && sh cov_build.sh && export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/lib/x86_64-linux-gnu:/lib/aarch64-linux-gnu:/usr/local/lib: && sh test/run_uploadstblogs_l2.sh"
 
       - name: Copy l2 test results to runner
         run: |

backup_logs/include/backup_logs.h

@@ -26,6 +26,13 @@
 extern "C" {
 #endif
 
+/** Sentinel written by backup_logs after successful completion.
+ *  Cross-repo interface: also referenced by reboot-manager's update-prev-reboot-info
+ *  and telemetry's telemetry2_0.
+ *  Any path change MUST be coordinated with the reboot-manager and telemetry repositories. */
+
+#define BACKUP_LOGS_DONE_FLAG  "/tmp/.backup_logs_done"
+
 /**
  * @brief Main entry point for backup_logs system
  * 

backup_logs/src/backup_logs.c

@@ -33,6 +33,8 @@
 #include "special_files.h"
 #include "system_utils.h"
 #include <secure_wrapper.h>
+#include <fcntl.h>
+#include <errno.h>
 
 #define BACKUP_LOGS_VERSION "1.0.0"
 #define BACKUP_LOGS_BUILD_DATE __DATE__
@@ -52,7 +54,7 @@ int backup_logs_init(backup_config_t *config) {
     rdk_LogOutput_File filelog;
     strncpy(filelog.fileName, "backup_logs.log", sizeof(filelog.fileName)-1);
     filelog.fileName[sizeof(filelog.fileName) - 1] = '\0';
-    strncpy(filelog.fileLocation, "/tmp/", sizeof(filelog.fileLocation)-1);
+    strncpy(filelog.fileLocation, "/opt/logs", sizeof(filelog.fileLocation)-1);
     filelog.fileLocation[sizeof(filelog.fileLocation) - 1] = '\0';
     filelog.fileSizeMax = 51200;  /* 50KB max file size */
     filelog.fileCountMax = 5;     /* Keep 5 rotated files */
@@ -302,6 +304,22 @@ int backup_logs_main(int argc, char *argv[]) {
     }
 
     RDK_LOG(RDK_LOG_INFO, LOG_BACKUP_LOGS, "Backup process completed successfully\n");
+    /* Write completion sentinel for downstream consumers (reboot-manager, telemetry).
+     * /tmp/ is volatile — no stale-sentinel risk across reboots.
+     * Non-fatal: if open() fails, downstream services will time out and annotate gracefully. */
+    {
+        int sentinel_fd = open(BACKUP_LOGS_DONE_FLAG, O_CREAT | O_WRONLY, 0644);
+        if (sentinel_fd < 0) 
+        {
+            RDK_LOG(RDK_LOG_WARN, LOG_BACKUP_LOGS, "Failed to create sentinel %s: %s\n", BACKUP_LOGS_DONE_FLAG, strerror(errno));
+        } 
+        else 
+        {
+            close(sentinel_fd);
+            RDK_LOG(RDK_LOG_INFO, LOG_BACKUP_LOGS, "Sentinel written: %s\n", BACKUP_LOGS_DONE_FLAG);
+        }
+    }
+
     return EXIT_SUCCESS;
 }
 #ifndef GTEST_ENABLE

dcm.c

@@ -37,7 +37,6 @@
 #include "dcm_rbus.h"
 #include "dcm_cronparse.h"
 #include "dcm_schedjob.h"
-#include "uploadstblogs.h"
 
 static DCMDHandle *g_pdcmHandle = NULL;
 
@@ -69,42 +68,7 @@ static VOID dcmRunJobs(const INT8* profileName, VOID *pHandle)
         pRDKPath = DCM_LIB_PATH;
     }
 
-    if(strcmp(profileName, DCM_LOGUPLOAD_SCHED) == 0) {
-        INT8 *pPrctl = dcmSettingsGetUploadProtocol(pdcmHandle->pDcmSetHandle);
-        INT8 *pURL   = dcmSettingsGetUploadURL(pdcmHandle->pDcmSetHandle);
-
-        if(pPrctl == NULL) {
-            DCMWarn("Log Upload protocol is NULL, using HTTP\n");
-            pPrctl = "HTTP";
-        }
-        if(pURL == NULL) {
-            DCMWarn("Log Upload URL is NULL, using %s\n", DCM_DEF_LOG_URL);
-            pURL = DCM_DEF_LOG_URL;
-        }
-
-        DCMInfo("\nStart log upload via library API\n");
-
-        // Call uploadstblogs library API instead of shell script
-        UploadSTBLogsParams params = {
-            .flag = 0,
-            .dcm_flag = 1,
-            .upload_on_reboot = false,
-            .upload_protocol = pPrctl,
-            .upload_http_link = pURL,
-            .trigger_type = TRIGGER_SCHEDULED,
-            .rrd_flag = false,
-            .rrd_file = NULL
-        };
-#ifndef GTEST_ENABLE
-        int result = uploadstblogs_run(&params);
-        if (result != 0) {
-            DCMError("Log upload failed with error code: %d\n", result);
-        } else {
-            DCMInfo("Log upload completed successfully\n");
-        }
-#endif
-    }
-    else if(strcmp(profileName, DCM_DIFD_SCHED) == 0) {
+    if(strcmp(profileName, DCM_DIFD_SCHED) == 0) {
         DCMInfo("Start FW update Script\n");
         snprintf(pExecBuff, EXECMD_BUFF_SIZE, "/bin/sh %s/swupdate_utility.sh 0 2 >> /opt/logs/swupdate.log 2>&1",
                                                pRDKPath);
@@ -202,15 +166,6 @@ INT32 dcmDaemonMainInit(DCMDHandle *pdcmHandle)
         return ret;
     }
 
-    /* Add log upload job to Schecduler */
-    pdcmHandle->pLogSchedHandle  = dcmSchedAddJob(DCM_LOGUPLOAD_SCHED,
-                                                  (DCMSchedCB)dcmRunJobs,
-                                                  (VOID *) pdcmHandle);
-    if(pdcmHandle->pLogSchedHandle == NULL) {
-        DCMError("Failed to Add Log Scheduler jobs\n");
-        return DCM_FAILURE;
-    }
-
     /* Add FW update job to Schecduler */
     pdcmHandle->pDifdSchedHandle = dcmSchedAddJob(DCM_DIFD_SCHED,
                                                   (DCMSchedCB)dcmRunJobs,
@@ -241,9 +196,7 @@ VOID dcmDaemonMainUnInit(DCMDHandle *pdcmHandle)
 
     dcmSettingsUnInit(pdcmHandle->pDcmSetHandle);
     dcmRbusUnInit(pdcmHandle->pRbusHandle);
-    dcmSchedStopJob(pdcmHandle->pLogSchedHandle);
     dcmSchedStopJob(pdcmHandle->pDifdSchedHandle);
-    dcmSchedRemoveJob(pdcmHandle->pLogSchedHandle);
     dcmSchedRemoveJob(pdcmHandle->pDifdSchedHandle);
     dcmSchedUnInit();
 
@@ -368,11 +321,11 @@ int main(int argc, char* argv[])
                 continue;
             }
 
+            INT8 unusedLogCron[16] = {0};
             ret = dcmSettingParseConf(g_pdcmHandle->pDcmSetHandle, pconfPath,
-                                      g_pdcmHandle->logCron,
+                                      unusedLogCron,
                                       g_pdcmHandle->difdCron);
             if(ret == DCM_SUCCESS) {
-                dcmSchedStartJob(g_pdcmHandle->pLogSchedHandle, g_pdcmHandle->logCron);
                 dcmSchedStartJob(g_pdcmHandle->pDifdSchedHandle, g_pdcmHandle->difdCron);
 
                 ret = dcmIARMEvntSend(DCM_IARM_COMPLETE);

dcm.h

@@ -26,7 +26,6 @@ extern "C"
 {
 #endif
 
-#define DCM_LOGUPLOAD_SCHED    "DCM_LOG_UPLOAD"
 #define DCM_DIFD_SCHED         "DCM_FW_UPDATE"
 
 typedef struct _dcmdHandle
@@ -35,10 +34,8 @@ typedef struct _dcmdHandle
     BOOL  isDCMRunning;
     VOID *pRbusHandle;
     VOID *pDcmSetHandle;
-    VOID *pLogSchedHandle;
     VOID *pDifdSchedHandle;
     INT8 *pExecBuff;
-    INT8  logCron[16];
     INT8  difdCron[16];
 
 } DCMDHandle;

dcm_parseconf.c

@@ -37,7 +37,6 @@
 #include "dcm_utils.h"
 #include "dcm_rbus.h"
 #include "dcm_parseconf.h"
-#include "uploadstblogs.h"
 
 static INT32 g_bMMEnable = 0;
 
@@ -578,72 +577,6 @@ INT32 dcmSettingParseConf(VOID *pHandle, INT8 *pConffile,
 
     DCMInfo("DCM_DIFD_CRON: %s\n", pDifdCron);
 
-    if(uploadCheck == 1 && pdcmSetHandle->bRebootFlag == 0) {
-        DCMInfo("Triggering log upload with reboot flag via library API\n");
-        UploadSTBLogsParams params = {
-            .flag = 1,
-            .dcm_flag = 1,
-            .upload_on_reboot = true,
-            .upload_protocol = pUploadprtl,
-            .upload_http_link = pUploadURL,
-            .trigger_type = TRIGGER_REBOOT,
-            .rrd_flag = false,
-            .rrd_file = NULL
-        };
-#ifndef GTEST_ENABLE
-        int result = uploadstblogs_run(&params);
-        if (result != 0) {
-            DCMError("Log upload (reboot=true) failed: %d\n", result);
-        }
-#endif
-    }
-    else if (uploadCheck == 0 && pdcmSetHandle->bRebootFlag == 0) {
-        DCMInfo("Triggering log upload without reboot flag via library API\n");
-        UploadSTBLogsParams params = {
-            .flag = 1,
-            .dcm_flag = 1,
-            .upload_on_reboot = false,
-            .upload_protocol = pUploadprtl,
-            .upload_http_link = pUploadURL,
-            .trigger_type = TRIGGER_SCHEDULED,
-            .rrd_flag = false,
-            .rrd_file = NULL
-        };
-#ifndef GTEST_ENABLE
-        int result = uploadstblogs_run(&params);
-        if (result != 0) {
-            DCMError("Log upload (reboot=false) failed: %d\n", result);
-        }
-#endif
-    }
-    else {
-        DCMWarn ("Nothing to do here for uploadCheck value = %d\n", uploadCheck);
-    }
-
-    if(strlen(pLogCron) == 0) {
-        DCMWarn ("Uploading logs as DCM response is either null or not present\n");
-
-        UploadSTBLogsParams params = {
-            .flag = 1,
-            .dcm_flag = 1,
-            .upload_on_reboot = false,
-            .upload_protocol = pUploadprtl,
-            .upload_http_link = pUploadURL,
-            .trigger_type = TRIGGER_SCHEDULED,
-            .rrd_flag = false,
-            .rrd_file = NULL
-        };
-#ifndef GTEST_ENABLE
-        int result = uploadstblogs_run(&params);
-        if (result != 0) {
-            DCMError("Log upload (empty cron) failed: %d\n", result);
-        }
-#endif
-    }
-    else {
-        DCMInfo ("%s is present setting cron jobs\n", DCM_LOGUPLOAD_CRON);
-    }
-
     if(strlen(pDifdCron) == 0) {
         DCMWarn ("difdCron is empty\n");
     }