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

[RunAllTests] Use faster Bazel setup with 4.0 LTS #2693

Merged
merged 4 commits into from
Feb 17, 2021

Conversation

BenHenning
Copy link
Member

This is a follow-up from #1904.

This is meant to speed up Bazel in CI by using a much leaner repository containing custom Bazel setup (just the Android tools from https://github.com/oppia/oppia-bazel-tools versus cloning the much larger oppia/bazel repository). This is expected to shave up to 30 seconds off each test action requiring Bazel (depending on how long the Bazel setup action itself takes).

BenHenning and others added 3 commits February 11, 2021 16:23
Switch to previously approved Bazel setup action.
Switch back to uses: abhinavsingh/setup-bazel@v3 for setup. The one we used previously doesn't seem to work currently with 4.0: jwlawson/actions-setup-bazel#11
@BenHenning
Copy link
Member Author

Ah nice. This is much faster: compute affected targets took ~18s instead of the ~1.5 minutes from before. There will still be a slight delay from cloning the custom tools repo, but it should still be a nice improvement over what the previous setup was doing.

Copy link
Contributor

@jcqli jcqli left a comment

Choose a reason for hiding this comment

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

LGTM, waiting on tests to finish

@jcqli jcqli assigned BenHenning and unassigned jcqli Feb 12, 2021
@BenHenning
Copy link
Member Author

Thanks! Looks like another JVM crash in CI. :( Restarting failing tests & enabling auto-merge.

@BenHenning BenHenning enabled auto-merge (squash) February 13, 2021 01:11
@BenHenning BenHenning merged commit dbb8ae7 into develop Feb 17, 2021
@BenHenning BenHenning deleted the speed-up-bazel-ci-tests branch February 17, 2021 03:18
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