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

DM TestRedirectRequestToLeader #4554

Closed
Tracked by #2246
Ehco1996 opened this issue Feb 10, 2022 · 0 comments · Fixed by #4556
Closed
Tracked by #2246

DM TestRedirectRequestToLeader #4554

Ehco1996 opened this issue Feb 10, 2022 · 0 comments · Fixed by #4556
Assignees
@Ehco1996
Labels
area/dm Issues or PRs related to DM. component/test Unit tests and integration tests component. type/enhancement The issue or PR belongs to an enhancement.

Comments

@Ehco1996
Copy link
Contributor

Which jobs are flaking?

verifyCI

Which test(s) are flaking?

TestRedirectRequestToLeader

https://ci.pingcap.net/blue/rest/organizations/jenkins/pipelines/ut-check/runs/1090/nodes/45/steps/49/log/?start=0

Jenkins logs or GitHub Actions link

[2022-02-09T10:01:31.274Z] goroutine 13617 [semacquire, 2 minutes]:
[2022-02-09T10:01:31.274Z] sync.runtime_Semacquire(0xc0032da1f0)
[2022-02-09T10:01:31.274Z] 	/usr/local/go/src/runtime/sema.go:56 +0x45
[2022-02-09T10:01:31.274Z] sync.(*WaitGroup).Wait(0xc0032da1e8)
[2022-02-09T10:01:31.274Z] 	/usr/local/go/src/sync/waitgroup.go:130 +0xe5
[2022-02-09T10:01:31.274Z] github.com/pingcap/tiflow/dm/dm/master.(*Server).Close(0xc0032da160)
[2022-02-09T10:01:31.274Z] 	/home/jenkins/agent/workspace/ut-check/tiflow/dm/dm/master/server.go:271 +0x15d
[2022-02-09T10:01:31.274Z] runtime.Goexit()
[2022-02-09T10:01:31.274Z] 	/usr/local/go/src/runtime/panic.go:632 +0x146
[2022-02-09T10:01:31.274Z] github.com/pingcap/check.(*C).stopNow(...)
[2022-02-09T10:01:31.274Z] 	/go/pkg/mod/github.com/pingcap/[email protected]/check.go:108
[2022-02-09T10:01:31.274Z] github.com/pingcap/check.(*C).Assert(0xc001906690, 0x5af7800, 0xc001edbd68, 0x7269578, 0x8e1f458, 0xc002d4b908, 0x1, 0x1)
[2022-02-09T10:01:31.274Z] 	/go/pkg/mod/github.com/pingcap/[email protected]/helpers.go:179 +0xc5
[2022-02-09T10:01:31.274Z] github.com/pingcap/tiflow/dm/dm/master.(*openAPISuite).TestRedirectRequestToLeader(0xc000b58560, 0xc001906690)
[2022-02-09T10:01:31.274Z] 	/home/jenkins/agent/workspace/ut-check/tiflow/dm/dm/master/openapi_test.go:121 +0xe46
[2022-02-09T10:01:31.274Z] reflect.Value.call(0x5eb1c40, 0xc000b58560, 0x1613, 0x6114672, 0x4, 0xc001cb7ef0, 0x1, 0x1, 0xc001cb7d08, 0x4b46bbe, ...)
[2022-02-09T10:01:31.274Z] 	/usr/local/go/src/reflect/value.go:476 +0x99c
[2022-02-09T10:01:31.274Z] reflect.Value.Call(0x5eb1c40, 0xc000b58560, 0x1613, 0xc001cb7ef0, 0x1, 0x1, 0xc001906780, 0x2, 0x2)
[2022-02-09T10:01:31.274Z] 	/usr/local/go/src/reflect/value.go:337 +0xd9
[2022-02-09T10:01:31.274Z] github.com/pingcap/check.(*suiteRunner).forkTest.func1(0xc001906690)
[2022-02-09T10:01:31.274Z] 	/go/pkg/mod/github.com/pingcap/[email protected]/check.go:850 +0xb3c
[2022-02-09T10:01:31.274Z] github.com/pingcap/check.(*suiteRunner).forkCall.func1(0xc0037983f0, 0xc001906690, 0xc00160c060)
[2022-02-09T10:01:31.274Z] 	/go/pkg/mod/github.com/pingcap/[email protected]/check.go:739 +0x11e
[2022-02-09T10:01:31.274Z] created by github.com/pingcap/check.(*suiteRunner).forkCall
[2022-02-09T10:01:31.274Z] 	/go/pkg/mod/github.com/pingcap/[email protected]/check.go:734 +0x4cd
[2022-02-09T10:01:31.274Z]

Anything else we need to know

  • Does this test exist for other branches as well?

  • Has there been a high frequency of failure lately?
@Ehco1996 Ehco1996 added component/test Unit tests and integration tests component. area/dm Issues or PRs related to DM. labels Feb 10, 2022
@Ehco1996 Ehco1996 self-assigned this Feb 10, 2022
This was referenced Feb 10, 2022
Closed
Merged
@Ehco1996 Ehco1996 added the type/enhancement The issue or PR belongs to an enhancement. label Feb 10, 2022
ti-chi-bot pushed a commit that referenced this issue Feb 14, 2022
@Ehco1996
test(dm): fix unsteable test TestRedirectRequestToLeader (#4556)
e230160 
close #4554
zhaoxinyu pushed a commit to zhaoxinyu/ticdc that referenced this issue Feb 16, 2022
@Ehco1996 @zhaoxinyu
test(dm): fix unsteable test TestRedirectRequestToLeader (pingcap#4556)
60557db 
close pingcap#4554
This was referenced Feb 22, 2022
Merged
Merged
ti-chi-bot added a commit that referenced this issue Feb 28, 2022
@ti-chi-bot
test(dm): fix unsteable test TestRedirectRequestToLeader (#4556) (#4649)
9d93277 
close #4554
ti-chi-bot added a commit that referenced this issue Mar 8, 2022
@ti-chi-bot
test(dm): fix unsteable test TestRedirectRequestToLeader (#4556) (#4650)
f391df6 
close #4554
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ehco1996 Ehco1996

Labels
area/dm Issues or PRs related to DM. component/test Unit tests and integration tests component. type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test(dm): fix unsteable test TestRedirectRequestToLeader
1 participant
@Ehco1996