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

Added service.instance.id into resource env #2259

Merged
merged 5 commits into from
Nov 14, 2023

Conversation

yuriolisa
Copy link
Contributor

Description:

Added the service.instance.id as os.Hostname into the traces resource Env.

Link to tracking Issue:
#1921

Testing:
Changed the unit tests to include the service.instance.id information.

Documentation:

Signed-off-by: Yuri Sa <yurimsa@gmail.com>
@yuriolisa yuriolisa requested a review from a team October 23, 2023 18:25
Signed-off-by: Yuri Sa <yurimsa@gmail.com>
Signed-off-by: Yuri Sa <yurimsa@gmail.com>
Signed-off-by: Yuri Sa <yurimsa@gmail.com>
@yuriolisa yuriolisa requested a review from jpkrohling November 14, 2023 05:46
@jaronoff97 jaronoff97 merged commit 7e6e3e0 into open-telemetry:main Nov 14, 2023
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* Added service.instance.id into resource env

Signed-off-by: Yuri Sa <yurimsa@gmail.com>

* Changed ServiceInstanceID following the RFC4122

Signed-off-by: Yuri Sa <yurimsa@gmail.com>

* Changed ServiceInstanceID following the RFC4122

Signed-off-by: Yuri Sa <yurimsa@gmail.com>

* Added semantic conventions

Signed-off-by: Yuri Sa <yurimsa@gmail.com>

---------

Signed-off-by: Yuri Sa <yurimsa@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants