Skip to content

chore: updated memory for sonar#2619

Draft
aryamohanan wants to merge 3 commits into
mainfrom
chore-sonar-mem
Draft

chore: updated memory for sonar#2619
aryamohanan wants to merge 3 commits into
mainfrom
chore-sonar-mem

Conversation

@aryamohanan

@aryamohanan aryamohanan commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

We're seeing an issue with SonarCloud coverage. PRs that have coverage enabled may fail with the following error:
The bridge server is unresponsive. It might be because you don't have enough memory.

Increases the memory available to the SonarCloud analysis step by:

Increasing the pod memory request to 6 GiB.

  • Setting NODE_OPTIONS=--max-old-space-size=6144 to increase the maximum heap size available to the Node.js process used by the Sonar scanner.

  • Setting SONAR_SCANNER_OPTS=-Xmx4g to increase the JVM heap size available to the Sonar Scanner.

These changes are intended to prevent out-of-memory conditions during analysis and improve the reliability of SonarCloud coverage processing.

@aryamohanan aryamohanan force-pushed the chore-sonar-mem branch 5 times, most recently from 0d1be89 to b06af05 Compare June 30, 2026 04:42

@abhilash-sivan abhilash-sivan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Shall we merge this and unblock PR pipeline ? Or we can remove sonar scanning till this gets sorted!

Comment thread .tekton/tasks/tasks-results.yaml Outdated
- name: print-all-results
image: public.ecr.aws/docker/library/node:$(params.node-version)
imagePullPolicy: IfNotPresent
resources:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you try it without?
Because AFAIK this is only to reserve a certain amount when the container starts.

I expected that

"--max-old-space-size=6144"

^ this config will be enough

@sonarqubecloud

Copy link
Copy Markdown

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