Skip to content

Conversation

@mishraomp
Copy link
Collaborator

@mishraomp mishraomp commented Dec 26, 2025

Increase heap size from 50 to 150MB to avoid unnecessary cpu spikes

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:


Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Copilot AI review requested due to automatic review settings December 26, 2025 21:47
@mishraomp mishraomp enabled auto-merge (squash) December 26, 2025 21:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR increases memory allocations to address CPU spikes caused by aggressive garbage collection. The changes adjust both the Kubernetes resource requests and the Node.js heap size limit to provide more headroom for the application.

  • Node.js heap size increased from 50MB to 150MB (3x increase)
  • Kubernetes memory requests increased from 75Mi to 200Mi (~2.67x increase) for both init and main containers
  • Proper headroom maintained between heap size (150MB) and container memory (200Mi) for non-heap memory usage

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
charts/app/templates/backend/templates/deployment.yaml Increases memory requests from 75Mi to 200Mi for both the init container (Flyway migrations) and main backend container
backend/Dockerfile Increases Node.js max-old-space-size from 50MB to 150MB and updates the corresponding comment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-project-automation github-project-automation bot moved this from New to Waiting in DevOps (NR) Dec 26, 2025
@mishraomp mishraomp merged commit 1d3b6a5 into main Dec 26, 2025
34 checks passed
@mishraomp mishraomp deleted the fix/cpu-spike branch December 26, 2025 21:57
@github-project-automation github-project-automation bot moved this from Waiting to Done in DevOps (NR) Dec 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants