Skip to content

test: flaky TestLookupResources#3171

Open
miparnisari wants to merge 1 commit into
mainfrom
test-TestLookupResources
Open

test: flaky TestLookupResources#3171
miparnisari wants to merge 1 commit into
mainfrom
test-TestLookupResources

Conversation

@miparnisari

Copy link
Copy Markdown
Contributor
--- FAIL: TestLookupResources (4.73s)
    permissions_test.go:740: found unexpected goroutines:
        [Goroutine 8432 in state IO wait, with internal/poll.runtime_pollWait on top of the stack:
        internal/poll.runtime_pollWait(0x10d4a6400, 0x72)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/runtime/netpoll.go:351 +0xa0
        internal/poll.(*pollDesc).wait(0xc070df78a0, 0x72, 0x0)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/internal/poll/fd_poll_runtime.go:84 +0xb8
        internal/poll.(*pollDesc).waitRead(...)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/internal/poll/fd_poll_runtime.go:89
        internal/poll.(*FD).Read(0xc070df7880, {0xc0767aa000, 0x1500, 0x1500})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/internal/poll/fd_unix.go:165 +0x34c
        net.(*netFD).Read(0xc070df7880, {0xc0767aa000, 0x1500, 0x1500})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/fd_posix.go:68 +0x48
        net.(*conn).Read(0xc05fb2f8a8, {0xc0767aa000, 0x1500, 0x1500})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/net.go:196 +0x88
        crypto/tls.(*atLeastReader).Read(0xc06c756240, {0xc0767aa000, 0x1500, 0x1500})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/crypto/tls/conn.go:815 +0x78
        bytes.(*Buffer).ReadFrom(0xc05fe2ad28, {0x109fefbc0, 0xc06c756240})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/bytes/buffer.go:229 +0xf0
        crypto/tls.(*Conn).readFromUntil(0xc05fe2aa88, {0x163955538, 0xc05fb2f8a8}, 0x5)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/crypto/tls/conn.go:837 +0x184
        crypto/tls.(*Conn).readRecordOrCCS(0xc05fe2aa88, 0x0)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/crypto/tls/conn.go:626 +0x284
        crypto/tls.(*Conn).readRecord(...)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/crypto/tls/conn.go:588
        crypto/tls.(*Conn).Read(0xc05fe2aa88, {0xc07a0d9000, 0x1000, 0x104f2a460?})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/crypto/tls/conn.go:1393 +0x1dc
        bufio.(*Reader).Read(0xc0723a6000, {0xc079b7c3c4, 0x9, 0x9})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/bufio/bufio.go:245 +0x374
        io.ReadAtLeast({0x109fe6e40, 0xc0723a6000}, {0xc079b7c3c4, 0x9, 0x9}, 0x9)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/io/io.go:335 +0xc0
        io.ReadFull(...)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/io/io.go:354
        net/http.http2readFrameHeader({0xc079b7c3c4, 0x9, 0x9}, {0x109fe6e40, 0xc0723a6000})
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/http/h2_bundle.go:1805 +0x64
        net/http.(*http2Framer).ReadFrameHeader(0xc079b7c380)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/http/h2_bundle.go:2071 +0x80
        net/http.(*http2Framer).ReadFrame(0xc079b7c380)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/http/h2_bundle.go:2130 +0x2c
        net/http.(*http2clientConnReadLoop).run(0xc00194bf90)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/http/h2_bundle.go:9550 +0x104
        net/http.(*http2ClientConn).readLoop(0xc011312540)
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/http/h2_bundle.go:9419 +0x88
        created by net/http.(*http2Transport).newClientConn in goroutine 8431
                /Users/miparnisari/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.26.4.darwin-arm64/src/net/http/h2_bundle.go:8171 +0x1524
        ]
FAIL

@github-actions github-actions Bot added area/api v1 Affects the v1 API area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) labels Jun 11, 2026
@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@miparnisari miparnisari force-pushed the test-TestLookupResources branch from cbf0f56 to e5263e0 Compare June 11, 2026 01:18
@miparnisari miparnisari marked this pull request as ready for review June 11, 2026 01:20
@miparnisari miparnisari requested a review from a team as a code owner June 11, 2026 01:20

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 917382d Previous: 8908ebe Ratio
BenchmarkDatastoreDriver/cockroachdb-overlap-static/TestTuple/SnapshotReverseRead (github.com/authzed/spicedb/internal/datastore/benchmark) 34815182 ns/op 174683 B/op 20199 allocs/op 7838746 ns/op 172932 B/op 20196 allocs/op 4.44
BenchmarkDatastoreDriver/cockroachdb-overlap-static/TestTuple/SnapshotReverseRead (github.com/authzed/spicedb/internal/datastore/benchmark) - ns/op 34815182 ns/op 7838746 ns/op 4.44
BenchmarkDatastoreDriver/cockroachdb-overlap-insecure/TestTuple/SnapshotReverseRead (github.com/authzed/spicedb/internal/datastore/benchmark) 32837136 ns/op 173756 B/op 20199 allocs/op 8164476 ns/op 172823 B/op 20195 allocs/op 4.02
BenchmarkDatastoreDriver/cockroachdb-overlap-insecure/TestTuple/SnapshotReverseRead (github.com/authzed/spicedb/internal/datastore/benchmark) - ns/op 32837136 ns/op 8164476 ns/op 4.02

This comment was automatically generated by workflow using github-action-benchmark.

@miparnisari miparnisari force-pushed the test-TestLookupResources branch from e5263e0 to 917382d Compare June 11, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api v1 Affects the v1 API area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) Skip-Changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant