Skip to content

FDN-4261: downsize heap for dependency-api and dependency-www#886

Merged
nicolagi merged 1 commit intomainfrom
FDN-4261-downsize-heap
Feb 16, 2026
Merged

FDN-4261: downsize heap for dependency-api and dependency-www#886
nicolagi merged 1 commit intomainfrom
FDN-4261-downsize-heap

Conversation

@nicolagi
Copy link
Contributor

@nicolagi nicolagi commented Feb 16, 2026

Summary

  • dependency-api: heap 2100m → 1536m, container 3000Mi → 2Gi
    • Live data P95 281M with periodic spikes to ~1G (batch operations)
    • 1536m provides headroom for spikes while reducing waste
  • dependency-www: heap 2100m → 512m, container 2700Mi → 768Mi
    • Live data P95 84M, max 278M
    • 512m gives ~2x headroom over peak

Test plan

  • ArgoCD diff passes
  • Monitor for Degenerated GCs after deploy
  • Validate with WARN_MS=1 SINCE=5m ./scripts/validate-gc-pauses.sh dependency-api-live
  • Validate with WARN_MS=1 SINCE=5m ./scripts/validate-gc-pauses.sh dependency-www-live

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Optimized memory resource allocations for system efficiency across application deployments.

dependency-api: heap 2100m → 1536m, container 3000Mi → 2Gi
  Live data P95 281M with spikes to ~1G. 1536m gives headroom for spikes.

dependency-www: heap 2100m → 512m, container 2700Mi → 768Mi
  Live data P95 84M, max 278M. 512m gives ~2x headroom over peak.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 16, 2026

📝 Walkthrough

Walkthrough

Two Helm configuration files updated: dependency-api increases container memory limits and requests while reducing JVM heap, and dependency-www reduces both container memory and JVM memory allocations.

Changes

Cohort / File(s) Summary
Helm Chart Resource Configs
deploy/dependency-api/values.yaml, deploy/dependency-www/values.yaml
Container memory limits/requests and JVM heap configurations adjusted; dependency-api increases container memory (3000Mi→2Gi) while reducing JVM heap (2100m→1536m); dependency-www reduces both container memory (2700Mi→768Mi) and JVM heap (2100m→512m).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title clearly and specifically describes the main change: downsizing JVM heap allocations for two Kubernetes deployments (dependency-api and dependency-www), which aligns with the core objective of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@flow-tech
Copy link
Contributor

Argo diff output for dependency-www:
===== apps/Deployment production/dependency-www-live ======
551c551
<           value: -Xms2100m -Xmx2100m -XX:+UseShenandoahGC -XX:+HeapDumpOnOutOfMemoryError
---
>           value: -Xms512m -Xmx512m -XX:+UseShenandoahGC -XX:+HeapDumpOnOutOfMemoryError
639c639
<             memory: 2700Mi
---
>             memory: 768Mi
642c642
<             memory: 2700Mi
---
>             memory: 768Mi

===== datadoghq.com/DatadogMonitor datadog/dependency-www-errorrate ======
146d145
<   - generated:kubernetes

@flow-tech
Copy link
Contributor

Argo diff output for dependency-api:
===== apps/Deployment production/dependency-api-live ======
551c551
<           value: -Xms2100m -Xmx2100m -XX:+UseShenandoahGC -XX:+HeapDumpOnOutOfMemoryError
---
>           value: -Xms1536m -Xmx1536m -XX:+UseShenandoahGC -XX:+HeapDumpOnOutOfMemoryError
639c639
<             memory: 3000Mi
---
>             memory: 2Gi
642c642
<             memory: 3000Mi
---
>             memory: 2Gi

===== datadoghq.com/DatadogMonitor datadog/dependency-api-errorrate ======
146d145
<   - generated:kubernetes

@nicolagi
Copy link
Contributor Author

Screenshot 2026-02-16 at 11 32 55 Screenshot 2026-02-16 at 11 33 17

@nicolagi nicolagi merged commit d55e9da into main Feb 16, 2026
7 checks passed
@nicolagi nicolagi deleted the FDN-4261-downsize-heap branch February 16, 2026 11:47
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