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

Change default arp cache size on nodes #13034

Merged
merged 1 commit into from
Feb 23, 2017
Merged

Conversation

pecameron
Copy link
Contributor

In OCP clusters with large numbers of routes (greater than the value of
net.ipv4.neigh.default.gc_thresh3, which is 1024 by default) the ARP
cache is not large enough to accommodate for all the entries needed by
the nodes running the router pods.

This change increases the cache size.

bug 1425388
https://bugzilla.redhat.com/show_bug.cgi?id=1425388

Signed-off-by: Phil Cameron [email protected]

In OCP clusters with large numbers of routes (greater than the value of
net.ipv4.neigh.default.gc_thresh3, which is 1024 by default) the ARP
cache is not large enough to accommodate for all the entries needed by
the nodes running the router pods.

This change increases the cache size.

bug 1425388
https://bugzilla.redhat.com/show_bug.cgi?id=1425388

Signed-off-by: Phil Cameron <[email protected]>
@eparis
Copy link
Member

eparis commented Feb 21, 2017

@jeremyeder is this what we want?

@jeremyeder
Copy link
Contributor

Yep -- no known downside; double-confirmed with kernel networking folks as well. Link to mail thread is in the BZ. Hitting this starts happening near 900+ routes. Several customers have encountered it and implemented the documented tuning. It's ready to be promoted to default for OpenShift. I hope this is possible in 3.5.

@pecameron
Copy link
Contributor Author

@eparis are you ready to do the merge? (I can't)

@eparis
Copy link
Member

eparis commented Feb 21, 2017

[merge] This PR will only land in 3.6. I'm super scared to make changes in 3.5....

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to ba84207

@jeremyeder
Copy link
Contributor

@eparis ack, 3.6 only -- thanks!

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/403/) (Base Commit: 2d20080)

@eparis
Copy link
Member

eparis commented Feb 22, 2017

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to ba84207

@imcsk8
Copy link
Contributor

imcsk8 commented Feb 22, 2017

LGTM!

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/468/) (Base Commit: 80b2f6d)

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.

5 participants