Skip to content

Bump catchup RAM burst limit to match other pubnet mission limit#408

Merged
anupsdf merged 1 commit into
stellar:mainfrom
bboston7:catchup-limits
Jul 14, 2026
Merged

Bump catchup RAM burst limit to match other pubnet mission limit#408
anupsdf merged 1 commit into
stellar:mainfrom
bboston7:catchup-limits

Conversation

@bboston7

Copy link
Copy Markdown
Contributor

PR #404 bumped RAM burst limits to account for the RAM usage increase in in-memory rebuild on pubnet. However, parallel catchup uses a different resource specification and so runs in into the same OOM issue that #404 resolved. This PR bumps the catchup burst RAM limit to match the limit in #404.

PR stellar#404 bumped RAM burst limits to account for the RAM usage increase in
in-memory rebuild on pubnet. However, parallel catchup uses a different
resource specification and so runs in into the same OOM issue that stellar#404
resolved. This PR bumps the catchup burst RAM limit to match the limit
in stellar#404.
Copilot AI review requested due to automatic review settings July 14, 2026 00:14

Copilot AI 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.

Pull request overview

This PR increases the Kubernetes memory burst limit for the parallel catchup mission to align with the higher pubnet RAM limits introduced in PR #404, preventing OOM kills during in-memory rebuild peaks.

Changes:

  • Bump ParallelCatchupCoreResourceRequirements memory limit from 16384Mi to 24000Mi.
  • Update the inline comment describing the parallel catchup resource spec (though the units in the comment should be corrected; see review comment).

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

Comment thread src/FSLibrary/StellarKubeSpecs.fs
@anupsdf anupsdf merged commit 88b08a4 into stellar:main Jul 14, 2026
3 checks passed
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