On Thu, Oct 1, 2020 at 11:24 PM Dmitry Shachnev <mity...@debian.org> wrote:
>
> Source: golang-github-lucas-clemente-quic-go
> Version: 0.18.0-2
> Severity: important
> Tags: ftbfs
>
> Dear Maintainer,
>
> This package sometimes fails to build from source. The relevant part of the
> log is:
>
> • Failure [0.435 seconds]
> Server
> /build/golang-github-lucas-clemente-quic-go-0.18.0/obj-x86_64-linux-gnu/src/github.com/lucas-clemente/quic-go/server_test.go:40
> server accepting sessions that completed the handshake
>
> /build/golang-github-lucas-clemente-quic-go-0.18.0/obj-x86_64-linux-gnu/src/github.com/lucas-clemente/quic-go/server_test.go:185
> handling packets
>
> /build/golang-github-lucas-clemente-quic-go-0.18.0/obj-x86_64-linux-gnu/src/github.com/lucas-clemente/quic-go/server_test.go:206
> drops packets if the receive queue is full [It]
>
> /build/golang-github-lucas-clemente-quic-go-0.18.0/obj-x86_64-linux-gnu/src/github.com/lucas-clemente/quic-go/server_test.go:625
>
> Timed out after 0.118s.
> Expected
> <uint32>: 727
> to be equivalent to
> <int>: 1025
>
> The full build log is attached. I cannot reproduce this reliably, thus it's
> Severity: important and not serious.
>
> It also fails on Reproducible Builds infrastructure:
>
> https://tests.reproducible-builds.org/debian/rb-pkg/unstable/amd64/golang-github-lucas-clemente-quic-go.html
>
> And in Ubuntu:
>
> https://launchpad.net/ubuntu/+source/golang-github-lucas-clemente-quic-go/0.18.0-2/+build/20091909
>
I can't reproduce this.
Could you apply the following patch to see if it works for Ubuntu?
diff --git a/server_test.go b/server_test.go
index c22136f0..05f568ff 100644
--- a/server_test.go
+++ b/server_test.go
@@ -677,7 +677,7 @@ var _ = Describe("Server", func() {
close(acceptSession)
Eventually(
func() uint32 { return
atomic.LoadUint32(&counter) },
- scaleDuration(100*time.Millisecond),
+ scaleDuration(1000*time.Millisecond),
).Should(BeEquivalentTo(protocol.MaxServerUnprocessedPackets + 1))
Consistently(func() uint32 { return
atomic.LoadUint32(&counter)
}).Should(BeEquivalentTo(protocol.MaxServerUnprocessedPackets + 1))
})
--
Shengjing Zhu
