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

ci: add node failure test #355

Merged
merged 14 commits into from
May 7, 2022

Conversation

amyangfei
Copy link
Collaborator

  • Refactor test case TestNodeFailure, adding server master and executor node restart and check job is running(or after failover) normally.

@amyangfei amyangfei changed the title ci: add node failure test [WIP] ci: add node failure test Apr 29, 2022
@codecov-commenter
Copy link

codecov-commenter commented Apr 29, 2022

Codecov Report

Merging #355 (ef34260) into master (80697fd) will decrease coverage by 0.0189%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             master       #355        +/-   ##
================================================
- Coverage   54.8521%   54.8332%   -0.0190%     
================================================
  Files           135        135                
  Lines         10552      10552                
================================================
- Hits           5788       5786         -2     
- Misses         4311       4313         +2     
  Partials        453        453                

@amyangfei amyangfei changed the title [WIP] ci: add node failure test ci: add node failure test May 5, 2022
@amyangfei amyangfei added the status/ptal Could you please take a look? label May 5, 2022
@amyangfei amyangfei requested review from liuzix and maxshuang May 5, 2022 08:55
@amyangfei amyangfei mentioned this pull request May 6, 2022
7 tasks
@amyangfei amyangfei merged commit 913dabf into hanfei1991:master May 7, 2022
@amyangfei amyangfei deleted the e2e-add-chaos-test-part-two branch May 7, 2022 05:46
liuzix pushed a commit to liuzix/microcosm that referenced this pull request May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/ptal Could you please take a look?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants