Skip to content

Cap the Node heap below the pod memory limit #31

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.

Sign up for GitHub

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

Open
andrzej-janczak wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Cap the Node heap below the pod memory limit #31

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions docker/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,9 @@ RUN npm install -g \
@codacy/codacy-cloud-cli \
@codacy/analysis-cli

# Heap cap below the 2Gi pod limit: a Node process that outgrows it dies with a readable heap error, not a silent cgroup OOM kill
ENV NODE_OPTIONS=--max-old-space-size=1536

# Pre-bake skills — Claude loads via --plugin-dir, Gemini installs from local path
# ADD'ing the master ref content makes Docker invalidate this layer whenever codacy-skills master moves,
# so a fresh `docker build` always gets the latest skills without --no-cache.
Expand Down