Skip to content

[pkg/translator/azurelogs] Support additional fields not in the common schema#46165

Open
rafaelrodrigues3092 wants to merge 16 commits intoopen-telemetry:mainfrom
rafaelrodrigues3092:fix/46162
Open

[pkg/translator/azurelogs] Support additional fields not in the common schema#46165
rafaelrodrigues3092 wants to merge 16 commits intoopen-telemetry:mainfrom
rafaelrodrigues3092:fix/46162

Conversation

@rafaelrodrigues3092
Copy link
Copy Markdown

@rafaelrodrigues3092 rafaelrodrigues3092 commented Feb 17, 2026

Description

Azure resource logs are not standard, see Common and service-specific schemas for Azure resource logs.

When consuming resource logs for services like Service Bus or Event Hub, timestamp parsing fails.
The log events are also not helpful identifying the resource or resource type causing the parsing issue.

This change:

  • Adds support for alternative timestamp fields: EventTimeString, EventTimestamp, startTime
  • Adds alternative field for properties: EventProperties
  • Improves warning logs to be able to better identify the resource and resource type that caused the parsing issue.

Link to tracking issue

Fixes #46162

Testing

  • Added testing for the added fields.
  • Tested two scenarios
    • Previously supported payload
    • Previously unsupported payload

Documentation

Updated Azure Event Hub Receiver ReadMe to include the additional supported fields.

Additional Notes

Assisted-by: Claude Opus 4.6

@rafaelrodrigues3092 rafaelrodrigues3092 requested review from a team and atoulme as code owners February 17, 2026 20:17
@github-actions github-actions Bot added the first-time contributor PRs made by new contributors label Feb 17, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

@rafaelrodrigues3092 rafaelrodrigues3092 changed the title added support for additional fields not in the common schema [pkg/translator/azurelogs] Support additional fields not in the common schema Feb 17, 2026
@thompson-tomo
Copy link
Copy Markdown
Contributor

Should we not be looking to introduce this in the unmarshaller so that we have consistency etc.

@paulojmdias
Copy link
Copy Markdown
Member

/workflow-approve

Copy link
Copy Markdown
Member

@MikeGoldsmith MikeGoldsmith left a comment

Choose a reason for hiding this comment

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

Looks good to me - thanks @rafaelrodrigues3092 👍🏻

Comment thread pkg/translator/azurelogs/resourcelogs_to_logs.go Outdated
Comment thread pkg/translator/azurelogs/resourcelogs_to_logs.go
@github-actions
Copy link
Copy Markdown
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 Mar 24, 2026
@MikeGoldsmith
Copy link
Copy Markdown
Member

MikeGoldsmith commented Mar 30, 2026

Not stale - please can someone from @open-telemetry/collector-contrib-approvers review?

@github-actions github-actions Bot removed the Stale label Mar 31, 2026
Copy link
Copy Markdown
Contributor

@axw axw left a comment

Choose a reason for hiding this comment

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

@rafaelrodrigues3092 sorry for the delay. Looks good overall, just one question.

Comment thread pkg/translator/azurelogs/resourcelogs_to_logs.go Outdated
@atoulme
Copy link
Copy Markdown
Contributor

atoulme commented Apr 23, 2026

Please address the CI and mark ready for review again.

@atoulme atoulme marked this pull request as draft April 23, 2026 01:00
@rafaelrodrigues3092
Copy link
Copy Markdown
Author

lint issues fixed. Should be ready for review

@rafaelrodrigues3092 rafaelrodrigues3092 marked this pull request as ready for review April 25, 2026 23:19
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.

[pkg/translator/azurelogs] Support additional fields not in the common schema

7 participants