Skip to content

[receiver/azureeventhubreceiver] Update default checkpoint when storage is enabled and storage compatibility#44461

Merged
atoulme merged 8 commits intoopen-telemetry:mainfrom
dyl10s:dylan/update-default-checkpoint-with-storage
Dec 31, 2025
Merged

[receiver/azureeventhubreceiver] Update default checkpoint when storage is enabled and storage compatibility#44461
atoulme merged 8 commits intoopen-telemetry:mainfrom
dyl10s:dylan/update-default-checkpoint-with-storage

Conversation

@dyl10s
Copy link
Contributor

@dyl10s dyl10s commented Nov 21, 2025

Description

  • When storage was enabled for either the legacy or the new implementation, the receiver would start at the beginning of the stream when it should be starting at the end of the stream.
  • Updated the new SDK checkpoint storage to be compatible with the legacy storage format. This makes sure there are no negative side effects from upgrading. The new SDK is still under a feature flag.

Testing

  • Make sure that both with legacy and azeventhub when storage is enabled and nothing is stored yet, that the default checkpoint is the latest. Tests were added to validate this case.
  • Verify that after running the current (legacy) version, you can enable the ezeventhub feature flag and the checkpoint is correctly resumed.

@dyl10s dyl10s force-pushed the dylan/update-default-checkpoint-with-storage branch 2 times, most recently from ed70177 to 3b41276 Compare November 25, 2025 13:57
@dyl10s dyl10s marked this pull request as ready for review November 25, 2025 13:57
@dyl10s dyl10s requested a review from a team as a code owner November 25, 2025 13:57
@dyl10s dyl10s requested a review from evan-bradley November 25, 2025 13:57
@github-actions github-actions bot requested a review from cparkins November 25, 2025 13:57
@dyl10s dyl10s requested a review from atoulme December 3, 2025 15:21
@dyl10s dyl10s force-pushed the dylan/update-default-checkpoint-with-storage branch from 3b41276 to d3d0e87 Compare December 11, 2025 21:19
@dyl10s dyl10s force-pushed the dylan/update-default-checkpoint-with-storage branch from d3d0e87 to 150a5d9 Compare December 12, 2025 13:55
@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Dec 27, 2025
@atoulme atoulme merged commit 50bc98f into open-telemetry:main Dec 31, 2025
189 checks passed
@github-actions github-actions bot added this to the next release milestone Dec 31, 2025
seongpil0948 pushed a commit to seongpil0948/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2026
…ge is enabled and storage compatibility (open-telemetry#44461)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
- When storage was enabled for either the legacy or the new
implementation, the receiver would start at the beginning of the stream
when it should be starting at the end of the stream.
- Updated the new SDK checkpoint storage to be compatible with the
legacy storage format. This makes sure there are no negative side
effects from upgrading. The new SDK is still under a feature flag.

<!--Describe what testing was performed and which tests were added.-->
#### Testing
- Make sure that both with `legacy` and `azeventhub` when storage is
enabled and nothing is stored yet, that the default checkpoint is the
latest. Tests were added to validate this case.
- Verify that after running the current (legacy) version, you can enable
the `ezeventhub` feature flag and the checkpoint is correctly resumed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants