-
Notifications
You must be signed in to change notification settings - Fork 270
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 flaky in Android build as out of memory #1360
Comments
The obvious next thing to try, and which I guess we should try, is to bump the max heap size further. It's not clear that will solve the problem, though — so we'll leave this issue open, and monitor for a while before declaring it fixed. Some builds of example apps within the Flutter test suite were flaking with a similar error a few months ago, flutter/flutter#156063; they bumped the heap size from 4 GiB to 8 GiB although not confident the heap size was the real issue: In particular I checked the latest example flake output from that thread, from 2024-11-06: Those flakes did eventually stop, as seen in that thread. The thread has no theories on what else might have fixed them. |
Fixes, potentially, zulip#1360. Even after the bump to 4 GiB, CI is still occasionally flaking with a "Java heap space" error in the `android` suite: https://github.com/zulip/zulip-flutter/actions/runs/13152142737/job/36817594452?pr=1327 So try bumping the memory limit still further. It's not entirely clear this will be enough: zulip#1360 (comment) but we'll find out.
Fixes, potentially, zulip#1360. Even after the bump to 4 GiB, CI is still occasionally flaking with a "Java heap space" error in the `android` suite: https://github.com/zulip/zulip-flutter/actions/runs/13152142737/job/36817594452?pr=1327 So try bumping the memory limit still further. It's not entirely clear this will be enough (or that an even bigger number would be): zulip#1360 (comment) but we'll find out.
… Hmm, never mind: that failed run I linked above was actually from before #1331 and even #1324. What happened is that I got got by the classic GitHub bug where it shows a page that's a mix of old and new data: the tab where I had #1327 open was showing the new version I'd pushed earlier today, and my comment today, and below that was saying that checks had failed. And the link for details led to that CI run I mentioned above. And this was a couple of hours after I'd pushed that new version, so if checks had completed and failed that meant the checks on the new version had failed, right? But in reality, that part of the page was just stale. It was still showing the old status from before my new revision — the revision I'd pushed hours earlier, and which was shown on the very same page right above the checks status. Reloading the page shows that CI succeeded, as expected. I guess if we do at some point see a flake of this kind post-#1331, we can re-open this thread. |
We're continuing to occasionally see the symptom of #1330 — CI runs sometimes fail with a "Java heap space" error in the
android
suite, as Gradle runs out of memory. Here's an example today:Before #1324, this failure was happening in 100% of CI runs — that was #1330.
With #1324 we bumped the Gradle JVM max heap size from 1.5 GiB to 3 GiB. That made CI mostly pass, but it continued to sometimes fail:
https://chat.zulip.org/#narrow/channel/243-mobile-team/topic/Gradle.20out.20of.20memory/near/2082072
With #1331 we bumped it further to 4 GiB. The failure today, quoted above, was with 4 GiB.
The text was updated successfully, but these errors were encountered: