Skip to content

ci: Make find for test reports more specific#47458

Merged
neersighted merged 1 commit intomoby:masterfrom
vvoland:ci-reports-better-find
Feb 29, 2024
Merged

ci: Make find for test reports more specific#47458
neersighted merged 1 commit intomoby:masterfrom
vvoland:ci-reports-better-find

Conversation

@vvoland
Copy link
Copy Markdown
Contributor

@vvoland vvoland commented Feb 27, 2024

Don't use all *.json files blindly, take only these that are likely to be reports from go test.

An attempt to prevent https://github.com/moby/moby/actions/runs/8064122892/job/22029045229?pr=47395 from happening

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@vvoland vvoland added status/2-code-review kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. area/testing kind/bugfix PR's that fix bugs labels Feb 27, 2024
@vvoland vvoland added this to the v-future milestone Feb 27, 2024
@vvoland vvoland self-assigned this Feb 27, 2024
Comment thread .github/workflows/.test.yml
Don't use all `*.json` files blindly, take only these that are likely to
be reports from go test.
Also, use `find ... -exec` instead of piping results to `xargs`.

Signed-off-by: Paweł Gronowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/testing kind/bugfix PR's that fix bugs kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. process/cherry-picked status/4-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants