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

[Backport stable/8.4] fix: only create one CREATED event per form on distribution #25745

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

backport-action
Copy link
Collaborator

Description

Backport of #25728 to stable/8.4.

relates to #25727
original author: @tmetzke

Upon deployment CREATE distribution, we only create one CREATED event per form with the defined
resource content per partition. We use the checksum to identify the unqiue form resources instead
of relying on resource names as those can occur multiple times per deployment.

Otherwise, we are creating too many CREATED events for each form resource, one for each other form
with the same name and the content of that other form. This can lead to forms having the wrong
content in the end, depending on which event was processed last for a form.

(cherry picked from commit 253a823)
Ensures forms are only deployed once per partition with their defined content on deployment
distribution, even when there are multiple forms with the same name in one deployment.

(cherry picked from commit 705768f)
Copy link
Contributor

github-actions bot commented Dec 9, 2024

The latest Buf updates on your PR. Results from workflow Zeebe CI / protobuf-checks (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed⏩ skipped⏩ skipped✅ passedDec 9, 2024, 8:48 AM

@camundait camundait added this pull request to the merge queue Dec 9, 2024
Merged via the queue into stable/8.4 with commit 7a444cc Dec 9, 2024
32 checks passed
@camundait camundait deleted the backport-25728-to-stable/8.4 branch December 9, 2024 09:45
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

Successfully merging this pull request may close these issues.

3 participants