Skip to content

RDKEMW-18459 : Ensure Proper Synchronization across Logs Backup, Reboot Reason and Log Upload - #385

Merged
shibu-kv merged 38 commits into
developfrom
feature/RDKEMW-16716
Jul 17, 2026
Merged

RDKEMW-18459 : Ensure Proper Synchronization across Logs Backup, Reboot Reason and Log Upload#385
shibu-kv merged 38 commits into
developfrom
feature/RDKEMW-16716

Conversation

@Abhinavpv28

@Abhinavpv28 Abhinavpv28 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Reason for change: Add changes for creating sentinal file onepreviouLogs grepping is completed , ensure backuplogs has been completed before previousLogs Grepping
Test Procedure: Focused Regression
Risks: Low
Signed-off-by: Abhinav P V Abhinav_Valappil@comcast.com

Copilot AI review requested due to automatic review settings June 16, 2026 15:48
@Abhinavpv28
Abhinavpv28 requested a review from a team as a code owner June 16, 2026 15:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings July 2, 2026 12:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/dcautil/dcautil.h
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Copilot AI review requested due to automatic review settings July 7, 2026 05:05
Comment thread source/dcautil/dcautil.c Fixed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c Outdated
Comment thread source/dcautil/dcautil.h
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 13, 2026 06:33
… restricting permissions'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.h
Comment thread source/dcautil/dcautil.c
Copilot AI review requested due to automatic review settings July 13, 2026 06:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c Outdated
Comment thread source/dcautil/dcautil.c
Copilot AI review requested due to automatic review settings July 13, 2026 10:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/dcautil/dcautil.c Outdated
Comment thread source/dcautil/dcautil.c Outdated
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Copilot AI review requested due to automatic review settings July 14, 2026 17:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings July 15, 2026 15:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

Code Coverage Summary

                                   Total:|61.2%   9949|89.3%   346|    -      0

1 similar comment
@github-actions

Copy link
Copy Markdown

Code Coverage Summary

                                   Total:|61.2%   9949|89.3%   346|    -      0

Copilot AI review requested due to automatic review settings July 15, 2026 17:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings July 16, 2026 13:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Abhinavpv28 Abhinavpv28 changed the title Feature/rdkemw 16716 RDKEMW-18459 : Ensure Proper Synchronization across Logs Backup, Reboot Reason and Log Upload Jul 16, 2026
@github-actions

Copy link
Copy Markdown

Code Coverage Summary

                                   Total:|61.2%   9949|89.3%   346|    -      0

Copilot AI review requested due to automatic review settings July 17, 2026 11:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings July 17, 2026 11:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Comment thread source/dcautil/dcautil.c
Copilot AI review requested due to automatic review settings July 17, 2026 11:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/dcautil/dcautil.c
Comment thread source/test/dcautils/dcautilTest.cpp
Comment thread source/dcautil/dcautil.c
Copilot AI review requested due to automatic review settings July 17, 2026 11:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread source/test/dcautils/dcautilTest.cpp
Comment thread source/test/dcautils/dcautilTest.cpp
Comment thread source/test/dcautils/dcautilTest.cpp
Comment thread source/test/dcautils/dcautilTest.cpp
Comment thread source/dcautil/dcautil.c
@shibu-kv
shibu-kv merged commit ace78e1 into develop Jul 17, 2026
16 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants