feat: increase default agent memory limit to 6GB and enable swap#1360
Merged
feat: increase default agent memory limit to 6GB and enable swap#1360
Conversation
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Increase default agent container memory limit from 2GB to 6GB
feat: increase default agent memory limit to 6GB and enable swap
Mar 18, 2026
Mossaka
approved these changes
Mar 18, 2026
Contributor
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR increases the agent container’s default memory limit and changes swap configuration to reduce OOM-kills (exit 137) when running heavier workloads on GitHub Actions runners.
Changes:
- Increase default agent
mem_limitfrom2gto6g. - Change default agent
memswap_limitfrom2g(no swap) to-1(unlimited swap), while keeping “no swap” when an explicit--memory-limitis provided. - Update CLI help text, JSDoc defaults, and unit test expectations.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/types.ts | Updates documented default for WrapperConfig.memoryLimit to 6g. |
| src/docker-manager.ts | Adjusts generated docker-compose resource limits (memory + swap behavior). |
| src/docker-manager.test.ts | Updates expected defaults for agent mem_limit/memswap_limit. |
| src/cli.ts | Updates --memory-limit help text and default value. |
Comments suppressed due to low confidence (1)
src/types.ts:322
- The
WrapperConfig.memoryLimitdocs now say the default is 6g, but the swap behavior changed as well (default swap enabled/unlimited vs swap disabled when the user explicitly sets--memory-limit). It would help to document this coupling here (or reference a separatememswapsetting) so API/SDK consumers understand why providingmemoryLimitalso changes swap behavior.
/**
* Memory limit for the agent execution container
*
* Accepts Docker memory format: a positive integer followed by a unit suffix
* (b, k, m, g). Controls the maximum amount of memory the container can use.
*
* @default '6g'
* @example '4g'
* @example '512m'
*/
memoryLimit?: string;
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+1215
to
+1216
| 'Memory limit for the agent container (e.g., 4g, 6g, 8g, 512m). Default: 6g', | ||
| '6g' |
| // with swap unlimited so the kernel can use swap as a pressure valve | ||
| // instead of immediately OOM-killing the agent process. | ||
| mem_limit: config.memoryLimit || '6g', | ||
| memswap_limit: config.memoryLimit ? config.memoryLimit : '-1', // Disable swap when user specifies limit |
Comment on lines
1160
to
1163
| // Verify resource limits | ||
| expect(agent.mem_limit).toBe('2g'); | ||
| expect(agent.memswap_limit).toBe('2g'); | ||
| expect(agent.mem_limit).toBe('6g'); | ||
| expect(agent.memswap_limit).toBe('-1'); | ||
| expect(agent.pids_limit).toBe(1000); |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Contributor
|
@Mossaka Z3 was hitting memory issues. |
Contributor
|
Smoke Test Results — Run 23444610776
Overall: PASS | Author:
|
Contributor
Chroot Version Comparison Results
Result: ❌ Not all versions match — Python and Node.js versions differ between host and chroot environments.
|
Contributor
|
Smoke Test Results
Overall: PASS
|
Closed
Contributor
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Contributor
Smoke Test ResultsPR titles:
Warning
|
This was referenced Mar 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Agent container defaults to 2GB with no swap, causing OOM kills (exit 137) on GitHub Actions runners with 7GB RAM. Four repos consistently fail: prettier, k9s, stern, cowboy.
Changes
mem_limit:2g→6g(~85% of 7GB runner RAM)memswap_limit:2g→-1(unlimited swap as kernel pressure valve instead of immediate OOM kill)--memory-limitstill disables swap (memswap_limit=mem_limit), preserving existing behavior for explicit configs📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.