-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Slow down creation of additional pods in density.go e2e #10020
Conversation
wg.Add(minionCount) | ||
for i := 1; i <= minionCount; i++ { | ||
name := additionalPodsPrefix + "-" + strconv.Itoa(i) | ||
labels := map[string]string{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can probably be moved outside of the loop.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
GCE e2e build/test failed for commit 21c4ec3ac71d1d71a83bd3b5944016c9b8b182e6. |
LGTM, short of small nit. |
GCE e2e build/test failed for commit 45263c3. |
@k8s-bot retest this please |
GCE e2e build/test passed for commit 45263c3. |
Issue? Risk Level? Milestone? Thanks. |
Since this is a test I shall merger it without an associated milestone. |
Slow down creation of additional pods in density.go e2e
Thanks @satnam6502. Issue is measuring performance in general. Risk level is low since it's only changing the test and actually the logic being changed is disabled by default as for now. |
for i := 1; i <= minionCount; i++ { | ||
name := additionalPodsPrefix + "-" + strconv.Itoa(i) | ||
go createRunningPod(&wg, c, name, ns, "gcr.io/google_containers/pause:go", podLabels) | ||
time.Sleep(200 * time.Millisecond) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: if you're trying to rate limit to 5qps you can use util.throttle
No description provided.