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

Fix test with proxy #309

Merged
merged 3 commits into from
Mar 5, 2020
Merged

Conversation

oraluben
Copy link
Contributor

@oraluben oraluben commented Mar 5, 2020

What problem does this PR solve?

#299

What is changed and how does it work?

Check List

Tests

  • No code

Code changes

  • Has CI related scripts change

Side effects

Related changes

Does this PR introduce a user-facing change?:

NONE

@codecov-io
Copy link

codecov-io commented Mar 5, 2020

Codecov Report

Merging #309 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #309   +/-   ##
=======================================
  Coverage   60.62%   60.62%           
=======================================
  Files          40       40           
  Lines        2494     2494           
=======================================
  Hits         1512     1512           
  Misses        882      882           
  Partials      100      100

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 eac1943...ff9356d. Read the comment docs.

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.

@YangKeao PTAL

Copy link
Member

@YangKeao YangKeao left a comment

Choose a reason for hiding this comment

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

LGTM

Makefile Outdated Show resolved Hide resolved
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

Copy link
Member

@fewdan fewdan 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

@sre-bot
Copy link
Contributor

sre-bot commented Mar 5, 2020

/run-all-tests

@zhouqiang-cl
Copy link
Contributor

/merge

1 similar comment
@zhouqiang-cl
Copy link
Contributor

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Mar 5, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Mar 5, 2020

@oraluben merge failed.

@zhouqiang-cl
Copy link
Contributor

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Mar 5, 2020

/run-all-tests

@sre-bot sre-bot merged commit 18c33e5 into chaos-mesh:master Mar 5, 2020
@you06
Copy link
Member

you06 commented Mar 5, 2020

The score is updated, check it out at https://pingcap.com/community-cn/tidb-usability-challenge/

@oraluben oraluben deleted the fix-test-with-proxy branch March 16, 2020 04:34
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