RDKEMW-18591: Reboot Reason log Timestamp missmatch in parodus logs with C modue

[madhubabutt]

No description provided.

[Copilot]

Pull request overview

This PR updates how “PreviousRebootInfo” is written into Parodus-related logs/files and adjusts logging verbosity/messages in the reboot-reason-fetcher flow to provide different runtime visibility.

Changes:

• Reorders PreviousRebootInfo fields to timestamp,custom,source,reason in multiple writers and updates unit test expectations accordingly.
• Updates update_previous_reboot_info.sh Parodus log append format (including removing the /bin/timestamp + $0: prefix in one path).
• Adds/adjusts several RDK_LOG messages and log levels in rebootreason_main.c, plus additional discovery logging in bootup_reason_checker.c.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
unittest/reboot_parodus_gtest.cpp	Updates expected PreviousRebootInfo field ordering in unit tests.
scripts/update_previous_reboot_info.sh	Changes Parodus log append lines and PreviousRebootInfo field ordering.
reboot-reason-fetcher/src/rebootreason_main.c	Adds/changes informational logs and log levels during reboot reason processing.
reboot-reason-fetcher/src/parodus_log_update.c	Changes PreviousRebootInfo formatting/order when updating parodus.log / fallback file output.
reboot-reason-fetcher/src/log_parser.c	Lowers “STT flag missing” log from INFO to DEBUG.
reboot-reason-fetcher/src/bootup_reason_checker.c	Adds log statements and changes fallback log messaging during previous reboot log discovery.

Comments suppressed due to low confidence (1)

reboot-reason-fetcher/src/parodus_log_update.c:160

• The info->reason argument line is tab-indented, while surrounding lines use spaces. This introduces inconsistent formatting in this block; please convert the tab indentation to spaces to match the file’s style.

    fprintf(out, "PreviousRebootInfo:%s,%s,%s,%s\n",
            info->timestamp,
            info->customReason,
            info->source,
	    info->reason);

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (1)

reboot-reason-fetcher/src/parodus_log_update.c:82

• update_parodus_log() now writes Parodus log lines as "<ts> <component>: ...", but handle_parodus_reboot_file() still writes "<ts>: <component>: ..." (see same file around lines 135-136). This produces mixed log formats in the same log file; consider standardizing on one format.

    fprintf(fp, "%s %s: Updating previous reboot info to Parodus\n", timestamp, "update_previous_reboot_info");
    fprintf(fp, "%s %s: PreviousRebootInfo:%s,%s,%s,%s\n",
            timestamp,
	    "update_previous_reboot_info",
            info->timestamp,
            info->reason,
            info->customReason,
            info->source);