-
Notifications
You must be signed in to change notification settings - Fork 850
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 grace period for pod kill chaos #663
add grace period for pod kill chaos #663
Conversation
Codecov Report
@@ Coverage Diff @@
## master #663 +/- ##
==========================================
- Coverage 55.78% 55.69% -0.09%
==========================================
Files 68 67 -1
Lines 4383 4417 +34
==========================================
+ Hits 2445 2460 +15
- Misses 1768 1782 +14
- Partials 170 175 +5
Continue to review full report at Codecov.
|
@namco1992 Thanks for your contribution! Can you fix |
272d60b
to
98a2df9
Compare
Signed-off-by: Gong Mengnan <[email protected]>
98a2df9
to
bba3bca
Compare
@cwen0 Fixed, thanks! |
/run-e2e-tests |
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 |
@namco1992 merge failed. |
/run-all-tests |
Signed-off-by: Gong Mengnan <[email protected]>Signed-off-by: lilin90 <[email protected]>
Signed-off-by: Gong Mengnan <[email protected]> Signed-off-by: shonge <[email protected]>
Signed-off-by: Gong Mengnan <[email protected]>
What problem does this PR solve?
Solve #569.
What is changed and how does it work?
Added a
gracePeriod
field in thePodChaos
CRD.Check List
Tests
Code changes
Related changes
gracePeriod
field.Does this PR introduce a user-facing change?