[receiver/hostmetrics] Add new process metrics #12972
Conversation
|
@evan-bradley did you submit a spec PR for adding semantic conventions for these metrics? |
|
I opened the following spec PR to add these metrics to the semantic conventions: open-telemetry/opentelemetry-specification#2706 |
receiver/hostmetricsreceiver/internal/scraper/processscraper/metadata.yaml
Outdated
Show resolved
Hide resolved
receiver/hostmetricsreceiver/internal/scraper/processscraper/process_scraper.go
Outdated
Show resolved
Hide resolved
Thanks! Let's make sure that PR is merged first |
receiver/hostmetricsreceiver/internal/scraper/processscraper/metadata.yaml
Outdated
Show resolved
Hide resolved
receiver/hostmetricsreceiver/internal/scraper/processscraper/documentation.md
Outdated
Show resolved
Hide resolved
receiver/hostmetricsreceiver/internal/scraper/processscraper/documentation.md
Outdated
Show resolved
Hide resolved
evan-bradley
force-pushed
the
issue-12482
branch
2 times, most recently
from
c30c79b
to
1a93fb8
Compare
|
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
|
This is currently waiting on open-telemetry/opentelemetry-specification#2726 to be resolved to determine which direction to take with open-telemetry/opentelemetry-specification#2706. I will update this PR as soon as the semantic conventions PR has been merged. |
|
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
evan-bradley
force-pushed
the
issue-12482
branch
from
1a93fb8
to
7c5f789
Compare
|
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
evan-bradley
force-pushed
the
issue-12482
branch
2 times, most recently
from
0645b27
to
ba57ec3
Compare
|
@dmitryax Please take a look. The PR for the semantic conventions for these metrics has been merged, though it is not yet released. |
|
@dmitryax These metrics have been included in the semantic conventions in version 1.14.0 of the specification. |
evan-bradley
force-pushed
the
issue-12482
branch
2 times, most recently
from
71c84a8
to
637ada7
Compare
|
@dmitryax Please take a look. The new metrics are part of the semantic conventions, and the PR has been rebased. |
|
Need #15274 to pass CI |
evan-bradley
force-pushed
the
issue-12482
branch
from
637ada7
to
188eac9
Compare
Add the following process metrics: - `process.context_switches` - `process.open_file_descriptors`
evan-bradley
force-pushed
the
issue-12482
branch
from
188eac9
to
567a28c
Compare
|
Can this be merged @dmitryax? |
Add the following process metrics: - `process.context_switches` - `process.open_file_descriptors` Co-authored-by: Evan Bradley <[email protected]>
Add the following process metrics: - `process.context_switches` - `process.open_file_descriptors` Co-authored-by: Evan Bradley <[email protected]>
Description:
Add the following process metrics:
process.context_switchesprocess.open_file_descriptorsLink to tracking Issue:#12482
Spec PR to add semantic conventions for the added metrics: open-telemetry/opentelemetry-specification#2706
Testing:
Unit tests and manual testing of the metrics.