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

JBTM-2853 JBTM-2858 Disable failing tests TODO revert #1142

Merged
merged 1 commit into from Mar 2, 2017

Conversation

tomjenkinson
Copy link
Contributor

@tomjenkinson tomjenkinson commented Mar 1, 2017

https://issues.jboss.org/browse/JBTM-2853
https://issues.jboss.org/browse/JBTM-2858

!MAIN !QA_JTA !QA_JTS_JDKORB !BLACKTIE !PERF NO_WIN

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@tomjenkinson
Copy link
Contributor Author

Hi Ondra - I will merge this temporarily once you can review and add a git revert into your 1138 PR

Copy link
Contributor

@ochaloup ochaloup left a comment

Choose a reason for hiding this comment

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

Tom, will be those exclusion enough? I think the follow-up tests from rts and xts modules fail too. E.g. outbound bridge will most probably fail too. And for rts there is inbound bridge test which I think will not pass as well.

If you want I can check which are those failing tests. Or the easiest would be exclude all testcases from those modules for now.

@tomjenkinson
Copy link
Contributor Author

well, the PR passes with just those exclusions so I think it should be fine?

Copy link
Contributor

@ochaloup ochaloup left a comment

Choose a reason for hiding this comment

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

Alright, I missed that fact. Sorry ;)

But it's interesting as the set of failing tests is different from what I saw last week. I need to check it out when I will be back at JBTM-2858., Thanks!

@tomjenkinson
Copy link
Contributor Author

I wonder if there have been fixes upstream in WFLY? Thanks for the review!

@tomjenkinson tomjenkinson merged commit 627dc49 into jbosstm:master Mar 2, 2017
@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/428/): Narayana rebase on master failed. Please rebase it manually

@jbosstm-bot
Copy link

BLACKTIE profile tests failed on Linux (https://ci-master-jenkins-csb-narayana.cloud.paas.psi.redhat.com/job/btny-pulls-narayana/PROFILE=BLACKTIE,jdk=jdk8.latest,label=swarm/428/): 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
Labels
None yet
Projects
None yet
3 participants