Skip to content

SimpleSecurityNetty4ServerTransportTests.testTracerLog is failing with appends to non-started mock appender #41890

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.

Sign up for GitHub

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

Jump to bottom
Closed
jasontedor opened this issue May 7, 2019 · 1 comment · Fixed by #42286
Closed

SimpleSecurityNetty4ServerTransportTests.testTracerLog is failing with appends to non-started mock appender #41890

jasontedor opened this issue May 7, 2019 · 1 comment · Fixed by #42286
Assignees
@original-brownbear
Labels
:Distributed Coordination/Network Http and internode communication implementations >test-failure Triaged test failures from CI

Comments

@jasontedor
Copy link
Member

This just failed for SimpleSecurityNetty4ServerTransportTests.testTracerLog:

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+intake/3437/

A slightly different error message this time:

Expected: an empty collection
     but: <[Attempted to append to non-started appender mock]>
	at __randomizedtesting.SeedInfo.seed([BC7BFAA89193A0EF:BFF80716909B0E3A]:0)
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
	at org.junit.Assert.assertThat(Assert.java:956)
	at org.junit.Assert.assertThat(Assert.java:923)
	at org.elasticsearch.test.ESTestCase.checkStaticState(ESTestCase.java:484)
	at org.elasticsearch.test.ESTestCase.after(ESTestCase.java:360)

Reproduce line is:

./gradlew :x-pack:plugin:security:test --tests "org.elasticsearch.xpack.security.transport.netty4.SimpleSecurityNetty4ServerTransportTests.testTracerLog" -Dtests.seed=BC7BFAA89193A0EF -Dtests.security.manager=true -Dtests.locale=ta-MY -Dtests.timezone=Mexico/BajaSur -Dcompiler.java=12 -Druntime.java=11

Relates #40586
@jasontedor jasontedor added :Distributed Coordination/Network Http and internode communication implementations >test-failure Triaged test failures from CI labels May 7, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@jasontedor jasontedor mentioned this issue May 7, 2019
Closed
@original-brownbear original-brownbear self-assigned this May 21, 2019
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this issue May 21, 2019
@original-brownbear
Fix testTracerLog Network Tests
f5b0987 
* Start appender before using it like we do for e.g. the Netty leak detection appender to avoid interference from actions on the network threads that might still be dangling from previous tests in the same suite
* Closes elastic#41890
@original-brownbear original-brownbear mentioned this issue May 21, 2019
Merged
pull bot pushed a commit to scher200/elasticsearch that referenced this issue May 22, 2019
@original-brownbear
Fix testTracerLog Network Tests (elastic#42286)
4a9788e 
* Fix testTracerLog Network Tests

* Start appender before using it like we do for e.g. the Netty leak detection appender to avoid interference from actions on the network threads that might still be dangling from previous tests in the same suite
* Closes elastic#41890
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this issue May 27, 2019
@original-brownbear
Fix testTracerLog Network Tests (elastic#42286)
541c46c 
* Fix testTracerLog Network Tests

* Start appender before using it like we do for e.g. the Netty leak detection appender to avoid interference from actions on the network threads that might still be dangling from previous tests in the same suite
* Closes elastic#41890
@original-brownbear original-brownbear mentioned this issue May 27, 2019
Merged
gurkankaymak pushed a commit to gurkankaymak/elasticsearch that referenced this issue May 27, 2019
@original-brownbear
Fix testTracerLog Network Tests (elastic#42286)
3766585 
* Fix testTracerLog Network Tests

* Start appender before using it like we do for e.g. the Netty leak detection appender to avoid interference from actions on the network threads that might still be dangling from previous tests in the same suite
* Closes elastic#41890
original-brownbear added a commit that referenced this issue May 27, 2019
@original-brownbear
Fix testTracerLog Network Tests (#42286) (#42565)
489616d 
* Fix testTracerLog Network Tests

* Start appender before using it like we do for e.g. the Netty leak detection appender to avoid interference from actions on the network threads that might still be dangling from previous tests in the same suite
* Closes #41890
@gwbrown gwbrown mentioned this issue Dec 18, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@original-brownbear original-brownbear

Labels
:Distributed Coordination/Network Http and internode communication implementations >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix testTracerLog Network Tests original-brownbear/elasticsearch
3 participants
@jasontedor @original-brownbear @elasticmachine