Skip to content

[ci] test azp coverage artifact logging #14691

Merged
lizan merged 7 commits intoenvoyproxy:mainfrom
asraa:add-artifact-path
Jan 29, 2021
Merged

[ci] test azp coverage artifact logging #14691
lizan merged 7 commits intoenvoyproxy:mainfrom
asraa:add-artifact-path

Conversation

@asraa
Copy link
Contributor

@asraa asraa commented Jan 13, 2021

Signed-off-by: Asra Ali asraa@google.com

Commit Message: Adds a log error message when coverage fails that points to the coverage report produced.
image

Risk Level: Low
Testing: Manually testing

asraa added 4 commits January 13, 2021 16:48
Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
@asraa
Copy link
Contributor Author

asraa commented Jan 14, 2021

still wip, will ping when ready

Signed-off-by: Asra Ali <asraa@google.com>
@asraa asraa changed the title [DRAFT] test azp coverage artifact logging [ci] test azp coverage artifact logging Jan 15, 2021
Signed-off-by: Asra Ali <asraa@google.com>
@asraa
Copy link
Contributor Author

asraa commented Jan 15, 2021

Ready for review

Base automatically changed from master to main January 15, 2021 23:02
Signed-off-by: Asra Ali <asraa@google.com>
@asraa
Copy link
Contributor Author

asraa commented Jan 29, 2021

friendly ping @lizan ?

@lizan lizan merged commit b941f6e into envoyproxy:main Jan 29, 2021
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