-
Notifications
You must be signed in to change notification settings - Fork 293
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
ResourceDetectors - Process #1506
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1506 +/- ##
==========================================
- Coverage 73.91% 73.36% -0.55%
==========================================
Files 267 254 -13
Lines 9615 9414 -201
==========================================
- Hits 7107 6907 -200
+ Misses 2508 2507 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
c966b54
to
425e3d9
Compare
<MinVerTagPrefix>ResourceDetectors.Process-</MinVerTagPrefix> | ||
</PropertyGroup> | ||
<ItemGroup> | ||
<PackageReference Include="OpenTelemetry" Version="1.6.0" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OTelSdkVersion
can be used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From PR description:
The plan is to release shortly as alpha with reference to OTel 1.6.0.
Then create follow up help-wanted issue to extend it for another attributes and update OTel to 1.7.0.
Will do, just after initial release.
Fixes N/A.
Changes
Initial implementation for
process.pid
detector from https://github.com/open-telemetry/semantic-conventions/blob/main/docs/resource/process.md#process (this attribute is what I need for now).The plan is to release shortly as alpha with reference to OTel 1.6.0.
Then create follow up
help-wanted
issue to extend it for another attributes and update OTel to 1.7.0.For significant contributions please make sure you have completed the following items:
CHANGELOG.md
updated for non-trivial changes[ ] Design discussion issue #Notes
Previous initial implementation of resource detector: #1449
You can expect one more similar, for https://github.com/open-telemetry/semantic-conventions/blob/main/docs/resource/host.md, espiecially
host.name
.