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-3298 LRA compenstators should compensate in opposite order from … #2301

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mmusgrov
Copy link
Member

@mmusgrov mmusgrov commented Sep 26, 2024

https://issues.redhat.com/browse/JBTM-3298

As a test and a fix for calling LRA participants in the opposite order from which they enlisted.

!CORE !AS_TESTS !RTS !JACOCO !XTS !QA_JTA !QA_JTS_OPENJDKORB !PERFORMANCE LRA !DB_TESTS !mysql !db2 !postgres oracle !JDK11

@jbosstm-bot
Copy link

LRA profile tests failed (https://ci-jenkins-csb-narayana.apps.ocp-c1.prod.psi.redhat.com/job/btny-pulls-narayana/PROFILE=LRA,jdk=openJDK11,label=jnlp-agent/513/): Narayana rebase on main failed. Please rebase it manually

@jbosstm-bot
Copy link

LRA profile tests failed (https://ci-jenkins-csb-narayana.apps.ocp-c1.prod.psi.redhat.com/job/btny-pulls-narayana/PROFILE=LRA,jdk=openJDK17,label=jnlp-agent/513/): Narayana rebase on main failed. Please rebase it manually

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@marcosgopen
Copy link
Member

Thanks Mike for your PR, could you have a look at the LRA failed job? It looks like the CoordinatorApiIT failed because 'Invalid link URI (http://complete.url:8080/; rel="complete"): missing compensator or after LRA callback'

@mmusgrov
Copy link
Member Author

The failure was in the FailedLRAIT testsuite on just the JDK17 run which I'm still investigating . We always get Invalid link URI ...: we have some tests, not sure which ones though, that provide invalid information and then verifies that the coordinator refuses the request, unfortunately they generate deep stack traces as part of the successful route through the code which make it harder to parse the output for genuine failures.

@marcosgopen
Copy link
Member

The failure was in the FailedLRAIT testsuite on just the JDK17 run which I'm still investigating . We always get Invalid link URI ...: we have some tests, not sure which ones though, that provide invalid information and then verifies that the coordinator refuses the request, unfortunately they generate deep stack traces as part of the successful route through the code which make it harder to parse the output for genuine failures.

Sorry, you are right. The failing test is 'FailedLRAIT' as you said. I'll have a look.

@mmusgrov
Copy link
Member Author

mmusgrov commented Sep 27, 2024

There's no evidence in the logs so I am going to push an update that will include some debug of the failing method (FailedLRAIT#validateStateAndRemove), but I can say it looks unrelated to the changes made by the PR. We can always back it out after the PR re-runs.

@jbosstm-bot
Copy link

LRA profile tests failed (https://ci-jenkins-csb-narayana.apps.ocp-c1.prod.psi.redhat.com/job/btny-pulls-narayana/PROFILE=LRA,jdk=openJDK17,label=jnlp-agent/526/): Narayana rebase on main failed. Please rebase it manually

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

LRA profile tests failed (https://ci-jenkins-csb-narayana.apps.ocp-c1.prod.psi.redhat.com/job/btny-pulls-narayana/PROFILE=LRA,jdk=openJDK17,label=jnlp-agent/529/): Narayana rebase on main failed. Please rebase it manually

@mmusgrov
Copy link
Member Author

@marcosgopen I've put it on Hold while I figure out why the changes have effected the FailedLRAIT tests.

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@mmusgrov
Copy link
Member Author

@marcosgopen Please bear with me while I investigate (it only seems to fail on FailedLRAIT on CI and on OpenJDK 17 so can't debug on my laptop).

I have pushed a debug commit and an experimental commit to give me more information, ie please don't review it further until I've figured out what the problem is.

@jbosstm-bot
Copy link

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

Successfully merging this pull request may close these issues.

3 participants