Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

disable TestQueueDepth due to flakiness #188

Merged
merged 1 commit into from
May 3, 2017

Conversation

kirg
Copy link
Contributor

@kirg kirg commented May 3, 2017

--- FAIL: TestQueueDepth (103.41s)
panic: Fail in goroutine after TestNetIntegrationSuiteParallelA has completed [recovered]
	panic: Fail in goroutine after TestNetIntegrationSuiteParallelA has completed
goroutine 25346 [running]:
panic(0x12c9b40, 0xc423576a80)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/runtime/panic.go:500 +0x1ae
testing.tRunner.func1(0xc421209440)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/testing/testing.go:579 +0x474
panic(0x12c9b40, 0xc423576a80)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/runtime/panic.go:458 +0x271
testing.(*common).Fail(0xc42017e900)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/testing/testing.go:412 +0x182
testing.(*common).Errorf(0xc42017e900, 0xc423957ee0, 0xcf, 0x0, 0x0, 0x0)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/testing/testing.go:484 +0x95
testing.(*T).Errorf(0xc42017e900, 0xc423957ee0, 0xcf, 0x0, 0x0, 0x0)
	<autogenerated>:10 +0x87
github.com/uber/cherami-server/vendor/github.com/stretchr/testify/assert.Fail(0x1cef8e0, 0xc42017e900, 0xc42442bc70, 0x41, 0x0, 0x0, 0x0, 0xc42358f1c0)
	/home/travis/gopath/src/github.com/uber/cherami-server/vendor/github.com/stretchr/testify/assert/assertions.go:226 +0x280
github.com/uber/cherami-server/vendor/github.com/stretchr/testify/assert.NoError(0x1cef8e0, 0xc42017e900, 0x1cf0520, 0xc420074ea0, 0x0, 0x0, 0x0, 0xc420f7b801)
	/home/travis/gopath/src/github.com/uber/cherami-server/vendor/github.com/stretchr/testify/assert/assertions.go:891 +0x12c
github.com/uber/cherami-server/vendor/github.com/stretchr/testify/require.NoError(0x1cf69e0, 0xc42017e900, 0x1cf0520, 0xc420074ea0, 0x0, 0x0, 0x0)
	/home/travis/gopath/src/github.com/uber/cherami-server/vendor/github.com/stretchr/testify/require/require.go:287 +0xa0
github.com/uber/cherami-server/vendor/github.com/stretchr/testify/require.(*Assertions).NoError(0xc4201a69a0, 0x1cf0520, 0xc420074ea0, 0x0, 0x0, 0x0)
	/home/travis/gopath/src/github.com/uber/cherami-server/vendor/github.com/stretchr/testify/require/require_forward.go:237 +0x8c
github.com/uber/cherami-server/test/integration.(*NetIntegrationSuiteParallelB).TestQueueDepth.func6(0x2, 0x0)
	/home/travis/gopath/src/github.com/uber/cherami-server/test/integration/integration_test.go:2417 +0x882
github.com/uber/cherami-server/test/integration.(*NetIntegrationSuiteParallelB).TestQueueDepth(0xc4200a1080)
	/home/travis/gopath/src/github.com/uber/cherami-server/test/integration/integration_test.go:2677 +0x2099
reflect.Value.call(0xc422ed7aa0, 0xc420033618, 0x13, 0x14429cf, 0x4, 0xc42358ff18, 0x1, 0x1, 0xc4200a1080, 0xc420630e30, ...)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/reflect/value.go:434 +0x6d6
reflect.Value.Call(0xc422ed7aa0, 0xc420033618, 0x13, 0xc42358ff18, 0x1, 0x1, 0x123f7df, 0xe, 0x0)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/reflect/value.go:302 +0xc1
github.com/uber/cherami-server/vendor/github.com/stretchr/testify/suite.Run.func2(0xc421209440)
	/home/travis/gopath/src/github.com/uber/cherami-server/vendor/github.com/stretchr/testify/suite/suite.go:101 +0x38a
testing.tRunner(0xc421209440, 0xc421ddbc80)
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/testing/testing.go:610 +0xca
created by testing.(*T).Run
	/home/travis/.gimme/versions/go1.7.linux.amd64/src/testing/testing.go:646 +0x530
exit status 2
FAIL	github.com/uber/cherami-server/test/integration	260.724s

@kirg kirg requested a review from GuillaumeBailey May 3, 2017 19:30
@kirg kirg self-assigned this May 3, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.7%) to 67.933% when pulling a3bed6e on disable-TestQueueDepth into 52b828e on master.

@GuillaumeBailey GuillaumeBailey merged commit d13f171 into master May 3, 2017
@kirg kirg deleted the disable-TestQueueDepth branch May 3, 2017 20:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants