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

JBTM-2628 Use log4j for qa testsuite logging #984

Closed
wants to merge 1 commit into from
Closed

JBTM-2628 Use log4j for qa testsuite logging #984

wants to merge 1 commit into from

Conversation

mmusgrov
Copy link
Contributor

https://issues.jboss.org/browse/JBTM-2628

!MAIN !XTS !QA_JTS_JACORB !QA_JTS_JDKORB !PERF !BLACKTIE QA_TRACE=1 QA_TESTGROUP=jtatests01 QA_TESTMETHODS=JTATests01_Test006

@tomjenkinson
Copy link
Contributor

If CI passes looks good to me, I will merge it once I have released 5.2.14.Final

@jbosstm-bot
Copy link

Started testing this pull request with QA_JTA profile: http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana-jdk8/PROFILE=QA_JTA,jdk=jdk8.latest,label=linux/1428/

@jbosstm-bot
Copy link

QA_JTA profile tests passed - Job complete http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana-jdk8/PROFILE=QA_JTA,jdk=jdk8.latest,label=linux/1428/

@mmusgrov
Copy link
Contributor Author

I have put this PR on hold whilst I investigate why the thread name still doesn't appear in the log output.

@mmusgrov mmusgrov removed the Hold label Feb 25, 2016
@jbosstm-bot
Copy link

Started testing this pull request with QA_JTA profile: http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana-jdk8/PROFILE=QA_JTA,jdk=jdk8.latest,label=linux/1432/

@jbosstm-bot
Copy link

QA_JTA profile tests passed - Job complete http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana-jdk8/PROFILE=QA_JTA,jdk=jdk8.latest,label=linux/1432/

@jbosstm-bot
Copy link

Started testing this pull request with QA_JTA profile: http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana-jdk8/PROFILE=QA_JTA,jdk=jdk8.latest,label=linux/1433/

@jbosstm-bot
Copy link

QA_JTA profile tests passed - Job complete http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana-jdk8/PROFILE=QA_JTA,jdk=jdk8.latest,label=linux/1433/

@mmusgrov mmusgrov closed this Feb 25, 2016
@mmusgrov mmusgrov deleted the JBTM-2628 branch February 25, 2016 16:05
@mmusgrov
Copy link
Contributor Author

@tomjenkinson I have just merged this PR (I forgot to refresh this page from yesterday and thought you had moved it back to ready for merge again). The only change I made since you reviewed it last night was to change the logging format string to ensure the thread name appears correctly.

If you see any issues with the format string then I can reopen the jira (which I am about to close).

@jbosstm-bot
Copy link

MAIN profile tests failed (https://ci-master-jenkins-csb-narayana.cloud.paas.psi.redhat.com/job/btny-pulls-narayana/PROFILE=MAIN,jdk=jdk8.latest,label=swarm/349/): Narayana rebase on master failed. Please rebase it manually

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mmusgrov mmusgrov

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@mmusgrov @tomjenkinson @jbosstm-bot