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.

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

Adds Azure Event Hubs request telemetry #437

Closed
stijnmoreels opened this issue Jul 13, 2022 · 0 comments · Fixed by #439
Closed

Adds Azure Event Hubs request telemetry #437

stijnmoreels opened this issue Jul 13, 2022 · 0 comments · Fixed by #439
Assignees
@stijnmoreels
Labels
application-insights All issues related to Azure Application Insights requests All issues related to requests specs-required All issues where the specifications are still being defined and implementation should be halted
Milestone
v2.6.0

Comments

@stijnmoreels
Copy link
Member

Is your feature request related to a problem? Please describe.
When we receive messages from an API or Azure Service Bus, we are able to track this request with a dedicated logger extensions, but we can't do that if we want to receive requests from Azure EventHubs.

Describe the solution you'd like
Add an ILogger extension called LogEventHubsRequest that takes in:

  • EventHubName as telemetry context property
  • ConsumerGroup as telemetry context property
  • EventHubsNamespace as telemetry context property

Additional context
This request tracking will be used in the arcus-azure/arcus.messaging#24
@stijnmoreels stijnmoreels added application-insights All issues related to Azure Application Insights requests All issues related to requests labels Jul 13, 2022
@stijnmoreels stijnmoreels added this to the v3.0.0 milestone Jul 13, 2022
@stijnmoreels stijnmoreels added the specs-required All issues where the specifications are still being defined and implementation should be halted label Jul 13, 2022
@stijnmoreels stijnmoreels modified the milestones: v3.0.0, v2.6.0 Jul 14, 2022
@stijnmoreels stijnmoreels self-assigned this Jul 18, 2022
@stijnmoreels stijnmoreels mentioned this issue Jul 19, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stijnmoreels stijnmoreels

Labels
application-insights All issues related to Azure Application Insights requests All issues related to requests specs-required All issues where the specifications are still being defined and implementation should be halted
Projects
None yet
Milestone
v2.6.0
Development

Successfully merging a pull request may close this issue.

feat: add eventhubs request tracking stijnmoreels/arcus.observability
1 participant
@stijnmoreels