Skip to content

Collect and maintain branch coverage#1575

Merged
vihangm merged 2 commits intopixie-io:mainfrom
vihangm:branch
Jun 26, 2023
Merged

Collect and maintain branch coverage#1575
vihangm merged 2 commits intopixie-io:mainfrom
vihangm:branch

Conversation

@vihangm
Copy link
Copy Markdown
Member

@vihangm vihangm commented Jun 26, 2023

Summary: gcov doesn't collect branch coverage by default and lcov
drops branch coverage from the files if it exists.
This change ensures that we maintain and track branch coverage.

Type of change: /kind cleanup

Test Plan: Ran the coverage build and then the coverage cleanup
script. Ensured that it now maintains branch coverage.

@vihangm vihangm requested a review from a team as a code owner June 26, 2023 18:16
@vihangm vihangm requested a review from a team June 26, 2023 18:16
Signed-off-by: Vihang Mehta <vihang@pixielabs.ai>
Signed-off-by: Vihang Mehta <vihang@pixielabs.ai>
@vihangm vihangm merged commit 1eaeee2 into pixie-io:main Jun 26, 2023
@vihangm vihangm deleted the branch branch June 26, 2023 18:39
ddelnano pushed a commit to k8sstormcenter/pixie that referenced this pull request Feb 25, 2026
Summary: gcov doesn't collect branch coverage by default and lcov
drops branch coverage from the files if it exists.
This change ensures that we maintain and track branch coverage.

Type of change: /kind cleanup

Test Plan: Ran the coverage build and then the coverage cleanup
script. Ensured that it now maintains branch coverage.

---------

Signed-off-by: Vihang Mehta <vihang@pixielabs.ai>
ddelnano pushed a commit to k8sstormcenter/pixie that referenced this pull request Feb 25, 2026
Summary: gcov doesn't collect branch coverage by default and lcov
drops branch coverage from the files if it exists.
This change ensures that we maintain and track branch coverage.

Type of change: /kind cleanup

Test Plan: Ran the coverage build and then the coverage cleanup
script. Ensured that it now maintains branch coverage.

---------

Signed-off-by: Vihang Mehta <vihang@pixielabs.ai>
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.

2 participants