Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swss gcov support #1738

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

pettershao-ragilenetworks
Copy link
Contributor

@pettershao-ragilenetworks pettershao-ragilenetworks commented May 8, 2021

What I did
Introduce gcov-related automation script to test coverage rate for swss module
Why I did it
Attempt to integrate coverage report for each module in SONiC to Azure pipeline
How I verified it
Using Azure pipeline to check whether a coverage report can be checked after a pull request
Details if related

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
@lguohan
Copy link
Contributor

lguohan commented May 9, 2021

can you check the here. your azp changes has syntax error. https://dev.azure.com/mssonic/build/_build?definitionId=15

@lguohan lguohan requested a review from xumia May 9, 2021 16:54
@pettershao-ragilenetworks pettershao-ragilenetworks marked this pull request as draft June 25, 2021 01:31
@lgtm-com
Copy link

lgtm-com bot commented Jun 25, 2021

This pull request introduces 4 alerts when merging 61c2456 into c37cc1c - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'
  • 2 for Redundant assignment

Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
Signed-off-by: pettershao-ragilenetworks <pettershao@ragilenetworks.com>
@lgtm-com
Copy link

lgtm-com bot commented Jun 25, 2021

This pull request introduces 4 alerts when merging 117f241 into c37cc1c - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'
  • 2 for Redundant assignment

@lgtm-com
Copy link

lgtm-com bot commented Jun 25, 2021

This pull request introduces 4 alerts when merging 5670006 into c37cc1c - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'
  • 2 for Redundant assignment

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