Skip to content
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

Merged
merged 2 commits into from
Jul 1, 2020

Conversation

namco1992
Copy link
Contributor

What problem does this PR solve?

Solve #569.

What is changed and how does it work?

Added a gracePeriod field in the PodChaos CRD.

Check List

Tests

  • Unit test
  • E2E test
  • Manual test (add detailed scripts or steps below)

Code changes

  • Has Go code change

Related changes

  • Need to update the documentation to add gracePeriod field.

Does this PR introduce a user-facing change?

The gracePeriod field is available in podChaos to set the duration before the pod is deleted.

@CLAassistant
Copy link

CLAassistant commented Jun 29, 2020

CLA assistant check
All committers have signed the CLA.

@codecov-commenter
Copy link

codecov-commenter commented Jun 29, 2020

Codecov Report

Merging #663 into master will decrease coverage by 0.08%.
The diff coverage is 49.23%.

Impacted file tree graph

@@            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     
Impacted Files Coverage Δ
api/v1alpha1/podchaos_types.go 37.14% <ø> (ø)
controllers/networkchaos/ipset/ipset.go 37.14% <0.00%> (ø)
pkg/store/experiment/experiment.go 32.20% <0.00%> (-6.58%) ⬇️
pkg/utils/chaosdaemon.go 51.21% <0.00%> (ø)
pkg/utils/grpc.go 0.00% <0.00%> (ø)
pkg/webhook/inject/inject.go 73.49% <0.00%> (-0.68%) ⬇️
controllers/twophase/types.go 54.20% <40.00%> (-1.68%) ⬇️
controllers/networkchaos/netem/types.go 47.27% <50.00%> (ø)
pkg/utils/selector.go 53.15% <66.66%> (+0.91%) ⬆️
controllers/podchaos/containerkill/types.go 64.38% <100.00%> (ø)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fcf0ac8...1ac98c9. Read the comment docs.

@cwen0
Copy link
Member

cwen0 commented Jun 30, 2020

@namco1992 Thanks for your contribution! Can you fix DCO checker?

@namco1992 namco1992 force-pushed the pod-kill-grace-period branch from 272d60b to 98a2df9 Compare June 30, 2020 03:43
@namco1992 namco1992 force-pushed the pod-kill-grace-period branch from 98a2df9 to bba3bca Compare June 30, 2020 03:45
@namco1992
Copy link
Contributor Author

@cwen0 Fixed, thanks!

@zhouqiang-cl
Copy link
Contributor

/run-e2e-tests

Copy link
Contributor

@zhouqiang-cl zhouqiang-cl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zhouqiang-cl
Copy link
Contributor

/merge

@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot
Copy link
Contributor

@namco1992 merge failed.

@cwen0
Copy link
Member

cwen0 commented Jul 1, 2020

/run-all-tests

@zhouqiang-cl zhouqiang-cl merged commit 4a57402 into chaos-mesh:master Jul 1, 2020
lilin90 pushed a commit to lilin90/chaos-mesh that referenced this pull request Jul 1, 2020
shonge pushed a commit to shonge/chaos-mesh that referenced this pull request Jul 7, 2020
sjwsl pushed a commit to sjwsl/chaos-mesh that referenced this pull request May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants