WFLY-3715 Async servlets cause lock timeouts for distributable sessions #6580
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Build 4437 is now running using a merge of 85fbe31 |
|
Build 4438 is now running using a merge of 7083345 |
|
Build 4437 outcome was FAILURE using a merge of 7083345 Build problems:Failed tests detected Failed tests |
|
Build 4438 outcome was FAILURE using a merge of 7083345 Build problems:Failed tests detected Failed tests |
|
Build 4440 is now running using a merge of 589c88d |
|
That test passes for me locally - which is strange, because it's a pretty simple test. |
|
Build 4440 outcome was FAILURE using a merge of 589c88d Build problems:Failed tests detected Failed tests |
|
That should do it... |
|
Build 4442 is now running using a merge of ec87d37 |
|
Build 4442 outcome was FAILURE using a merge of ec87d37 Build problems:Failed tests detected Failed tests |
|
Build 4445 is now running using a merge of 5790b96 |
|
Build 4445 outcome was FAILURE using a merge of 5790b96 Build problems:Failed tests detected Failed tests |
|
Build 4448 is now running using a merge of 8e04fac |
|
Build 4449 is now running using a merge of bb529b7 |
|
Build 4448 outcome was FAILURE using a merge of bb529b7 Build problems:Failed tests detected Failed tests |
|
Build 4450 is now running using a merge of 241c492 |
|
Build 4449 outcome was FAILURE using a merge of 241c492 Build problems:Failed tests detected Failed tests |
|
Build 4450 outcome was FAILURE using a merge of 241c492 Build problems:Failed tests detected Failed tests |
|
Build 4453 is now running using a merge of a947eec |
|
Build 4453 outcome was FAILURE using a merge of a947eec Build problems:Failed tests detected Failed tests |
|
Build 4455 is now running using a merge of da9c343 |
|
Build 4455 outcome was FAILURE using a merge of da9c343 Build problems:Process exited with code 1 |
|
Build 4456 is now running using a merge of ac4e4e8 |
|
Once this passes once, to prove testsuite stability can you launch about 10 customized runs of http://brontes.lab.eng.brq.redhat.com/viewType.html?buildTypeId=WFPR configured to use your repo and branch instead of wildfly/master? They can all run at the same time. Changing -DallTests to -Dts.clustering will make them go faster. Ping me if you don't know how to do that. |
|
OK. |
|
Build 4457 is now running using a merge of cce1825 |
|
Build 4456 outcome was FAILURE using a merge of cce1825 Build problems:Failed tests detected Failed tests |
|
This is a bit of a stumper... ReplicationForNegotiationAuthenticatorTestCase isn't doing much that most of the other web tests aren't already doing. I also cannot reproduce this failure locally - it only seems to fail in the CI runs. I'm going to setup a private job to try to debug this - so excuse the noise on this PR thread... |
|
Build 4460 is now running using a merge of cce1825 |
|
Build 4460 outcome was FAILURE using a merge of cce1825 Build problems:Failed tests detected Failed tests |
|
Build 4497 outcome was SUCCESS using a merge of 99d18ff |
|
@bstansberry OK - this seems to passing consistently now. Ready for review. |
|
Can someone review this? |
|
Can someone review this soon? |
|
Resolving merge conflicts... |
|
Build 4583 is now running using a merge of e463c67 |
|
Build 4583 outcome was FAILURE using a merge of e463c67 Build problems:Failed tests detected Failed tests |
|
This above failure is a regression introduced by: #6619 |
|
I have muted this test on CI for now and assigned it to @stuartwdouglas |
|
retest this please |
|
Build 4596 is now running using a merge of e463c67 |
|
Build 4596 outcome was SUCCESS using a merge of e463c67 |
|
Linux Build 4607 is now running using a merge of b20976d |
|
Linux Build 4607 outcome was SUCCESS using a merge of b20976d |
|
Linux Build 4629 is now running using a merge of e1fb097 |
|
Linux Build 4629 outcome was SUCCESS using a merge of e1fb097 |
bstansberry
added a commit
that referenced
this pull request
Aug 28, 2014
WFLY-3715 Async servlets cause lock timeouts for distributable sessions
|
Woohoo! Thanks @bstansberry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://issues.jboss.org/browse/WFLY-3715
Adds 2 new modules: org.wildfly.clustering.ee.spi and org.wildfly.clustering.ee.infinispan
These modules contain logic common to org.wildfly.clustering.ejb.spi and org.wildfly.clustering.web.spi; and org.wildfly.clustering.ejb.infinispan and org.wildfly.clustering.web.infinispan, respectively.