-
Notifications
You must be signed in to change notification settings - Fork 810
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
[update] add container-wise StressChaos in dashboard #794
Conversation
Codecov Report
@@ Coverage Diff @@
## master #794 +/- ##
==========================================
- Coverage 55.78% 53.81% -1.98%
==========================================
Files 68 71 +3
Lines 4383 4776 +393
==========================================
+ Hits 2445 2570 +125
- Misses 1768 2002 +234
- Partials 170 204 +34
Continue to review full report at Codecov.
|
@@ -342,6 +342,10 @@ func (s *Service) createStressChaos(exp *core.ExperimentInfo) error { | |||
chaos.Spec.Duration = &exp.Scheduler.Duration | |||
} | |||
|
|||
if exp.Target.StressChaos.ContainerName != nil { |
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.
Do we need to check if exp.Target.StressChaos
is nil
?
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.
Seems all Create Experiment API
s should need to be checked since all ChaosInfo
s are pointers. 😢
We can plan a refactor
for our server APIs.
Signed-off-by: Yue Yang <g1enyy0ung@gmail.com>
@cwen0 The dashboard has already followed the changes, PTAL. |
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
/merge |
/run-all-tests |
@fewdan merge failed. |
/merge |
/run-all-tests |
* update Signed-off-by: “fewdan” <fewdan@hotmail.com> * chore: add stress container_name in dashboard Signed-off-by: Yue Yang <g1enyy0ung@gmail.com> Co-authored-by: Yue Yang <g1enyy0ung@gmail.com> Co-authored-by: ti-srebot <66930949+ti-srebot@users.noreply.github.com>
Signed-off-by: “fewdan” fewdan@hotmail.com
What problem does this PR solve?
After #759 merged, the container name was added to the StressChaos.
This PR adds container name to the dashboard
What is changed and how does it work?
add container-wise StressChaos in dashboard
Tests
Code changes
Side effects
Related changes
Does this PR introduce a user-facing change?