Skip to content

[monitor-opentelemetry-exporter] Remove custom properties truncation#45118

Merged
hectorhdzg merged 11 commits intoAzure:mainfrom
hectorhdzg:hectorhdzg/removetrunc
Feb 23, 2026
Merged

[monitor-opentelemetry-exporter] Remove custom properties truncation#45118
hectorhdzg merged 11 commits intoAzure:mainfrom
hectorhdzg:hectorhdzg/removetrunc

Conversation

@hectorhdzg
Copy link
Member

Remove 64kb limitation for custom properties field

Copilot AI review requested due to automatic review settings February 11, 2026 00:15
@github-actions github-actions bot added the Monitor - Exporter Monitor OpenTelemetry Exporter label Feb 11, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Azure Monitor OpenTelemetry Exporter utilities to stop truncating custom property values (previously capped at 64KiB), allowing full values to be sent through as strings.

Changes:

  • Removed the 64KiB value truncation logic from _filter_custom_properties.
  • Renamed the local accumulator variable from truncated_properties to filtered_properties to reflect the new behavior.

Copy link
Member

@rads-1996 rads-1996 left a comment

Choose a reason for hiding this comment

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

LGTM

@hectorhdzg hectorhdzg enabled auto-merge (squash) February 23, 2026 22:10
@hectorhdzg hectorhdzg merged commit 2ca8efb into Azure:main Feb 23, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Monitor - Exporter Monitor OpenTelemetry Exporter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants