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

DO NOT MERGE: Compute ext-service OpenTelemetry golden metrics from metrics instead of spans #1780

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jack-berg
Copy link
Contributor

These are the same query definitions which drive the OpenTelemetry metrics-based UI view.

select: rate(count(*), 1 minute)
from: Span
where: span.kind LIKE 'server' OR span.kind LIKE 'consumer' OR kind LIKE 'server' OR kind LIKE 'consumer'
select: rate(count(%.server%duration), 1 minute)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do we expect any issues defining golden metrics with wildcard syntax?

github-actions[bot]
github-actions bot previously approved these changes Nov 11, 2024
select: average(duration.ms)
from: Span
where: span.kind LIKE 'server' OR span.kind LIKE 'consumer' OR kind LIKE 'server' OR kind LIKE 'consumer'
select: average(convert(%.server%duration, unit OR 's', 'ms'))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do we expect any issues defining golden metrics with the convert function?

github-actions[bot]
github-actions bot previously approved these changes Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants