-
Notifications
You must be signed in to change notification settings - Fork 9
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
Implement CPU stressor #374
Conversation
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
8c1ff63
to
0661a8f
Compare
79d17ae
to
0661a8f
Compare
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
c231177
to
3868609
Compare
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
3868609
to
88bd863
Compare
pkg/agent/integration_test.go
Outdated
|
||
ctx := context.TODO() | ||
|
||
// start the container with the agent using a fake upstream address because the agent requires it |
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 might be a copy-paste artifact, right? (At least I don't see any address)
cmd/agent/commands/stress.go
Outdated
return err | ||
} | ||
|
||
return s.Apply(cmd.Context(), d, duration) |
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.
In other commands, we use the agent to apply the disruption rather than calling the disruptor directly. Is there a reason to not it like that for this command?
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.
Good catch. It should follow the same pattern as other commands. I forgot to change this part from the initial PoC implementation.
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
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.
Looking great, awesome work!
Hi, |
Hi @fslev This experiemntal feature is still not released in the API. We hope to make this available soon. |
Description
Implements a CPU stress disruptor
Checklist:
make lint
) and all checks pass.make test
) and all tests pass.make integration-xxx
for affected packages)make e2e-xxx
fordisruptors
, orcluster
related changes)