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

Writer timer triggered event with process instance key #9519

Closed
saig0 opened this issue Jun 8, 2022 · 3 comments · Fixed by #9520
Closed

Writer timer triggered event with process instance key #9519

saig0 opened this issue Jun 8, 2022 · 3 comments · Fixed by #9520
Assignees
Labels
kind/feature Categorizes an issue or PR as a feature, i.e. new behavior scope/broker Marks an issue or PR to appear in the broker section of the changelog version:8.1.0-alpha3 Marks an issue as being completely or in parts released in 8.1.0-alpha3 version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0

Comments

@saig0
Copy link
Member

saig0 commented Jun 8, 2022

Is your feature request related to a problem? Please describe.
I deployed a process with a timer start event. If the timer is triggered then a new process instance is created and an event of type timer with intent triggered is written.

As a monitoring application, I want to track which process instance was created by this timer. But currently, the timer: triggered event doesn't contain the key of the created process instance.

Describe the solution you'd like
The timer: triggered event contains the key of the created process instance.

Describe alternatives you've considered
Add a reference of the timer to the created process instance. But this option seems not practical because a process instance can be created by different triggers, for example, a message.

Additional context
In a monitoring application, I want to track that a process instance was created by a triggered timer start event.

@saig0 saig0 added kind/feature Categorizes an issue or PR as a feature, i.e. new behavior scope/broker Marks an issue or PR to appear in the broker section of the changelog team/process-automation labels Jun 8, 2022
@saig0 saig0 self-assigned this Jun 8, 2022
@skayliu
Copy link
Contributor

skayliu commented Jun 8, 2022

Hi, @saig0 may be the elementInstanceKey shoub be include too?

@saig0
Copy link
Member Author

saig0 commented Jun 8, 2022

@skayliu interesting question. But no, the element instance key would not fit in my mind. The timer creates a new process instance but the rest (e.g. activation of the start event) happens later. So what should be the element instance key? The timer is not bound to an element instance.

@skayliu
Copy link
Contributor

skayliu commented Jun 8, 2022

@saig0 Thank you ,I get it .

zeebe-bors-camunda bot added a commit that referenced this issue Jun 9, 2022
9520: Writer timer triggered event with process instance key r=remcowesterhoud a=saig0

## Description

* minimal change in the timer processor to write the timer `triggered` event with the key of the created process instance 

## Related issues

closes #9519



Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
zeebe-bors-camunda bot added a commit that referenced this issue Jun 9, 2022
9520: Writer timer triggered event with process instance key r=remcowesterhoud a=saig0

## Description

* minimal change in the timer processor to write the timer `triggered` event with the key of the created process instance 

## Related issues

closes #9519



Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
zeebe-bors-camunda bot added a commit that referenced this issue Jun 10, 2022
9520: Writer timer triggered event with process instance key r=saig0 a=saig0

## Description

* minimal change in the timer processor to write the timer `triggered` event with the key of the created process instance 

## Related issues

closes #9519



Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
@oleschoenburg oleschoenburg added the version:8.1.0-alpha3 Marks an issue as being completely or in parts released in 8.1.0-alpha3 label Jul 5, 2022
@Zelldon Zelldon added the version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0 label Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes an issue or PR as a feature, i.e. new behavior scope/broker Marks an issue or PR to appear in the broker section of the changelog version:8.1.0-alpha3 Marks an issue as being completely or in parts released in 8.1.0-alpha3 version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants