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

Test LRA extension with the latest Narayana LRA coordinator #2285

Closed
jamesnetherton opened this issue Feb 24, 2021 · 4 comments
Closed

Test LRA extension with the latest Narayana LRA coordinator #2285

jamesnetherton opened this issue Feb 24, 2021 · 4 comments
Assignees
Milestone

Comments

@jamesnetherton
Copy link
Contributor

I found some problems when testing with the latest Narayana LRA coordinator, so for now, tests are performed against an outdated version. We should upgrade when they are resolved.

https://issues.apache.org/jira/browse/CAMEL-16253
https://issues.redhat.com/browse/JBTM-3433

jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Feb 24, 2021
@ppalaga
Copy link
Contributor

ppalaga commented May 26, 2021

CAMEL-16253 is resolved, but JBTM-3433 is not. Should we perhaps ping them again?
Is this something for Camel Quarkus 2.0?

@jamesnetherton
Copy link
Contributor Author

Is this something for Camel Quarkus 2.0?

Not strictly because we can bump the version of the LRA container image used in the itests at any time. But it'd be nice to get a fix ASAP.

I dropped a comment into the JBTM issue to ask for an update. There is still the (not ideal) workaround of using the older LRA coordinator.

I could also look into tweaking camel-lra to see if it's possible to at least work around the problem.

@zhfeng
Copy link
Contributor

zhfeng commented Jun 21, 2021

I think the LRA got the Final 1.0 release recently and it needs to check with it. I'd like to take a look.

zhfeng added a commit to zhfeng/camel-quarkus that referenced this issue Jun 25, 2021
@zhfeng
Copy link
Contributor

zhfeng commented Jun 25, 2021

The lastest lra-coordinator 5.12.0.Final image looks good and it was built by using Quarkus. We also need to take a look when the quarkus lra extension gets merged in upstream.

@jamesnetherton jamesnetherton added this to the 2.0.0 milestone Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants