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

[ci]: Support code diff coverage #934

Merged
merged 2 commits into from
Sep 24, 2021
Merged

Conversation

xumia
Copy link
Collaborator

@xumia xumia commented Sep 20, 2021

Support code diff coverage

  1. Support to add the pull request checks for Code diff coverage in GitHub.
  2. Support to display the code diff coverage html report in Azure Pipelines.

@xumia
Copy link
Collaborator Author

xumia commented Sep 22, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xumia xumia marked this pull request as ready for review September 22, 2021 22:35
@xumia xumia requested a review from lguohan September 22, 2021 22:35
@xumia xumia marked this pull request as draft September 22, 2021 22:36
@xumia xumia requested a review from qiluo-msft September 23, 2021 03:29
@qiluo-msft qiluo-msft requested a review from kcudnik September 23, 2021 06:21
@kcudnik
Copy link
Collaborator

kcudnik commented Sep 23, 2021

how this entire feature should work ? is already framework for support that build in azpipeline and sairedis azpipeline configuration?

@xumia xumia marked this pull request as ready for review September 23, 2021 08:40
@xumia
Copy link
Collaborator Author

xumia commented Sep 23, 2021

@kcudnik , I added an azure devops extension to support it, see https://github.com/Azure/sonic-build-web/tree/main/code-diff-coverage. Azure DevOps still does not support it. It is a workaround, so we do not need to change it in all sub-modules, only enable it by a variable flag.

qiluo-msft
qiluo-msft previously approved these changes Sep 23, 2021
kcudnik
kcudnik previously approved these changes Sep 23, 2021
@xumia xumia dismissed stale reviews from kcudnik and qiluo-msft via 2e97f3a September 23, 2021 09:18
@xumia
Copy link
Collaborator Author

xumia commented Sep 24, 2021

@kcudnik , can we merge the code, there is one failure check, looks like it is not relative to the change.

@kcudnik kcudnik merged commit e545b17 into sonic-net:master Sep 24, 2021
@xumia xumia deleted the azp-diff-cover branch September 24, 2021 13:24
@xumia xumia restored the azp-diff-cover branch September 24, 2021 13:24
@xumia xumia deleted the azp-diff-cover branch September 24, 2021 13:24
@kcudnik
Copy link
Collaborator

kcudnik commented Sep 25, 2021

your diff coverage plugin broken full coverage stats file by file:
https://dev.azure.com/mssonic/build/_build/results?buildId=39429&view=results

clicking you can't see any more how much covered is each added file , please revert your change or make it possible to view both results

pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
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.

3 participants