-
Notifications
You must be signed in to change notification settings - Fork 81
chore: Add solution for Docker problems #369
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
Conversation
✨ Coder.com for PR #369 deployed! It will be updated on every commit.
|
You can use a DaemonSet to apply the following values to increase the limits: | ||
|
||
``` | ||
sudo sysctl -w kernel.keys.maxkeys=20000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggest we expand on this and add a daemonset example, we can reuse some of the content from the inotify watch limit doc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jawnsy Do you mind reviewing the content I added re: DaemonSets? I'm not super familiar w/ these so am open to all suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the long delay, I wanted to test on our cluster before approving. LGTM!
Co-authored-by: Jonathan Yu <jonathan@coder.com>
Co-authored-by: Jonathan Yu <jonathan@coder.com>
@khorne3 I left some really poor english, and not a ton of insight here. Feel free to entirely refactor, reword, or rewrite the whole thing.
We ran into this problem with many users packed on the same instance... if you'd like better elaboration just let me know!