-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Update AzureEventsHubReceiver with new resource logs translator #35357
Update AzureEventsHubReceiver with new resource logs translator #35357
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.
Looks great!
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
…tor-contrib into mike/azure-events-receiver-translator # Conflicts: # cmd/otelcontribcol/go.mod # receiver/azureeventhubreceiver/go.mod
Updated branch with latest on main. Please can we get more of @open-telemetry/collector-contrib-approvers to take a look? |
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.
Thanks @MikeGoldsmith
…-telemetry#35357) **Description:** Updates the AzureEventsHubReceiver to use the new azure resource logs translator. Includes the following changes: - Add config option to enable the new behaviour, default is off so requires users to opt-in - Add interface to allow switching over the previous and new translator logic - Update receiver to use new config option to return the appropriate translator - Update README with new configuration option Follow-up PR to adding the translator: - open-telemetry#34830 **Link to tracking Issue:** N/A **Testing:** Includes unit test to verify configuration defaults and changes. **Documentation:** N/A --------- Co-authored-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Description:
Updates the AzureEventsHubReceiver to use the new azure resource logs translator.
Includes the following changes:
Follow-up PR to adding the translator:
Link to tracking Issue:
N/A
Testing:
Includes unit test to verify configuration defaults and changes.
Documentation:
N/A