-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Auditlog to syslog tests #5237
Auditlog to syslog tests #5237
Conversation
Build 993 is now running using a merge of 05516752ea282346b7101ee0ce1e5b832dff6391 |
Build 993 outcome was FAILURE using a merge of 05516752ea282346b7101ee0ce1e5b832dff6391 Build problems:Failed tests detected
Failed tests
|
The logging parts of this look okay to me. I still have the same rant/opinion on using a time based solution to check for log messages as I had in the other PR :) #5084. |
Build 1001 is now running using a merge of 5036dda9fded23828525646e7e4924d7cc49f598 |
Build 1001 outcome was FAILURE using a merge of 5036dda9fded23828525646e7e4924d7cc49f598 Build problems:Failed tests detected
Failed tests
|
Could you retest this please? This seems like a Bouncy-castle related issue:
@ctomc Is the BC provider registered in the JDKs used by TeamCity agents? |
Build 1005 is now running using a merge of 5036dda9fded23828525646e7e4924d7cc49f598 |
Build 1005 outcome was FAILURE using a merge of 5036dda9fded23828525646e7e4924d7cc49f598 Build problems:Failed tests detected
Failed tests
|
retest this please |
Build 1011 is now running using a merge of 0682b2d |
Build 1011 outcome was SUCCESS using a merge of 0682b2d |
@kabir you should be the reviewer for this one. :) |
@bstansberry @kwart |
Merged. |
@kwart just to answer your question, no modification of jdk is done on any of build agents/slaves. |
This PR contains fixes for existing syslog-handler tests and also set of new tests to cover TCP and TLS protocol support.
TLS test fails, because of https://issues.jboss.org/browse/WFLY-2255, but it'll be OK, after merging the fix #5230