feat(ci): add GHA to report when Bazel test ownership drops below 70% threshold#62985
Conversation
5fd89f4 to
ac4f111
Compare
|
Warning Test ownership level below 70% on this branch. We aim to have ~70% of Bazel test targets to help us (dev-infra) and you get insight into the impact of tests on CI times. This is a non-blocking requirement, but keeping on top of test attribution to maintain this threshold is appreciated. See the list of tag variable names to use here, and reach out to #discuss-dev-infra if you need to add/change this list. See an example of how to use the variables here. |
b4f64d7 to
e6edff6
Compare
39d1358 to
6021409
Compare
af0dd5c to
08eddc1
Compare
| env: | ||
| PERCENTAGE: ${{ steps.bazel-query.outputs.PERCENTAGE }} | ||
| with: | ||
| script: | |
There was a problem hiding this comment.
sooooooooooooo much cleaner
|
@Strum355 could you add a Notion page documenting what test ownership is and link that from the message: From reading the two kinds of messages:
It's not clear what "test ownership" means, how to get the number higher etc. |
This PR adds a non-blocking GHA check to report when a branch's Bazel test ownership drops below 70%. See example messaging below to see how it looks like: https://github.com/sourcegraph/sourcegraph/pull/62985#issuecomment-2139439084. The message will be updated if the threshold is reached/breached whenever the branch changes.
Test plan
Extensive iteration in this PR, see below message https://github.com/sourcegraph/sourcegraph/pull/62985#issuecomment-2139439084
Changelog