Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-14408

BpmnError thrown from an execution listener doesn't cancel activity instance




      Environment (Required on creation):

      Any Camunda-supported distribution or a JUnit test case.

      Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket):

      A BPMNError thrown from an ExecutionListener doesn't cancel the Activity Instance of the corresponding Execution.

      Steps to reproduce (Required on creation):

      Start a process instance of errorProcess.bpmn on any Camunda-supported distribution or a JUnit test case.

      Observed Behavior (Required on creation):

      The process instance is marked as completed. However:

      • The history view in Cockpit EE shows the Service Task activity instance as running.
      • There is no end time set on the Service Task (historic) activity instance.
      • The Historic Activity Instance state remains at '0' (running).

      Expected behavior (Required on creation):

      When a BpmnError is thrown from an ExecutionListener, the corresponding activity instance is marked as "canceled" and an "end time" is set.

      Root Cause (Required on prioritization):

      Solution Ideas (Optional):

      • We should then make sure that the activity instance always transitions to the ending state (with its side effects, like creating history events), while still ensuring that no execution listener is executed twice (on end, and then on cancel).
      • Maybe it makes sense to couple the transition from one activity state to the next stronger to the effects that must happen with that transition, so that such inconsistencies are less likely to occur in the future. Might be hard to achieve though.

      Hints (optional):


        This is the controller panel for Smart Panels app


          Issue Links



                Unassigned Unassigned
                nikola.koevski Nikola Koevski
                Tobias Metzke-Bernstein Tobias Metzke-Bernstein
                Nikola Koevski Nikola Koevski
                0 Vote for this issue
                3 Start watching this issue