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.

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

Zipkin exporter should support Array attributes #1110

Closed
codeboten opened this issue Sep 14, 2020 · 2 comments · Fixed by #1285
Closed

Zipkin exporter should support Array attributes #1110

codeboten opened this issue Sep 14, 2020 · 2 comments · Fixed by #1285
Assignees
Labels
1.0.0rc2 release candidate 2 for tracing GA good first issue Good first issue help wanted release:required-for-ga To be resolved before GA release

Comments

@codeboten
Copy link
Contributor

As per spec: https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/sdk_exporters/zipkin.md#attribute

@codeboten codeboten added good first issue Good first issue help wanted release:required-for-ga To be resolved before GA release labels Sep 14, 2020
@rydzykje
Copy link
Contributor

@codeboten will be happy to help with that :)

robwknox added a commit to robwknox/opentelemetry-python that referenced this issue Oct 26, 2020
robwknox added a commit to robwknox/opentelemetry-python that referenced this issue Oct 26, 2020
robwknox added a commit to robwknox/opentelemetry-python that referenced this issue Oct 27, 2020
@robwknox
Copy link
Contributor

robwknox commented Oct 29, 2020

@codeboten took a stab at this with PR #1285

srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this issue Nov 1, 2020
@codeboten codeboten added the 1.0.0rc2 release candidate 2 for tracing GA label Nov 26, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.0.0rc2 release candidate 2 for tracing GA good first issue Good first issue help wanted release:required-for-ga To be resolved before GA release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants