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.

Sign up for GitHub

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

Jump to bottom

AWS kube-up: set net.ipv4.neigh.default.gc_thresh1=0 to avoid ARP over-caching #27682

Merged
merged 1 commit into from
Jun 19, 2016
Merged

AWS kube-up: set net.ipv4.neigh.default.gc_thresh1=0 to avoid ARP over-caching #27682

merged 1 commit into from
Jun 19, 2016
+14 −0

Conversation

justinsb
Copy link
Member

This works around a linux kernel bug with overly aggressive caching of
ARP entries, which was causing problems when we reused IP addresses in
VPCs, for example with an ASG in a relatively small subnet.

See #23395 for more explanation.

Fixes #23395

@justinsb
AWS kube-up: set net.ipv4.neigh.default.gc_thresh1=0
d62fe2a 
This works around a linux kernel bug with overly aggressive caching of
ARP entries, which was causing problems when we reused IP addresses in
VPCs, for example with an ASG in a relatively small subnet.

See kubernetes#23395 for more explanation.

Fixes kubernetes#23395
@justinsb justinsb added this to the v1.3 milestone Jun 19, 2016
@justinsb justinsb added the priority/backlog Higher priority than priority/awaiting-more-evidence. label Jun 19, 2016
@k8s-github-robot k8s-github-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. release-note-label-needed labels Jun 19, 2016
@roberthbailey roberthbailey added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 19, 2016
@k8s-github-robot k8s-github-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 19, 2016
@roberthbailey roberthbailey added lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-label-needed labels Jun 19, 2016
@justinsb justinsb mentioned this pull request Jun 19, 2016
Closed
@davidopp
Copy link
Member

Failure is

SUCCESS  ./hack/../hack/verify-generated-docs.sh    140s
Verifying ./hack/../hack/verify-generated-protobuf.sh
+++ [0618 21:36:44] Verifying Prerequisites....
+++ [0618 21:36:44] Building Docker image kube-build:build-fb0e8c9e20.
+++ [0618 21:36:45] Running build command....
Error: No such image or container: kube-build-data-fb0e8c9e20
+++ [0618 21:36:45] Creating data container
Go version: go version go1.6.2 linux/amd64
+++ [0618 21:36:47] Building the toolchain targets:
    k8s.io/kubernetes/hack/cmd/teststale
+++ [0618 21:36:48] Building go targets for linux/amd64:
    cmd/libs/go2idl/go-to-protobuf
    cmd/libs/go2idl/go-to-protobuf/protoc-gen-gogo
+++ [0618 21:36:55] Placing binaries

@davidopp
Copy link
Member

@k8s-bot test this issue: #27693

@k8s-bot
Copy link

k8s-bot commented Jun 19, 2016

GCE e2e build/test passed for commit d62fe2a.

@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-bot
Copy link

k8s-bot commented Jun 19, 2016

GCE e2e build/test passed for commit d62fe2a.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit a911948 into kubernetes:master Jun 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@roberthbailey roberthbailey

Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/backlog Higher priority than priority/awaiting-more-evidence. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
v1.3
Development

Successfully merging this pull request may close these issues.
6 participants
@justinsb @davidopp @k8s-bot @k8s-github-robot @googlebot @roberthbailey