rebase by nhanasi · Pull Request #110 · rdkcentral/tr69hostif

nhanasi · 2025-04-03T19:49:17Z

No description provided.

CLAassistant · 2025-04-03T19:49:25Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
15 out of 18 committers have signed the CLA.

✅ NatrajMuthusamygithubpractice
✅ tpaul627
✅ shibu-kv
✅ madhubabutt
✅ DamianoBaroneSky
✅ AravindanNC
✅ nhanasi
✅ venkat0557
✅ leenaS-d
✅ Abhinavpv28
✅ MonekaLakshmi
✅ Alan-Ryan
✅ apatel859
✅ NareshM1702
✅ Vismalskumar0
❌ Tharun Kumar Venkatachalem
❌ udaykrishnag
❌ naveenkumarhanasi

Tharun Kumar Venkatachalem seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

github-advanced-security · 2025-07-16T12:07:46Z

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

[RDKEMW-5582] -[RDKE] tr69hostif.log contains any garbage data for the paramValue field

Reason for change: Set Gamepad RFC as True Test Procedure: verify the steps in Ticket description Risks: Medium Priority: P1 Signed-off-by: Tharun Kumar Venkatachalem <tharunkumar_venkatachalem@comcast.com>

RDK-56291 - [RDKE] Increase L2 Test Coverage For Remote Debugger : Target 80% [ Phase 2 ]

+    name: Test coverage report for release
+    runs-on: ubuntu-latest
+    container:
+      image: ghcr.io/rdkcentral/docker-rdk-ci:latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Run unit tests with coverage flags enabled
+        run: |
+          sh run_ut.sh --enable-cov 
+      - name: Caculate the code coverage summary
+        run: |
+          lcov --list tr69hostif_coverage.info | grep "Lines\|Total" > /tmp/coverage_summary.txt
+          cd -
+
+      - name: Update the coverage report to Pull request using actions
+        uses: actions/github-script@v4
+        with:
+          script: |
+            const fs = require('fs');
+            const lcov_result = fs.readFileSync('/tmp/coverage_summary.txt', 'utf8'); 
+
+            github.issues.createComment({
+              issue_number: context.issue.number,
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              body: 
+                    '## Code Coverage Summary \n' +
+                    ' ' +
+                    '```' +
+                    lcov_result + 
+                    '```'
+            });
+      - name: Generate the html report
+        run: |
+          genhtml tr69hostif_coverage.info --output-directory /tmp/coverage_report
+          cd -
+      - name: Upload the coverage report to Pull request using actions
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-report
+          path: /tmp/coverage_report


To fix the problem, explicitly set the permissions key for the job (or at the workflow root) to grant only the minimum required privileges. In this workflow, the job needs contents: read (to check out code) and issues: write (to create a comment on the pull request). The best way to fix this is to add a permissions block under the execute-unit-code-coverage-report-on-release job, specifying contents: read and issues: write. This change should be made directly under the job definition (after name: and before runs-on:) in .github/workflows/code-coverage.yml. No additional imports or definitions are needed.

1.2.3

rebase

* Update cov_build.sh * Update hostIf_utils.cpp * Update cov_build.sh * Update cov_build.sh * Update cov_build.sh * Update cov_build.sh * Update hostIf_utils.cpp

Rebase

…th function "hostIf_WiFi_SSID::get_Device_WiFi_SSID_Fields" and "81598460" fingerprint (#223) * Update Device_WiFi.cpp * Update Device_WiFi_SSID.cpp * Update Device_WiFi_EndPoint.cpp --------- Co-authored-by: nhanasi <navihansi@gmail.com> Co-authored-by: Venkata Bojja <39968865+venkat0557@users.noreply.github.com>

…ode from RDKE (#505) Reason for change: Deprecated DataModel Removal for HWSelftest and SNMP code from RDKE Test Procedure: Build and verify Risks: Medium Priority: P1 Signed-off-by: Tirumala, Madhubabu (Contractor) <Madhubabu_Tirumala@comcast.com> Co-authored-by: mtirum011 <madhubabu_tirumala@comcast.com>

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

tr69hostif release for 1.4.8 tag

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

* Refactor Docker exec command in L2-tests.yml * Update L2-tests.yml --------- Co-authored-by: nhanasi <navihansi@gmail.com>

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Co-authored-by: mtirum011 <madhubabu_tirumala@comcast.com>

Copilot