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

Update OTEL Host golden metrics and dashboards #1356

Closed
jlegoff opened this issue Aug 2, 2022 · 4 comments
Closed

Update OTEL Host golden metrics and dashboards #1356

jlegoff opened this issue Aug 2, 2022 · 4 comments

Comments

@jlegoff
Copy link
Contributor

jlegoff commented Aug 2, 2022

Following this change in the specs, direction will disappear as an attribute and be part of the metric name. For instance system.network.io will be replaced by system.network.io.transmit and system.network.io.receive. This directly impacts the golden metric definitions and dashboard, which should be updated.

@davidgit davidgit added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/short-term Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Aug 22, 2022
@josemore josemore removed triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/short-term Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Oct 4, 2022
@workato-integration
Copy link

@workato-integration
Copy link

Jira CommentId: 118943
Commented by jmore:

agreed to block this until we clarify what's the final decision on the spec (latest collector is not deprecating the "direction" attribute nor changing the metric names)

@workato-integration
Copy link

Jira CommentId: 125892
Commented by rruizdegauna:

For some metrics, this decision was reverted (network among those)

[https://github.com/open-telemetry/opentelemetry-specification/issues/2726|https://github.com/open-telemetry/opentelemetry-specification/issues/2726]

So we can close this issue :)

@workato-integration
Copy link

This issue won't be actioned.

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

No branches or pull requests

3 participants