Skip to content

fix: handle exporter opening if record counters are not present in me… #37012

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 20, 2025
Merged

fix: handle exporter opening if record counters are not present in me… #37012

merged 1 commit into from
Aug 20, 2025

Conversation

RomanJRW
Copy link
Contributor

@RomanJRW RomanJRW commented Aug 20, 2025 •
edited
Loading

…tadata

Description

We identified a case (maybe multiple cases) whereby the exporter metadata contained no record counters. This prevented the exporter from opening successfully and no data can be exported. Context.

This PR adds a filter to handle this scenario and allow the Exporter to open successfully.

Checklist

Related issues

closes #37014

@github-actions github-actions bot added the component/zeebe Related to the Zeebe component/team label Aug 20, 2025
@RomanJRW RomanJRW force-pushed the jrw-handle-empty-record-counters branch from a85efe3 to a8e7084 Compare August 20, 2025 18:00
@RomanJRW RomanJRW added backport stable/8.6 Backport a pull request to stable/8.6 backport stable/8.7 Backport a pull request to stable/8.7 labels Aug 20, 2025
houssain-barouni
houssain-barouni approved these changes Aug 20, 2025
View reviewed changes
Copy link
Collaborator

@houssain-barouni houssain-barouni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
@RomanJRW Do you think we need to backport this ?

@RomanJRW
Copy link
Contributor Author

@houssain-barouni - Yes I think so, the backport labels are on the PR 👍

@houssain-barouni
Copy link
Collaborator

@houssain-barouni - Yes I think so, the backport labels are on the PR 👍

oh yes, I missed that

@RomanJRW RomanJRW added this pull request to the merge queue Aug 20, 2025
Merged via the queue into main with commit e0568f4 Aug 20, 2025
184 of 186 checks passed
@RomanJRW RomanJRW deleted the jrw-handle-empty-record-counters branch August 20, 2025 19:44
@backport-action
Copy link
Collaborator

Successfully created backport PR for stable/8.6:

This was referenced Aug 20, 2025
Merged
Merged
@backport-action
Copy link
Collaborator

Successfully created backport PR for stable/8.7:

github-merge-queue bot pushed a commit that referenced this pull request Aug 20, 2025
@camundait
[Backport stable/8.6] fix: handle exporter opening if record counters…
f415af8 
… are not present in me… (#37026)

# Description
Backport of #37012 to `stable/8.6`.

relates to #37014
github-merge-queue bot pushed a commit that referenced this pull request Aug 20, 2025
@RomanJRW
[Backport stable/8.7] fix: handle exporter opening if record counters…
5ca8d03 
… are not present in me… (#37027)

# Description
Backport of #37012 to `stable/8.7`.

relates to #37014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@houssain-barouni houssain-barouni houssain-barouni approved these changes

Assignees
No one assigned
Labels
backport stable/8.6 Backport a pull request to stable/8.6 backport stable/8.7 Backport a pull request to stable/8.7 component/zeebe Related to the Zeebe component/team version:8.6.25 version:8.7.11
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Empty exporter record metdata prevents the ES/OS exporters resuming successfully
3 participants
@RomanJRW @houssain-barouni @backport-action