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: use ubuntu for Android & double ram to 4096M #5922

Closed
wants to merge 1 commit into from

Conversation

cakebaker
Copy link
Contributor

This PR uses ubuntu instead of macos for the Android workflow and doubles its ram size to 4096M. It's based on the following comment: ReactiveCircus/android-emulator-runner#46 (comment)

Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/rm/rm1 (fails in this run but passes in the 'main' branch)

@sylvestre sylvestre force-pushed the ci_use_linux_for_android branch from 222c01c to 9c8b221 Compare January 31, 2024 10:43
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/rm/rm1 (passes in this run but fails in the 'main' branch)

@cre4ture
Copy link
Contributor

cre4ture commented Feb 3, 2024

for me just switching to ubuntu-latest wasn't improving anything. In fact, it was much slower until I read the documentation which tells to additionally activate KVM. This helped a lot and caused linux to be actually faster than macos. It's all in the PR #5939

@cakebaker
Copy link
Contributor Author

Closing this PR in favor of #5939

@cakebaker cakebaker closed this Feb 4, 2024
@cakebaker cakebaker deleted the ci_use_linux_for_android branch February 4, 2024 12:41
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