Skip to content

RDKEMW-17673 : Add info coverage gate to CI#81

Open
swethasukumarr wants to merge 2 commits into
developfrom
feature/RDKEMW-17673
Open

RDKEMW-17673 : Add info coverage gate to CI#81
swethasukumarr wants to merge 2 commits into
developfrom
feature/RDKEMW-17673

Conversation

@swethasukumarr

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 17, 2026 03:41
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-cpp-client/81/rdkcentral/firebolt-native-transport

  • Commit: 7dad05a

Report detail: gist'

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

Adds an informational coverage “gate” to the CI workflow by producing/archiving LCOV artifacts from unit + component test runs, comparing them against a stored baseline on a build-metadata branch, and (on develop pushes) updating that baseline automatically.

Changes:

  • Generate and upload LCOV (filtered_coverage.info) artifacts for unit and component test jobs.
  • Add coverage_gate job to fetch the baseline and report threshold/regression status (non-blocking).
  • Add update_baseline job (push to develop only) plus a new Python script + tests to compute and persist baseline coverage.

Reviewed changes

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

File Description
.github/workflows/ci.yml Adds LCOV generation/artifacts, plus new coverage_gate and update_baseline jobs.
.github/scripts/compare_coverage.py New script to parse LCOV, compare vs baseline, print report, and optionally emit baseline JSON.
.github/scripts/compare_coverage_test.py New unittest suite covering parsing/analysis and end-to-end report scenarios.

Comment thread .github/scripts/compare_coverage.py
Comment thread .github/scripts/compare_coverage.py Outdated
Comment thread .github/scripts/compare_coverage_test.py
Comment thread .github/scripts/compare_coverage_test.py
Comment thread .github/workflows/ci.yml

@mhughesacn mhughesacn left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi @swethasukumarr : If the two python files are from RDK, not Comcast, then please add this to the end of NOTICE at top level:

Copyright 2026 RDK Management
Licensed under the Apache License, Version 2.0

or they should have Comcast Apache headers. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants