Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docker: Increase ulimit to infinity #1003

Merged
merged 1 commit into from
Apr 2, 2024
Merged

Conversation

jramosf
Copy link
Contributor

@jramosf jramosf commented Mar 19, 2024

The current Colima VM (Ubuntu) is using the default value of 1024 file descriptors. This is insufficient for software such as Kafka that requires a high amount of open files, even in development environments.

https://docs.confluent.io/platform/current/kafka/deployment.html#file-descriptors-and-mmap

Since the principal purpose of the Colima VM is just to run docker, this sets the value to infinity to limit to the OS limits.

@abiosoft
Copy link
Owner

Thanks :)

@abiosoft abiosoft enabled auto-merge (squash) March 20, 2024 06:22
@abiosoft
Copy link
Owner

@jramosf can you kindly rebase? I fixed the broken CI.

@jramosf jramosf disabled auto-merge April 2, 2024 06:32
The current Colima VM (Ubuntu) is using the default value of 1024 file descriptors. This is insufficient for software such as Kafka that requires a high amount of open files, even in development environments.

https://docs.confluent.io/platform/current/kafka/deployment.html

Since the principal purpose of the Colima VM is just to run docker, this sets the value to infinity to limit to the OS limits.
@jramosf
Copy link
Contributor Author

jramosf commented Apr 2, 2024

@jramosf can you kindly rebase? I fixed the broken CI.

@abiosoft just rebased it, thanks.

@abiosoft abiosoft merged commit bd99ce2 into abiosoft:main Apr 2, 2024
10 checks passed
@jramosf jramosf deleted the patch-1 branch April 2, 2024 14:52
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