Skip to content

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

Merged
swethasukumarr merged 2 commits into
developfrom
feature/RDKEMW-17673
Jun 26, 2026
Merged

RDKEMW-17673 : Add info coverage gate to CI#81
swethasukumarr merged 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.

@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.

@swethasukumarr swethasukumarr merged commit 4ef4a75 into develop Jun 26, 2026
15 checks passed
@swethasukumarr swethasukumarr deleted the feature/RDKEMW-17673 branch June 26, 2026 17:24
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 26, 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