-
Notifications
You must be signed in to change notification settings - Fork 561
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
[Backport stable/0.26] Test that a new incident is created when resolve incident fails to continue the processing #7139
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment
.startEvent() | ||
.serviceTask( | ||
"task", | ||
b -> b.zeebeJobType("task").zeebeInputExpression("unknown_var", "output")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like I mentioned in the other review, the input variable is still called output
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh wow, I thought you meant the comment 😨
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed here (I'll also update the others)
As part of 7127, I investigated failure cases of resolving an incident. This test makes sure that continuation failure for the resolution of an incident leads to a new incident. (cherry picked from commit bf11019, with modifications)
398db93
to
e2e77d7
Compare
bors merge |
7139: [Backport stable/0.26] Test that a new incident is created when resolve incident fails to continue the processing r=korthout a=korthout # Description Backport of #7130 to `stable/0.26`. relates to #7127 Modifications were necessary to resolve the conflicts (i.e. naming changes from 0.26 to 1.0.0). Co-authored-by: Nico Korthout <nico.korthout@camunda.com>
Build failed: |
Flaky test bors retry |
7140: test(engine): use correct input expression target r=pihme a=korthout ## Description <!-- Please explain the changes you made here. --> I missed a review comment in #7130. It came back up in [a review comment](#7139 (comment)) on the manual backport PR. This fixes it for the develop branch. ## Related issues <!-- Which issues are closed by this PR or are related --> relates to #7127 Co-authored-by: Nico Korthout <nico.korthout@camunda.com>
Build succeeded: |
7140: test(engine): use correct input expression target r=pihme a=korthout ## Description <!-- Please explain the changes you made here. --> I missed a review comment in #7130. It came back up in [a review comment](#7139 (comment)) on the manual backport PR. This fixes it for the develop branch. ## Related issues <!-- Which issues are closed by this PR or are related --> relates to #7127 Co-authored-by: Nico Korthout <nico.korthout@camunda.com>
Description
Backport of #7130 to
stable/0.26
.relates to #7127
Modifications were necessary to resolve the conflicts (i.e. naming changes from 0.26 to 1.0.0).