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

[WFLY-2501] Improved CLI message where HTTP Upgrade not supported. #5754

Closed
wants to merge 1 commit into from

Conversation

darranl
Copy link
Contributor

@darranl darranl commented Jan 23, 2014

If a server is sending a redirect it did not understand the HTTP Upgrade request so convert to an appropriate error message.

…he HTTP Upgrade request so convert to an appropriate error message.
@wildfly-ci
Copy link

Build 2311 is now running using a merge of dfa6898

@wildfly-ci
Copy link

Build 2311 outcome was FAILURE using a merge of dfa6898
Summary: Tests failed: 1 (1 new), passed: 5538, ignored: 73 Build time: 1:30:14

Build problems:

Failed tests detected

java.lang.AssertionError: expected:<0> but was:<2>
java.lang.AssertionError: expected:<0> but was:<2>
    at org.junit.Assert.fail(Assert.java:88)
    at org.junit.Assert.failNotEquals(Assert.java:743)
    at org.junit.Assert.assertEquals(Assert.java:118)
    at org.junit.Assert.assertEquals(Assert.java:555)
    at org.junit.Assert.assertEquals(Assert.java:542)
    at org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulTimeoutTestCase.timeout(StatefulTimeoutTestCase.java:119)

Failed tests

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulTimeoutTestCase(SYNC-tcp).timeout: java.lang.AssertionError: expected:<0> but was:<2>
    at org.junit.Assert.fail(Assert.java:88)
    at org.junit.Assert.failNotEquals(Assert.java:743)

@darranl
Copy link
Contributor Author

darranl commented Jan 23, 2014

Retest this please

Feature request for the pull process - If the word 'clustering' appears in the failure message have one automatic retest this please ;-)

@wildfly-ci
Copy link

Build 2321 is now running using a merge of dfa6898

@wildfly-ci
Copy link

Build 2321 outcome was SUCCESS using a merge of dfa6898
Summary: Tests passed: 5539, ignored: 73 Build time: 1:34:03

@bstansberry
Copy link
Contributor

Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants