Closed evanyeyeye closed 10 months ago
[!WARNING]
Rate Limit Exceeded
@evanyeyeye has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 27 minutes and 2 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 61cd508fe06c10ab1a9f3b0ad2e358ebeeebf4d1 and 1dc41126944dcdffc1228698729571198b9cabe7.
The recent updates involve enhancing resource management for Docker containers in an autograding system. The changes allow for dynamic allocation of CPU cores and memory limits based on configurations or job-specific requirements. This is achieved by modifying method signatures to accept resource parameters and updating the logic to apply these constraints when creating and running Docker containers.
File Path | Change Summary |
---|---|
restful_tango/tangoREST.py |
Modified createTangoMachine method to handle vmObj as an optional parameter and set cores and memory based on the vmObj if provided. Also retrieves cores and memory limits from the Config module. |
vmms/localDocker.py |
Appended CPU and memory constraints to the args list used for running a Docker container in the runJob method. |
config.template.py |
Added declarations for Docker autograding container resource limits (DOCKER_CORES_LIMIT and DOCKER_MEMORY_LIMIT ) within the Config class. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Adds the following resource limits to new jobs:
Addresses: @139
Resource limits can be verified using
docker stats
andhtop
.