-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Add stress test tool #1331
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 stress test tool #1331
Conversation
This adds a `stress` binary to help stress test containerd. It is different from a benchmarking tool as it only gives a simple summary at the end. It is built to run long, multi hour/day stress tests across builds of containerd. Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #1331 +/- ##
=======================================
Coverage 35.56% 35.56%
=======================================
Files 23 23
Lines 2874 2874
=======================================
Hits 1022 1022
Misses 1627 1627
Partials 225 225 Continue to review full report at Codecov.
|
Nice; definitely a good tool to have around--this can replace my hacky gist that I use instead of bucketbench for direct stress :) |
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.
LGTM
LGTM Does this make sense as a subcommand of |
@stevvooe i think its fine separate. its small and does a very specific job |
tctx, cancel := context.WithTimeout(ctx, c.Duration) | ||
go func() { | ||
s := make(chan os.Signal, 1) | ||
signal.Notify(s, syscall.SIGTERM, syscall.SIGINT) |
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.
os.Interrupt for windows portability?
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.
i don't think it matters to have that with term being there
Fix using `http` for localhost, 127.0.0.1 and ::1 by default
This adds a
stress
binary to help stress test containerd. It isdifferent from a benchmarking tool as it only gives a simple summary at
the end.
It is built to run long, multi hour/day stress tests across builds of
containerd.
Output:
As a side note, i have a 24hr stress test running on an instance for the latest build of containerd+runc.
Signed-off-by: Michael Crosby crosbymichael@gmail.com