Skip to content

[exporter/awsxrayexporter] Merge in latest semconv for awsxrayexporter.#36894

Merged
andrzej-stencel merged 4 commits into
open-telemetry:mainfrom
zzhlogin:otel_semconv
Jan 3, 2025
Merged

[exporter/awsxrayexporter] Merge in latest semconv for awsxrayexporter.#36894
andrzej-stencel merged 4 commits into
open-telemetry:mainfrom
zzhlogin:otel_semconv

Conversation

@zzhlogin

Copy link
Copy Markdown
Contributor

Description

The PR merge in latest semconv pkg and update existing semantic conventions utilization. And further update awsxrayexporter to be able to handle following both new versions of http
span attributes:
http.response.status_code migrated from http.status_code
url.query migrated from http.target.

Otel Java SDK has renamed telemetry.auto.version to telemetry.distro.version (Upstream PR), update awsxrayexporter to be able to detect both attributes.

Link to tracking issue

N/A

Testing

Unit test passed

Documentation

N/A

Comment thread exporter/awsxrayexporter/internal/translator/aws.go Outdated
@andrzej-stencel
andrzej-stencel merged commit e38d233 into open-telemetry:main Jan 3, 2025
@github-actions github-actions Bot added this to the next release milestone Jan 3, 2025
AkhigbeEromo pushed a commit to sematext/opentelemetry-collector-contrib that referenced this pull request Jan 13, 2025
…r. (open-telemetry#36894)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
The PR merge in latest semconv pkg and update existing semantic
conventions utilization. And further update `awsxrayexporter` to be able
to handle following both new versions of http
span attributes:
`http.response.status_code` migrated from
[http.status_code](https://opentelemetry.io/docs/specs/semconv/attributes-registry/http/#http-status-code)
`url.query` migrated from
[http.target](https://opentelemetry.io/docs/specs/semconv/attributes-registry/http/#http-target).

Otel Java SDK has renamed `telemetry.auto.version` to
`telemetry.distro.version` ([Upstream
PR](open-telemetry/opentelemetry-java-instrumentation#9065)),
update `awsxrayexporter` to be able to detect both attributes.

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
N/A

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Unit test passed

<!--Describe the documentation added.-->
#### Documentation
N/A
zeck-ops pushed a commit to zeck-ops/opentelemetry-collector-contrib that referenced this pull request Apr 23, 2025
…r. (open-telemetry#36894)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
The PR merge in latest semconv pkg and update existing semantic
conventions utilization. And further update `awsxrayexporter` to be able
to handle following both new versions of http
span attributes:
`http.response.status_code` migrated from
[http.status_code](https://opentelemetry.io/docs/specs/semconv/attributes-registry/http/#http-status-code)
`url.query` migrated from
[http.target](https://opentelemetry.io/docs/specs/semconv/attributes-registry/http/#http-target).

Otel Java SDK has renamed `telemetry.auto.version` to
`telemetry.distro.version` ([Upstream
PR](open-telemetry/opentelemetry-java-instrumentation#9065)),
update `awsxrayexporter` to be able to detect both attributes.

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
N/A

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Unit test passed

<!--Describe the documentation added.-->
#### Documentation
N/A
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.

4 participants