Skip to content

Conversation

@datho7561
Copy link
Contributor

@datho7561 datho7561 commented Nov 10, 2025

This lets us smoke test the binaries for every platform instead of just linux x86_64.

The builds are run in parallel, which looks a bit messy in the output but hopefully will mean it takes around 10 minutes to get all the results.

I wrote the windows part so that it can be expanded to include arm64 in the future.

Also fixes a bug preventing the linux arm64 binary from being used.

@datho7561 datho7561 force-pushed the 1114-smoke-tests-matrix branch 2 times, most recently from 3747ab9 to ab147a1 Compare November 10, 2025 17:55
@datho7561
Copy link
Contributor Author

(Draft because it's not working and I still need to add windows)

@datho7561 datho7561 force-pushed the 1114-smoke-tests-matrix branch 3 times, most recently from b8ae47b to 59431a3 Compare November 10, 2025 18:28
@datho7561
Copy link
Contributor Author

Not pleased with having to wait for all the binaries to build in order to run the tests with them; I'll have to look into if I can improve that

@datho7561 datho7561 force-pushed the 1114-smoke-tests-matrix branch 6 times, most recently from 9beaf9e to f0c9738 Compare November 10, 2025 20:57
@datho7561
Copy link
Contributor Author

datho7561 commented Nov 10, 2025

I'm checking to see if the binary is actually broken on apple arm64..... nope, I named the binary wrong in the build process. Hopefully that's fixed; after the tests pass, I'll clean up the commits and mark this as ready for review.

@datho7561 datho7561 force-pushed the 1114-smoke-tests-matrix branch 9 times, most recently from 690902c to 44b55ce Compare November 11, 2025 19:27
@datho7561 datho7561 changed the title [WIP] matrix build for smoke tests Smoke test all binaries using matrix build Nov 11, 2025
@datho7561 datho7561 marked this pull request as ready for review November 11, 2025 19:28
@datho7561
Copy link
Contributor Author

Ugh seems like I'll need to adjust the timeouts for Windows for it to be consistent :(

@datho7561 datho7561 force-pushed the 1114-smoke-tests-matrix branch from 44b55ce to 94c43b3 Compare November 11, 2025 19:48
@datho7561
Copy link
Contributor Author

going to push a few blank amends to check the consistency of the tests passing.

@datho7561 datho7561 force-pushed the 1114-smoke-tests-matrix branch from 94c43b3 to 812cc85 Compare November 11, 2025 20:39
This lets us smoke test the binaries for every platform instead of just linux
x86_64.
The builds are run in parallel, which looks a bit messy in the output
but hopefully will mean it takes around 10 minutes to get all the
results.
I wrote the windows part so that it can be expanded to include arm64 in
the future.

Signed-off-by: David Thompson <davthomp@redhat.com>
@datho7561 datho7561 force-pushed the 1114-smoke-tests-matrix branch from 812cc85 to 5705851 Compare November 11, 2025 21:20
@datho7561 datho7561 requested a review from fbricon November 11, 2025 21:21
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.

1 participant