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

Add netperf tester tool #10

Merged
merged 1 commit into from Aug 18, 2015

Conversation

Projects
None yet
5 participants
@BenTheElder
Member

BenTheElder commented Aug 18, 2015

Adds a tool for running netperf tests on a k8s cluster.

Add netperf tester tool
Adds a tool for running netperrf tests on a k8s cluster.

@googlebot googlebot added the cla: yes label Aug 18, 2015

@BenTheElder

This comment has been minimized.

Member

BenTheElder commented Aug 18, 2015

/cc @thockin

@BenTheElder

This comment has been minimized.

Member

BenTheElder commented Aug 18, 2015

From the previous PR over at: kubernetes/kubernetes#12838

Here are some example results from a GCE cluster for the iptables and userspace proxy.

Tests were run with:

export KUBERNETES_PROVIDER=gce
export NUM_MINIONS=4
export KUBE_ENABLE_CLUSTER_MONITORING=google
export KUBE_ENABLE_CLUSTER_UI=false
./../kubernetes/cluster/kube-up.sh
go run netperf-tester/netperf_tester.go --output=$HOME/gce-netperf-userspace.csv
@thockin

This comment has been minimized.

Member

thockin commented Aug 18, 2015

lgtm

@erictune

This comment has been minimized.

Member

erictune commented Aug 18, 2015

X and y axis labels reversed.
On Aug 17, 2015 8:51 PM, "Tim Hockin" notifications@github.com wrote:

lgtm


Reply to this email directly or view it on GitHub
#10 (comment).

@BenTheElder

This comment has been minimized.

Member

BenTheElder commented Aug 18, 2015

Huh, I thought I copied over the right one. Fixing, thanks. I mis-labeled
them when I made the histogram the first time on the issue over at k8s.
Swapping the image out now.
On Aug 18, 2015 9:31 AM, "Eric Tune" notifications@github.com wrote:

X and y axis labels reversed.
On Aug 17, 2015 8:51 PM, "Tim Hockin" notifications@github.com wrote:

lgtm


Reply to this email directly or view it on GitHub
#10 (comment).


Reply to this email directly or view it on GitHub
#10 (comment).

@BenTheElder

This comment has been minimized.

Member

BenTheElder commented Aug 18, 2015

Should be fixed now.

brendandburns added a commit that referenced this pull request Aug 18, 2015

@brendandburns brendandburns merged commit c48f599 into kubernetes:master Aug 18, 2015

2 checks passed

cla/google All necessary CLAs are signed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment