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.

Sign up for GitHub

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

Jump to bottom

TestMicroserviceTSOClient/TestUpdateAfterResetTSO is unstable #7381

Closed
lhy1024 opened this issue Nov 16, 2023 · 4 comments · Fixed by #7385
Closed

TestMicroserviceTSOClient/TestUpdateAfterResetTSO is unstable #7381

lhy1024 opened this issue Nov 16, 2023 · 4 comments · Fixed by #7385
Assignees
@lhy1024
Labels
type/ci The issue is related to CI.

Comments

@lhy1024
Copy link
Contributor

lhy1024 commented Nov 16, 2023

Flaky Test

Which jobs are failing

2023-11-16T02:33:28.4044391Z     client_test.go:317: 
2023-11-16T02:33:28.4046089Z         	Error Trace:	/home/runner/work/pd/pd/tests/integrations/tso/client_test.go:317
2023-11-16T02:33:28.4047611Z         	Error:      	Should not be: "pd3"
2023-11-16T02:33:28.4049040Z         	Test:       	TestMicroserviceTSOClient/TestUpdateAfterResetTSO

CI link

https://github.com/tikv/pd/actions/runs/6885411338/job/18729884805?pr=7367

Reason for failure (if possible)

Anything else
@lhy1024 lhy1024 added the type/ci The issue is related to CI. label Nov 16, 2023
@lhy1024
Copy link
Contributor Author

lhy1024 commented Nov 16, 2023

https://github.com/tikv/pd/actions/runs/6885411338/job/18729510702?pr=7367

@lhy1024 lhy1024 mentioned this issue Nov 16, 2023
Closed
@CabinfeverB
Copy link
Member

https://github.com/tikv/pd/actions/runs/6885833531/job/18730587679?pr=7378

@lhy1024
Copy link
Contributor Author

lhy1024 commented Nov 16, 2023

https://github.com/tikv/pd/actions/runs/6887165881/job/18733945329?pr=7368

@lhy1024 lhy1024 self-assigned this Nov 16, 2023
@lhy1024 lhy1024 mentioned this issue Nov 16, 2023
Merged
@lhy1024
Copy link
Contributor Author

lhy1024 commented Nov 16, 2023

It is introduced by #7301

ti-chi-bot bot added a commit that referenced this issue Nov 16, 2023
@lhy1024 @ti-chi-bot
tests: make TestUpdateAfterResetTSO stable (#7385)
f2eaf23 
close #7381

Signed-off-by: lhy1024 <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@lhy1024 lhy1024 mentioned this issue Nov 21, 2023
Closed
rleungx pushed a commit to rleungx/pd that referenced this issue Dec 1, 2023
@lhy1024 @ti-chi-bot @rleungx
tests: make TestUpdateAfterResetTSO stable (tikv#7385)
225f90f 
close tikv#7381

Signed-off-by: lhy1024 <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lhy1024 lhy1024

Labels
type/ci The issue is related to CI.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests: make TestUpdateAfterResetTSO stable lhy1024/pd
2 participants
@lhy1024 @CabinfeverB