all: drop attribute values restrictions #707
Open
+24
−90
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR removes attribute value restrictions from the OpenTelemetry protocol definitions, allowing receivers to handle complex values that were previously restricted. This change aligns with the OpenTelemetry specification's extension to support complex attribute values.
- Removes
SHOULD NOTrestrictions on empty values, bytes values, complex arrays, and kvlist values from attribute documentation - Adds consistent warning about unpredictable behavior when duplicate keys are received
- Updates changelog to document the breaking change
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| opentelemetry/proto/trace/v1/trace.proto | Removes attribute value restrictions from span, event, and link attributes |
| opentelemetry/proto/resource/v1/resource.proto | Removes attribute value restrictions from resource attributes |
| opentelemetry/proto/profiles/v1development/profiles.proto | Removes attribute value restrictions from profile attributes |
| opentelemetry/proto/metrics/v1/metrics.proto | Removes attribute value restrictions from metric data points and adds duplicate key warning |
| opentelemetry/proto/logs/v1/logs.proto | Adds duplicate key warning to log record attributes |
| opentelemetry/proto/common/v1/common.proto | Adds duplicate key warnings to KeyValueList and InstrumentationScope |
| CHANGELOG.md | Documents the breaking change in the changelog |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
pellared
changed the title
trask
approved these changes
Aug 26, 2025
lmolkova
approved these changes
Aug 26, 2025
tigrannajaryan
approved these changes
Aug 26, 2025
tigrannajaryan
previously requested changes
Aug 26, 2025
aalexand
reviewed
Aug 28, 2025
MrAlias
approved these changes
Aug 29, 2025
carlosalberto
approved these changes
Sep 1, 2025
pellared
changed the title
@tigrannajaryan , it can be unblocked per https://github.com/open-telemetry/opentelemetry-proto/releases/tag/v1.8.0. Thanks a lot for your help 👍 |
bogdandrutu
approved these changes
Sep 4, 2025
reyang
approved these changes
Sep 5, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related spec PR: open-telemetry/opentelemetry-specification#4636
Towards open-telemetry/opentelemetry-specification#4602
Follows open-telemetry/opentelemetry-specification#4614
Per https://github.com/open-telemetry/opentelemetry-specification/blob/main/oteps/4485-extending-attributes-to-support-complex-values.md#proto
We are dropping the attributes values restriction.
Receivers of
opentelemetry-proto(most probably)v1.9.0are expected to handle attribute values which before were expected only in logs signal (like complex values, bytes, or empty value).See Copilot's description: #707 (review)