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

Intermittent test failure #136

Open
peterbroadhurst opened this issue Apr 30, 2024 · 0 comments
Open

Intermittent test failure #136

peterbroadhurst opened this issue Apr 30, 2024 · 0 comments
Assignees

Comments

@peterbroadhurst
Copy link
Contributor

peterbroadhurst commented Apr 30, 2024

This is affecting build stability in the repo:

Tooks like buildTLSListener is being allowed to run on a routine and not being waited to complete before the end of a test.

?   	github.com/hyperledger/firefly-common/pkg/i18n/es	[no test files]
time="2024-03-13T23:29:54Z" level=error msg="Failed TLS DN check: Does not match CN =~ /^right\\.example\\.com$/"
time="2024-03-13T23:29:54Z" level=error msg="Failed TLS DN check: Nil cert chain"
time="2024-03-13T23:29:54Z" level=error msg="Failed TLS DN check: Empty cert chain 0"
PASS
panic: Log in goroutine after TestConnectSkipVerification has completed: read failed: EOF

goroutine 41 [running]:
testing.(*common).logDepth(0xc000244340, {0xc000014470, 0x10}, 0x3)
	/opt/hostedtoolcache/go/1.21.8/x64/src/testing/testing.go:1022 +0x4c5
testing.(*common).log(...)
	/opt/hostedtoolcache/go/1.21.8/x64/src/testing/testing.go:1004
testing.(*common).Logf(0xc000244340, {0x79eba9?, 0x1?}, {0xc000509fc0?, 0x0?, 0x538440?})
	/opt/hostedtoolcache/go/1.21.8/x64/src/testing/testing.go:1055 +0x54
github.com/hyperledger/firefly-common/pkg/fftls.buildTLSListener.func1()
	/home/runner/work/firefly-common/firefly-common/pkg/fftls/fftls_test.go:115 +0x5e
created by github.com/hyperledger/firefly-common/pkg/fftls.buildTLSListener in goroutine 40
	/home/runner/work/firefly-common/firefly-common/pkg/fftls/fftls_test.go:103 +0x111
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants