[JBoss JIRA] (JBTM-3293) LRA coordinator jar cannot be deployed on WildFly
by Ondrej Chaloupka (Jira)
[ https://issues.redhat.com/browse/JBTM-3293?page=com.atlassian.jira.plugin... ]
Ondrej Chaloupka commented on JBTM-3293:
----------------------------------------
[~mstefank] ok, I see. Would then make sense to consider of not publishing the {{jar}} archive type to Maven nexus but only the {{war}} archive. And using the jar only as a deliverable for the internal projects?
Or are not the thorntail and quarkus capable to work with {{war}} in the same way as with {{jar}}?
(I assume here that the jar is now consumed only for the Narayana purposes. When consumed e.g. by OpenLiberty it's wrong as the {{war}} should be provided for them.)
> LRA coordinator jar cannot be deployed on WildFly
> -------------------------------------------------
>
> Key: JBTM-3293
> URL: https://issues.redhat.com/browse/JBTM-3293
> Project: JBoss Transaction Manager
> Issue Type: Bug
> Components: LRA
> Affects Versions: 5.10.4.Final
> Reporter: Martin Stefanko
> Assignee: Martin Stefanko
> Priority: Major
> Fix For: 5.next
>
>
> I am not sure whether this should be possible but the lra-coordinator.jar from lra-coordinator-jar module can't be deployed on latest WildFly. The deployment fails with the following error:
> {noformat}
> Caused by: org.jboss.weld.exceptions.DeploymentException: WELD-001408: Unsatisfied dependencies for type LRAService with qualifiers @Default
> at injection point [BackedAnnotatedField] @Inject io.narayana.lra.coordinator.api.RecoveryCoordinator.lraService
> at io.narayana.lra.coordinator.api.RecoveryCoordinator.lraService(RecoveryCoordinator.java:0)
> WELD-001474: Class io.narayana.lra.coordinator.domain.service.LRAService is on the classpath, but was ignored because a class it references was not found: com.arjuna.ats.arjuna.recovery.RecoveryModule from [Module "deployment.lra-coordinator.jar" from Service Module Loader].
> {noformat}
--
This message was sent by Atlassian Jira
(v7.13.8#713008)
4 years, 10 months
[JBoss JIRA] (JBTM-3293) LRA coordinator jar cannot be deployed on WildFly
by Martin Stefanko (Jira)
[ https://issues.redhat.com/browse/JBTM-3293?page=com.atlassian.jira.plugin... ]
Martin Stefanko commented on JBTM-3293:
---------------------------------------
[~ochaloup] yes, it does. Changing the packaging to war fixes this issue. However, I would prefer to add a new module rather than change the packaging of the existing lra-coordinator-jar module as this module is used in quarkus and thorntail deliverables.
> LRA coordinator jar cannot be deployed on WildFly
> -------------------------------------------------
>
> Key: JBTM-3293
> URL: https://issues.redhat.com/browse/JBTM-3293
> Project: JBoss Transaction Manager
> Issue Type: Bug
> Components: LRA
> Affects Versions: 5.10.4.Final
> Reporter: Martin Stefanko
> Assignee: Martin Stefanko
> Priority: Major
> Fix For: 5.next
>
>
> I am not sure whether this should be possible but the lra-coordinator.jar from lra-coordinator-jar module can't be deployed on latest WildFly. The deployment fails with the following error:
> {noformat}
> Caused by: org.jboss.weld.exceptions.DeploymentException: WELD-001408: Unsatisfied dependencies for type LRAService with qualifiers @Default
> at injection point [BackedAnnotatedField] @Inject io.narayana.lra.coordinator.api.RecoveryCoordinator.lraService
> at io.narayana.lra.coordinator.api.RecoveryCoordinator.lraService(RecoveryCoordinator.java:0)
> WELD-001474: Class io.narayana.lra.coordinator.domain.service.LRAService is on the classpath, but was ignored because a class it references was not found: com.arjuna.ats.arjuna.recovery.RecoveryModule from [Module "deployment.lra-coordinator.jar" from Service Module Loader].
> {noformat}
--
This message was sent by Atlassian Jira
(v7.13.8#713008)
4 years, 10 months
[JBoss JIRA] (JBTM-3293) LRA coordinator jar cannot be deployed on WildFly
by Ondrej Chaloupka (Jira)
[ https://issues.redhat.com/browse/JBTM-3293?page=com.atlassian.jira.plugin... ]
Ondrej Chaloupka commented on JBTM-3293:
----------------------------------------
[~mstefank] By me it makes much of sense. As the LRA coordinator provides the JAX-RS HTTP endpoints it should be defined as war archive. Have you tried that the changing from {{jar}} to {{war}} fixes the issue? If it so i would be for to change the archive type to {{war}}.
> LRA coordinator jar cannot be deployed on WildFly
> -------------------------------------------------
>
> Key: JBTM-3293
> URL: https://issues.redhat.com/browse/JBTM-3293
> Project: JBoss Transaction Manager
> Issue Type: Bug
> Components: LRA
> Affects Versions: 5.10.4.Final
> Reporter: Martin Stefanko
> Assignee: Martin Stefanko
> Priority: Major
> Fix For: 5.next
>
>
> I am not sure whether this should be possible but the lra-coordinator.jar from lra-coordinator-jar module can't be deployed on latest WildFly. The deployment fails with the following error:
> {noformat}
> Caused by: org.jboss.weld.exceptions.DeploymentException: WELD-001408: Unsatisfied dependencies for type LRAService with qualifiers @Default
> at injection point [BackedAnnotatedField] @Inject io.narayana.lra.coordinator.api.RecoveryCoordinator.lraService
> at io.narayana.lra.coordinator.api.RecoveryCoordinator.lraService(RecoveryCoordinator.java:0)
> WELD-001474: Class io.narayana.lra.coordinator.domain.service.LRAService is on the classpath, but was ignored because a class it references was not found: com.arjuna.ats.arjuna.recovery.RecoveryModule from [Module "deployment.lra-coordinator.jar" from Service Module Loader].
> {noformat}
--
This message was sent by Atlassian Jira
(v7.13.8#713008)
4 years, 10 months
[JBoss JIRA] (JBTM-3300) Enable the checkstyle plugin on as many non Arjuna modules as makes sense
by Michael Musgrove (Jira)
[ https://issues.redhat.com/browse/JBTM-3300?page=com.atlassian.jira.plugin... ]
Michael Musgrove closed JBTM-3300.
----------------------------------
Resolution: Rejected
Rejecting in favour of the following policy that we've been following since 2005:
* if you add a new file it MUST adhere to our checkstyle ruleset
* if you change a file (in a non trivial way) you are allowed (MAY) to reformat the code to conform to our checkstyle ruleset. If you choose not to reformat it then you SHOULD, where possible, try to follow the same style that's already in use for that file.
We will add a build property called checkstyle that you can pass to the compilation. Note that we need our own property because you cannot override a plugin property that is already set in the plugin configuration section (ie -Dcheckstyle.skip=false does not work).
We will also update the "New Pull Request" github message to reference this policy.
> Enable the checkstyle plugin on as many non Arjuna modules as makes sense
> -------------------------------------------------------------------------
>
> Key: JBTM-3300
> URL: https://issues.redhat.com/browse/JBTM-3300
> Project: JBoss Transaction Manager
> Issue Type: Task
> Components: LRA, REST, TxBridge, XTS
> Affects Versions: 5.10.4.Final
> Reporter: Michael Musgrove
> Assignee: Michael Musgrove
> Priority: Optional
> Fix For: 5.next
>
>
> The checkstyle plugin is intended for use on new files but it is currently disabled. We should enable for as many of our modules as possible.
> Arjuna modules should be precluded.
> I am not sure if we want to enable it for STM.
> Note that we use the WildFly checkstyle rule set (https://github.com/wildfly/wildfly-checkstyle-config/tree/master/src/main...).
--
This message was sent by Atlassian Jira
(v7.13.8#713008)
4 years, 10 months
[JBoss JIRA] (JBTM-3300) Enable the checkstyle plugin on as many non Arjuna modules as makes sense
by Michael Musgrove (Jira)
[ https://issues.redhat.com/browse/JBTM-3300?page=com.atlassian.jira.plugin... ]
Michael Musgrove commented on JBTM-3300:
----------------------------------------
Rejecting in favour of the following policy that we've been following since 2005:
- if you add a new file it MUST adhere to our checkstyle ruleset
- if you change a file (in a non trivial way) you are allowed (MAY) to reformat the code to conform to our checkstyle ruleset. If you choose not to reformat it then you SHOULD, where possible, try to follow the same style that's already in use for that file.
We will add a build property called checkstyle that you can pass to the compilation. Note that we need our own property because you cannot override a plugin property that is already set in the plugin configuration section (ie -Dcheckstyle.skip=false does not work).
We will also update the "New Pull Request" github message to reference this policy.
> Enable the checkstyle plugin on as many non Arjuna modules as makes sense
> -------------------------------------------------------------------------
>
> Key: JBTM-3300
> URL: https://issues.redhat.com/browse/JBTM-3300
> Project: JBoss Transaction Manager
> Issue Type: Task
> Components: LRA, REST, TxBridge, XTS
> Affects Versions: 5.10.4.Final
> Reporter: Michael Musgrove
> Assignee: Michael Musgrove
> Priority: Optional
> Fix For: 5.next
>
>
> The checkstyle plugin is intended for use on new files but it is currently disabled. We should enable for as many of our modules as possible.
> Arjuna modules should be precluded.
> I am not sure if we want to enable it for STM.
> Note that we use the WildFly checkstyle rule set (https://github.com/wildfly/wildfly-checkstyle-config/tree/master/src/main...).
--
This message was sent by Atlassian Jira
(v7.13.8#713008)
4 years, 10 months
[JBoss JIRA] (JBTM-3299) Expose optional additional description of LRARecord status
by Martin Stefanko (Jira)
Martin Stefanko created JBTM-3299:
-------------------------------------
Summary: Expose optional additional description of LRARecord status
Key: JBTM-3299
URL: https://issues.redhat.com/browse/JBTM-3299
Project: JBoss Transaction Manager
Issue Type: Enhancement
Components: LRA
Affects Versions: 5.10.4.Final
Reporter: Martin Stefanko
Assignee: Martin Stefanko
The LRA record can be in a state, for instance, Completing or Compensating into which it got based on different criteria:
* on users request (by user returning 202 status code from Complete or Compensate method)
* because the Complete or Compensate invocation didn't succeed and the coordinator will retry in the next recovery run
This distinction may be useful for users querying LRAs so we can include it in the JSON response representing the LRARecord.
--
This message was sent by Atlassian Jira
(v7.13.8#713008)
4 years, 10 months