ci: use a matrix for test configurations

[cagatay-y]

Use a test configuration matrix to increase architecture testing parity, decrease CI configuration duplication and run tests at parallel, at the cost of increased compute and storage usage.

[mkroening]

Thanks a lot for the PR! The CI times look very promising. :)

I added some comments.

Also, I'd like the cache to be restricted to main before merging this.

Nit: I think we can remove the parentheses from most literals.

[cagatay-y]

After configuring with a single matrix, I think it may make more sense to create a basis action for the setup and use it with two separate matrices for the networked and not networked jobs.

[cagatay-y]

I think the cache change works because the last cache item from this pull request on the list was added right before the commit that disabled caching. I normally wouldn't mix the two changes as they are not really related but I forgot that I was on the branch with the cache change.

[mkroening]

I think it may make more sense to create a basis action for the setup

I'd rather not split the setup into different files. Having all this together makes it straightforward to understand for newcomers.

How about removing the netdev dimension, but explicitly adding it in the required cases? That would be inverting the description and work similar to what you are doing with flags. I am not sure, if that is possible, though.

think the cache change works

Looks good! 👍

I normally wouldn't mix the two changes as they are not really related but I forgot that I was on the branch with the cache change.

Feel free to wildly rebase and force-push on PRs. You could also extract that into another PR, so we can merge that already. :)
In the end, you can also squash most PRs. If you need any assistance with that, I am happy to help.

PS: You can re-request a review from me in the top right, when you want me to review again.

[cagatay-y]

How about removing the netdev dimension, but explicitly adding it in the required cases? That would be inverting the description and work similar to what you are doing with flags. I am not sure, if that is possible, though.

I tried that but the configurations added via include overrode each other.

I opened a separate PR for the cache issue. After it is merged I will rebase this one on it and request a review.

[mkroening]

This looks great. Only one more nit. Thereafter, I'll merge this. 👍

[cagatay-y]

This looks great. Only one more nit. Thereafter, I'll merge this. 👍

Makes a lot of sense, I had to resize the left pane via inspect element during testing 😂