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

feat: added @opentelemetry/resource-detector-instana to cart service #402

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

kirrg001
Copy link
Collaborator

@kirrg001 kirrg001 commented Jul 19, 2022

This PR adds the newly added Instana Otel resource detector to the cart codebase.

The resource detection processor can be used to detect resource information from the host.

In Opentelemetry there is the concept of a resource, see [Resource SDK](https://opentelemetry.io/docs/reference/specification/resource/sdk/). There are process resources, environment resources and custom resources. All will be merged into one resource object, which is attached to every trace object.

The goal for Instana is to use this component as a detector for our Instana agent and attach the real PID and the agent uuid to the custom resource. This will correlate the Otel service spans with the Otel process.

@kirrg001 kirrg001 requested a review from basti1302 July 19, 2022 20:32
@kirrg001 kirrg001 changed the title [WIP] Added resource detection for cart service Added resource detection for cart service Aug 10, 2022
@basti1302 basti1302 removed their request for review August 10, 2022 07:52
@basti1302 basti1302 removed their assignment Aug 10, 2022
@kirrg001 kirrg001 force-pushed the feat/resource-detection branch from 434d867 to c78ca99 Compare August 10, 2022 15:53
@kirrg001 kirrg001 changed the title Added resource detection for cart service feat: added @opentelemetry/resource-detector-instana to cart service Aug 10, 2022
@kirrg001 kirrg001 requested a review from basti1302 August 10, 2022 15:56
Copy link

@basti1302 basti1302 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kirrg001 kirrg001 merged commit cab8688 into master Aug 17, 2022
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.

2 participants