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

Detect process resources #1261

Closed
dyladan opened this issue Jun 30, 2020 · 3 comments · Fixed by #1531
Closed

Detect process resources #1261

dyladan opened this issue Jun 30, 2020 · 3 comments · Fixed by #1531
Assignees
@mihirsoni
Labels
feature-request

Comments

@dyladan
Copy link
Member

dyladan commented Jun 30, 2020

open-telemetry/opentelemetry-specification#635

Suggest adding a resource detector for process resources. This may be in a separate package, or at least should be node-only as the web browser does not make sense in this context.
@dyladan dyladan added feature-request up-for-grabs Good for taking. Extra help will be provided by maintainers labels Jun 30, 2020
@mihirsoni mihirsoni mentioned this issue Sep 14, 2020
Merged
@dyladan
Copy link
Member Author

dyladan commented Sep 14, 2020

@mihirsoni can you comment on this issue so I can assign it to you?

@mihirsoni
Copy link
Contributor

@dyladan sure. I have already raised PR for the same.

@dyladan
Copy link
Member Author

dyladan commented Sep 14, 2020

@dyladan sure. I have already raised PR for the same.

That's why I wanted to assign you :)

@dyladan dyladan removed the up-for-grabs Good for taking. Extra help will be provided by maintainers label Sep 14, 2020
pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this issue Dec 15, 2023
@blumamir
docs: add instrumentation dependencies guidelines (open-telemetry#1261)
d39595a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mihirsoni mihirsoni

Labels
feature-request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: process resource detector mihirsoni/opentelemetry-js
2 participants
@mihirsoni @dyladan