Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: Test on several Bazel and Ubuntu versions. #521

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

CI: Test on several Bazel and Ubuntu versions. #521

wants to merge 1 commit into from

Conversation

smolkaj
Copy link
Member

@smolkaj smolkaj commented Dec 18, 2024

Also makes some improvements to the CI script taken from https://github.com/google/netkat.
They will allow us to make better use of the cache while avoiding hitting GitHub's cache limits.

@smolkaj smolkaj changed the title Ci CI: Test on several Bazel and Ubuntu versions. Dec 18, 2024
@smolkaj smolkaj force-pushed the ci branch 3 times, most recently from 2b388bf to 876ea89 Compare December 18, 2024 05:08
Also makes some improvements to the CI script taken from https://github.com/google/netkat.
They will allow us to make better use of the cache while avoiding hitting GitHub's cache limits.

Signed-off-by: Steffen Smolka <[email protected]>
@smolkaj smolkaj marked this pull request as ready for review December 18, 2024 18:21
@smolkaj
Copy link
Member Author

smolkaj commented Dec 18, 2024

This is now ready for review, @jonathan-dilorenzo and @chrispsommers.
Looking mostly for a sanity check.

strategy:
matrix:
# We only test on the oldest version we want to support and latest.
# We trust that things also work for versions in the middle.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's brave ;)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot less brave than what we have today, which is to only test a single version.

Copy link
Contributor

@jonathan-dilorenzo jonathan-dilorenzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me, but I don't know these github workflows very well, but I assume that it would run in the automated check done on this PR, so given that it passes...

@smolkaj
Copy link
Member Author

smolkaj commented Dec 18, 2024

FYI: Since this is changing the name of some checks, we need to bypass branch protections to submit and/or update our branch protections rules (probably afterwards).

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