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

goleak: goroutine in state sync.Cond.Wait, 5 minutes, with sync.runtime_notifyListWait on top of the stack #41777

Closed
Tracked by #41316
YangKeao opened this issue Feb 28, 2023 · 2 comments
Labels
affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.0 affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. component/test severity/major sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.

Comments

@YangKeao
Copy link
Member

A goroutine leak is found in test: https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_check2/detail/ghpr_check2/15046/pipeline

goleak: Errors on successful test run: found unexpected goroutines:

[Goroutine 454293 in state sync.Cond.Wait, 5 minutes, with sync.runtime_notifyListWait on top of the stack:

goroutine 454293 [sync.Cond.Wait, 5 minutes]:

sync.runtime_notifyListWait(0xc0105b5210, 0x0)

	GOROOT/src/runtime/sema.go:527 +0x14c

sync.(*Cond).Wait(0xc002341f70?)

	GOROOT/src/sync/cond.go:70 +0x8c

google.golang.org/grpc/internal/transport.(*http2Client).keepalive(0xc00c758b40)

	external/org_golang_google_grpc/internal/transport/http2_client.go:1693 +0x245

created by google.golang.org/grpc/internal/transport.newHTTP2Client

	external/org_golang_google_grpc/internal/transport/http2_client.go:386 +0x1dcc

]
@YangKeao YangKeao added the type/bug The issue is confirmed as a bug. label Feb 28, 2023
@ti-chi-bot ti-chi-bot added may-affects-4.0 This bug maybe affects 4.0.x versions. may-affects-5.0 This bug maybe affects 5.0.x versions. may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.0 may-affects-6.1 may-affects-6.2 may-affects-6.3 may-affects-6.4 may-affects-6.5 may-affects-6.6 labels Feb 28, 2023
@ChenPeng2013 ChenPeng2013 removed may-affects-4.0 This bug maybe affects 4.0.x versions. may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-5.0 This bug maybe affects 5.0.x versions. may-affects-6.0 may-affects-6.1 may-affects-6.2 may-affects-6.3 may-affects-6.4 may-affects-6.5 may-affects-6.6 labels Feb 28, 2023
@VelocityLight VelocityLight added the affects-7.1 This bug affects the 7.1.x(LTS) versions. label Apr 20, 2023
@jebter jebter added the sig/sql-infra SIG: SQL Infra label Sep 11, 2023
@jackysp
Copy link
Member

jackysp commented Oct 14, 2023

It should also affect v6.5.

@jackysp jackysp added the affects-6.5 This bug affects the 6.5.x(LTS) versions. label Oct 14, 2023
@ti-chi-bot ti-chi-bot added the affects-7.5 This bug affects the 7.5.x(LTS) versions. label Oct 23, 2023
@tiancaiamao
Copy link
Contributor

This seems to be a grpc leak and the recent leak I known is this one #48870
I'm not sure they are related.

Can't reproduce and from the above information provided, I have no idea how to debug this issue.
So close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.0 affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. component/test severity/major sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.
Projects
None yet
Development

No branches or pull requests

7 participants