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

Restore some kube-proxy defaults #1699

Merged
merged 1 commit into from
Jul 14, 2017
Merged

Conversation

r2d4
Copy link
Contributor

@r2d4 r2d4 commented Jul 14, 2017

Set some kube-proxy defaults that got unset through the new way of
configuring kube-proxy. Add more delay to the ip tables syncing reduces
idle CPU load a lot.

See
#1158 (comment)

$ ps uaxS | grep localkube

Without these options

USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root      3643 89.3 19.7 11469032 404500 ?     Ssl  18:04   0:55 /usr/local/bin/localkube --dns-domain=cluster.local --node-ip=192.168.99.101 --generate-certs=false --logtostderr
=true --enable-dns=false

With

USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root      3901 13.4 13.0 11403508 267744 ?     Ssl  17:52   1:00 /usr/local/bin/localkube --dns-domain=cluster.local --node-ip=192.168.99.101 --generate-certs=false --logtostderr
=true --enable-dns=false

Set some kube-proxy defaults that got unset through the new way of
configuring kube-proxy.  Add more delay to the ip tables syncing reduces
idle CPU load a lot.

See
kubernetes#1158 (comment)
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 14, 2017
@codecov-io
Copy link

Codecov Report

Merging #1699 into master will decrease coverage by 0.1%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1699      +/-   ##
==========================================
- Coverage   38.55%   38.45%   -0.11%     
==========================================
  Files          51       51              
  Lines        2674     2681       +7     
==========================================
  Hits         1031     1031              
- Misses       1463     1470       +7     
  Partials      180      180
Impacted Files Coverage Δ
pkg/localkube/proxy.go 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c06d18...afb7c63. Read the comment docs.

@r2d4 r2d4 merged commit 601e762 into kubernetes:master Jul 14, 2017
@r2d4 r2d4 deleted the proxy-defaults branch July 14, 2017 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants