Skip to content

Commit

Permalink
Merge #102258
Browse files Browse the repository at this point in the history
102258: leaktest: ignore pgconn async close worker r=lidorcarmel a=lidorcarmel

Unfortunately I cannot repro this locally :(

Epic: none

Fixes: #97698
Fixes: #97346

Release note: None

Co-authored-by: Lidor Carmel <[email protected]>
  • Loading branch information
craig[bot] and lidorcarmel committed Apr 25, 2023
2 parents a7bf183 + 8c2ae78 commit 1927969
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions pkg/util/leaktest/leaktest.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,8 @@ func interestingGoroutines() map[int64]string {
strings.Contains(stack, "sentry-go.(*HTTPTransport).worker") ||
// Ignore the opensensus worker, which is created by the event exporter.
strings.Contains(stack, "go.opencensus.io/stats/view.(*worker).start") ||
// Ignore pgconn which creates a goroutine to do an async cleanup.
strings.Contains(stack, "github.com/jackc/pgconn.(*PgConn).asyncClose.func1") ||
// Seems to be gccgo specific.
(runtime.Compiler == "gccgo" && strings.Contains(stack, "testing.T.Parallel")) ||
// Ignore intentionally long-running logging goroutines that live for the
Expand Down

0 comments on commit 1927969

Please sign in to comment.