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.

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

WFLY-2885 HttpListenerService start doesn't clean up if start fails #7679

Merged
merged 1 commit into from
Jul 2, 2015
Merged

WFLY-2885 HttpListenerService start doesn't clean up if start fails #7679

merged 1 commit into from
Jul 2, 2015

Conversation

ctomc
Copy link
Contributor

@ctomc ctomc commented Jun 30, 2015

No description provided.

@wildfly-ci
Copy link

Linux with security manager Build 1735 is now running using a merge of 991c0a2

@wildfly-ci
Copy link

Windows Build 1913 is now running using a merge of 991c0a2

@wildfly-ci
Copy link

Linux Build 6838 is now running using a merge of 991c0a2

@wildfly-ci
Copy link

Linux with security manager Build 1735 outcome was SUCCESS using a merge of 991c0a2
Summary: Tests passed: 855, ignored: 321 Build time: 0:06:26

@wildfly-ci
Copy link

Windows Build 1913 outcome was SUCCESS using a merge of 991c0a2
Summary: Tests passed: 2889, ignored: 377 Build time: 0:46:56

@wildfly-ci
Copy link

Linux Build 6838 outcome was FAILURE using a merge of 991c0a2
Summary: Tests failed: 2 (2 new), passed: 2886, ignored: 377, muted: 1 Build time: 0:49:58

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.ejb.remote.suspend.EjbRemoteSuspendTestCase.testSuspendedCallRejected: <no details avaliable>

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

bstansberry
bstansberry reviewed Jun 30, 2015
View reviewed changes
undertow/src/main/java/org/wildfly/extension/undertow/ListenerService.java
throw new StartException("Could not start http listener", e);
}
}

public abstract void cleanFailedStart();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

protected?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that would be better for sure.

@wildfly-ci
Copy link

Windows Build 1925 is now running using a merge of da4ce64

@wildfly-ci
Copy link

Linux with security manager Build 1747 is now running using a merge of da4ce64

@wildfly-ci
Copy link

Linux Build 6851 is now running using a merge of da4ce64

@wildfly-ci
Copy link

Linux with security manager Build 1747 outcome was SUCCESS using a merge of da4ce64
Summary: Tests passed: 855, ignored: 321 Build time: 0:06:08

@wildfly-ci
Copy link

Windows Build 1925 outcome was SUCCESS using a merge of da4ce64
Summary: Tests passed: 2889, ignored: 377 Build time: 0:46:48

@wildfly-ci
Copy link

Linux Build 6851 outcome was FAILURE using a merge of da4ce64
Summary: Tests failed: 1 (1 new), passed: 2887, ignored: 377, muted: 1 Build time: 0:48:28

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testConcurrentFailover: <no details avaliable>

@bstansberry bstansberry added the ready-for-merge Only for use by those with merge permissions! label Jul 1, 2015
bstansberry added a commit that referenced this pull request Jul 2, 2015
@bstansberry
Merge pull request #7679 from ctomc/undertow-failed-start
19156d1 
WFLY-2885 HttpListenerService start doesn't clean up if start fails
@bstansberry bstansberry merged commit 19156d1 into wildfly:master Jul 2, 2015
@ctomc ctomc deleted the undertow-failed-start branch November 6, 2017 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ready-for-merge Only for use by those with merge permissions!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ctomc @wildfly-ci @bstansberry