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

Removed ulimit reduction #206

Merged
merged 1 commit into from
Feb 26, 2019
Merged

Conversation

micahhausler
Copy link
Member

Issue #, if available:

Fixes #193
Reverts #186

Description of changes:

Revert ulimit reduction

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@micahhausler micahhausler requested a review from mogren February 26, 2019 17:04
@micahhausler micahhausler merged commit 9f20002 into awslabs:master Feb 26, 2019
@cpanato
Copy link

cpanato commented Mar 2, 2019

hello this change is applied in the current AMI for eks version 1.11 ? (ami-0eeeef929db40543c)

@lekspyl
Copy link

lekspyl commented Mar 6, 2019

Hello,

looks like amazon-eks-node-1.11-v20190220 is the latest AMI and it does not contain this patch.
Could you tell when is it expected for a new AMI to be released? Asking due to #193 which I can't make to work on the current AMI version. If there is any working way to override max file descriptors value I will be more than thankful if you'll share it.

@dmarkey
Copy link

dmarkey commented Mar 12, 2019

Hey, is there an ETA for a release with this fix in it?

@annawinkler
Copy link

I would also be interested to find out when this fix will be released - thank you! ✨

@sherifkayad
Copy link

@micahhausler ,

Is it possible to know when that fix will be available/when a new AMI is going to be available?

@tabern
Copy link

tabern commented Mar 19, 2019

This will be in the marketplace-available optimized AMI with K8s version 1.11.8. See aws/containers-roadmap#188

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.

8 participants