Skip to content

ci: give analysis and QA enough memory - #389

Merged
tobiasKaminsky merged 2 commits into
mainfrom
ci/analysis-heap-max-runner
Sep 25, 2026
Merged

tobiasKaminsky merged 2 commits into
mainfrom
ci/analysis-heap-max-runner

Conversation

@AndyScherzinger

Copy link
Copy Markdown
Member

The analysis workflow writes org.gradle.jvmargs=-Xmx1g into ~/.gradle/gradle.properties, which overrides each app's own gradle.properties. On Talk, lintGplayDebug fails with Java heap space, and on push the wrapper still reports success without uploading a lint baseline (Talk's master.html has not been updated since 2026-06-01), so PRs are compared against a stale report.

Jobs on the plain ubuntu-latest label also land on the smaller self-hosted runners, which get killed mid-build with "The runner has received a shutdown signal".

Changes:

  • analysis.yml: heap -Xmx1g → -Xmx6g, runs-on: ubuntu-latest-max
  • qa.yml: runs-on: ubuntu-latest-max

Verified on nextcloud/talk-android#6643: lint now completes (Analysis there only fails on the stale baseline) and QA passes on ubuntu-latest-max.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@tobiasKaminsky

Copy link
Copy Markdown
Member

@SystemKeeper how much RAM/memory do our -max worker have?
I think "only" 6GB, so allowing jvm to use 100% of memory might kill it on other apps.

@SystemKeeper

Copy link
Copy Markdown

14GB

AndyScherzinger and others added 2 commits September 25, 2026 10:22
The analysis workflow wrote org.gradle.jvmargs=-Xmx1g into
~/.gradle/gradle.properties, which overrides the app's own
gradle.properties. lintGplayDebug then fails with "Java heap space",
and on push the wrapper still reports success without uploading a lint
baseline, so PR lint results are compared against a stale report.

Raise the heap to 6g and run analysis and QA on ubuntu-latest-max:
jobs landing on the smaller runners behind ubuntu-latest get killed
mid-build ("The runner has received a shutdown signal").

Verified on nextcloud/talk-android#6643, where lint now completes.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@tobiasKaminsky
tobiasKaminsky force-pushed the ci/analysis-heap-max-runner branch from 01a34e4 to 6270370 Compare September 25, 2026 08:23
@tobiasKaminsky
tobiasKaminsky merged commit 9bf7bac into main Sep 25, 2026
5 checks passed
@tobiasKaminsky
tobiasKaminsky deleted the ci/analysis-heap-max-runner branch September 25, 2026 08:46
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.

3 participants