Skip to content

Comments

feat: Support for --cpus and --memory resource options#56

Merged
Mcrich23 merged 1 commit intoMcrich23:mainfrom
cstea:main
Feb 20, 2026
Merged

feat: Support for --cpus and --memory resource options#56
Mcrich23 merged 1 commit intoMcrich23:mainfrom
cstea:main

Conversation

@cstea
Copy link
Contributor

@cstea cstea commented Feb 20, 2026

Summary

  • Adds support for --cpus and --memory container resource limit flags during compose up
  • Reads values from deploy.resources.limits.cpus and deploy.resources.limits.memory in the compose file and passes them to the container run command

Test plan

  • Run compose up with a service that has deploy.resources.limits.cpus and deploy.resources.limits.memory set
  • Verify the container is created with the correct resource constraints
  • Run compose up without resource limits and verify no extra flags are passed

🤖 Generated with Claude Code

@Mcrich23
Copy link
Owner

@cstea I love this addition! Can you please add tests for it?

@cstea
Copy link
Contributor Author

cstea commented Feb 20, 2026

@Mcrich23 Tests added.

Copy link
Owner

@Mcrich23 Mcrich23 left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you.

@Mcrich23 Mcrich23 merged commit dabb2d4 into Mcrich23:main Feb 20, 2026
1 check 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.

2 participants