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

xds: Allow Gradle to use more memory when building interop - GCE #9354

Merged
merged 1 commit into from
Jul 9, 2022

Conversation

sergiitk
Copy link
Member

@sergiitk sergiitk commented Jul 9, 2022

Same as #9347, but for GCE framework too (xds and xds_v3 jobs).

Should fix "Expiring Daemon because JVM heap space is exhausted".

PR #9269 probably pushed the build
over the edge, but there's been evidence via flakes for a good while
that we've been reaching the limit.

b/238334438

@sergiitk sergiitk requested a review from ejona86 July 9, 2022 01:28
Same as grpc#9347, but for GCE framework too (xds and xds_v3 jobs).

Should fix "Expiring Daemon because JVM heap space is exhausted".

PR grpc#9269 probably pushed the build
over the edge, but there's been evidence via flakes for a good while
that we've been reaching the limit.

b/238334438
@sergiitk sergiitk force-pushed the xds-interop-gradle-memory branch from e8b554a to 094efe8 Compare July 9, 2022 01:29
@sergiitk
Copy link
Member Author

sergiitk commented Jul 9, 2022

cc @YifeiZhuang

@sergiitk
Copy link
Member Author

sergiitk commented Jul 9, 2022

@sergiitk
Copy link
Member Author

sergiitk commented Jul 9, 2022

Confirming this fixed gradle: BUILD SUCCESSFUL in 2m 45s

@sergiitk sergiitk enabled auto-merge (squash) July 9, 2022 01:38
@sergiitk sergiitk merged commit d7a6c1e into grpc:master Jul 9, 2022
@sergiitk sergiitk deleted the xds-interop-gradle-memory branch July 9, 2022 03:02
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants