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

Add support for map and array attribute values across all receivers and exporters #1590

Closed
kbrockhoff opened this issue Aug 18, 2020 · 3 comments
Closed

Add support for map and array attribute values across all receivers and exporters #1590

kbrockhoff opened this issue Aug 18, 2020 · 3 comments
Assignees
@kbrockhoff
Labels
enhancement New feature or request
Milestone
Backlog

Comments

@kbrockhoff
Copy link
Member

Is your feature request related to a problem? Please describe.
The specification and OTLP recently added map and array attribute values which need to be supported by all collector components.

Describe the solution you'd like
Add map and array values to the golden dataset and fix translators so all broken correctness tests pass again.

Describe alternatives you've considered
N/A

Additional context
Depends on merge of PR #1523
@kbrockhoff
Copy link
Member Author

I have started working on map support. Will add array support after merge of the PR mentioned above.

@tigrannajaryan tigrannajaryan added this to the Backlog milestone Aug 19, 2020
@tigrannajaryan
Copy link
Member

@kbrockhoff sorry for the delay, #1523 is now merged.

@kbrockhoff kbrockhoff mentioned this issue Aug 27, 2020
Merged
tigrannajaryan pushed a commit that referenced this issue Sep 16, 2020
@kbrockhoff
Add map and array attribute value type support (#1656)
471c4a6 
Map and array attribute value types which were recently added to the specification had not been implemented in the collector yet. This PR implements across all tracing receivers and exporters.

**Link to tracking Issue:** #1590

**Testing:** Added map and array attribute values to golden dataset. All unit and correctness tests continue to pass.
@kbrockhoff
Copy link
Member Author

Merged PR fixes this

@andrewhsu andrewhsu added the enhancement New feature or request label Jan 6, 2021
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this issue Apr 27, 2023
@jeffreyc-splunk @rmfitzpatrick
Add ubuntu 22.04 support to ansible (open-telemetry#1590)
ead1bb4 
* Add ubuntu 22.04 support to ansible

* Update deployments/ansible/CHANGELOG.md

Co-authored-by: Ryan Fitzpatrick <rmfitzpatrick@users.noreply.github.com>

* Update deployments/ansible/README.md

Co-authored-by: Ryan Fitzpatrick <rmfitzpatrick@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kbrockhoff kbrockhoff

Labels
enhancement New feature or request
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
3 participants
@andrewhsu @kbrockhoff @tigrannajaryan