Skip to content

tests: fix the rest failed nextgen tests#9819

Merged
ti-chi-bot[bot] merged 2 commits intotikv:masterfrom
rleungx:fix-test-26h1-13
Oct 21, 2025
Merged

tests: fix the rest failed nextgen tests#9819
ti-chi-bot[bot] merged 2 commits intotikv:masterfrom
rleungx:fix-test-26h1-13

Conversation

@rleungx
Copy link
Copy Markdown
Member

@rleungx rleungx commented Oct 13, 2025 •
edited
Loading

What problem does this PR solve?

Issue Number: close #9806

What is changed and how does it work?

Check List

Tests

  • Unit test

Release note

None.

@ti-chi-bot ti-chi-bot bot added dco-signoff: yes Indicates the PR's author has signed the dco. release-note-none Denotes a PR that doesn't merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 13, 2025
@ti-chi-bot ti-chi-bot bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 13, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 13, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 34.78261% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.85%. Comparing base (14ac685) to head (a6bc47a).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #9819      +/-   ##
==========================================
- Coverage   76.87%   76.85%   -0.02%     
==========================================
  Files         491      491              
  Lines       78475    78583     +108     
==========================================
+ Hits        60324    60397      +73     
- Misses      14457    14476      +19     
- Partials     3694     3710      +16
Flag Coverage Δ
unittests 76.85% <34.78%> (-0.02%) ⬇️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ti-chi-bot ti-chi-bot bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 13, 2025
@rleungx rleungx force-pushed the fix-test-26h1-13 branch 2 times, most recently from a1838c1 to 312a204 Compare October 14, 2025 06:59
@ti-chi-bot ti-chi-bot bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 14, 2025
@rleungx rleungx changed the title tests: fix TestGCStateManager tests: fix the rest failed nextgen tests Oct 14, 2025
@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 14, 2025

/test pull-unit-test-next-gen

@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 14, 2025

/test pull-build-next-gen

@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 14, 2025

/test pull-unit-test-next-gen

1 similar comment
@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 14, 2025

/test pull-unit-test-next-gen

@ti-chi-bot ti-chi-bot bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 15, 2025
@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 15, 2025

/retest

@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 15, 2025

/test pull-unit-test-next-gen

4 similar comments
@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 15, 2025

/test pull-unit-test-next-gen

@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 15, 2025

/test pull-unit-test-next-gen

@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 15, 2025

/test pull-unit-test-next-gen

@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 16, 2025

/test pull-unit-test-next-gen

@rleungx rleungx requested review from JmPotato and lhy1024 and removed request for JmPotato October 16, 2025 06:56
@ti-chi-bot ti-chi-bot bot added needs-1-more-lgtm Indicates a PR needs 1 more LGTM. approved labels Oct 16, 2025
MyonKeminta
MyonKeminta reviewed Oct 17, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@MyonKeminta MyonKeminta left a comment

Choose a reason for hiding this comment

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

GC part LGTM

@rleungx
fix TestGCStateManager
9d19548 
Signed-off-by: Ryan Leung <rleungx@gmail.com>
@rleungx
fix
a6bc47a 
Signed-off-by: Ryan Leung <rleungx@gmail.com>
@ti-chi-bot
Copy link
Copy Markdown
Contributor

ti-chi-bot bot commented Oct 21, 2025

@rleungx: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
non-block/pull-unit-test-next-gen a6bc47a link false /test pull-unit-test-next-gen

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 21, 2025

/retest

@ti-chi-bot
Copy link
Copy Markdown
Contributor

ti-chi-bot bot commented Oct 21, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bufferflies, JmPotato

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [JmPotato,bufferflies]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot added lgtm and removed needs-1-more-lgtm Indicates a PR needs 1 more LGTM. labels Oct 21, 2025
@ti-chi-bot
Copy link
Copy Markdown
Contributor

ti-chi-bot bot commented Oct 21, 2025

[LGTM Timeline notifier]

Timeline:

  • 2025-10-16 09:01:39.944317291 +0000 UTC m=+344006.021569851: ☑️ agreed by bufferflies.
  • 2025-10-21 05:18:39.493117949 +0000 UTC m=+762625.570370509: ☑️ agreed by JmPotato.

@ti-chi-bot ti-chi-bot bot merged commit 538a218 into tikv:master Oct 21, 2025
35 of 40 checks passed
@rleungx rleungx deleted the fix-test-26h1-13 branch October 21, 2025 07:05
@rleungx
Copy link
Copy Markdown
Member Author

rleungx commented Oct 22, 2025

/cherry-pick release-nextgen-20251011

@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Oct 22, 2025

@rleungx: new pull request created to branch release-nextgen-20251011: #9860.
But this PR has conflicts, please resolve them!

Details

In response to this:

/cherry-pick release-nextgen-20251011

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

@ti-chi-bot ti-chi-bot mentioned this pull request Oct 22, 2025
Merged
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this pull request Oct 22, 2025
@rleungx @ti-chi-bot
This is an automated cherry-pick of tikv#9819
ce7b16b 
close tikv#9806

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
ti-chi-bot bot pushed a commit that referenced this pull request Oct 23, 2025
@ti-chi-bot @rleungx
tests: fix the rest failed nextgen tests (#9819) (#9860)
54510bd 
close #9806

Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: Ryan Leung <rleungx@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JmPotato JmPotato JmPotato approved these changes

@bufferflies bufferflies bufferflies approved these changes

@lhy1024 lhy1024 Awaiting requested review from lhy1024

+1 more reviewer

@MyonKeminta MyonKeminta MyonKeminta left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved dco-signoff: yes Indicates the PR's author has signed the dco. lgtm release-note-none Denotes a PR that doesn't merit a release note. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Stabilize nextgen testing

5 participants

@rleungx @ti-chi-bot @JmPotato @MyonKeminta @bufferflies
⚡