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

ms, tso, tso proxy: enable stress test for TSO Proxy and adjust some test parameters. #6583

Closed

Conversation

binshi-bing
Copy link
Contributor

What problem does this PR solve?

Issue Number: Ref #5895

What is changed and how does it work?

enable stress test for TSO Proxy and adjust some test parameters.

Check List

Tests

  • Unit test
None.

@ti-chi-bot
Copy link
Contributor

ti-chi-bot bot commented Jun 9, 2023

[REVIEW NOTIFICATION]

This pull request has not been approved.

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot ti-chi-bot bot added the release-note-none Denotes a PR that doesn't merit a release note. label Jun 9, 2023
@ti-chi-bot ti-chi-bot bot requested review from lhy1024 and Yisaer June 9, 2023 04:01
@binshi-bing binshi-bing requested review from disksing and rleungx and removed request for Yisaer June 9, 2023 04:04
@codecov
Copy link

codecov bot commented Jun 9, 2023

Codecov Report

Patch coverage has no change and project coverage change: -1.21 ⚠️

Comparison is base (181e613) 74.75% compared to head (375be21) 73.55%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6583      +/-   ##
==========================================
- Coverage   74.75%   73.55%   -1.21%     
==========================================
  Files         417      417              
  Lines       42693    42693              
==========================================
- Hits        31915    31402     -513     
- Misses       7976     8528     +552     
+ Partials     2802     2763      -39     
Flag Coverage Δ
unittests 73.55% <ø> (-1.21%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 46 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@binshi-bing binshi-bing force-pushed the enable-tso-proxy-stress-test branch 4 times, most recently from eb95bec to b43638d Compare June 9, 2023 22:00
Signed-off-by: Bin Shi <binshi.bing@gmail.com>
Signed-off-by: Bin Shi <binshi.bing@gmail.com>
It grows to quick. Increase timeout value for now, and we can split the test jobs into different chunks later.

Signed-off-by: Bin Shi <binshi.bing@gmail.com>
Signed-off-by: Bin Shi <binshi.bing@gmail.com>
@binshi-bing binshi-bing force-pushed the enable-tso-proxy-stress-test branch 2 times, most recently from f2a05e6 to 3a9878f Compare June 9, 2023 23:55
… to timeout.

Signed-off-by: Bin Shi <binshi.bing@gmail.com>
@binshi-bing binshi-bing force-pushed the enable-tso-proxy-stress-test branch from 3a9878f to 375be21 Compare June 10, 2023 00:11
@binshi-bing
Copy link
Contributor Author

covered by #6585

@binshi-bing binshi-bing changed the title ms, tso: enable stress test for TSO Proxy and adjust some test parameters. ms, tso, TSO Proxy: enable stress test for TSO Proxy and adjust some test parameters. Jun 15, 2023
@binshi-bing binshi-bing changed the title ms, tso, TSO Proxy: enable stress test for TSO Proxy and adjust some test parameters. ms, tso, tso proxy: enable stress test for TSO Proxy and adjust some test parameters. Jun 15, 2023
@ti-chi-bot ti-chi-bot bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note-none Denotes a PR that doesn't merit a release note. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant