-
Notifications
You must be signed in to change notification settings - Fork 363
fix: Dockerstats receiver otel golden-metrics #2407
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
base: main
Are you sure you want to change the base?
Conversation
lujop
left a comment
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.
Validations don't pass, look at networkTrafficTotal where you have bad identation
| select: max(docker.container.cpuPercent) or max(k8s.container.cpuCoresUtilization) AS 'CPU Utilization (%)' | ||
| opentelemetry: | ||
| # via dockerstatsreceiver & kubeletstatsreceiver – which send equivalent value named container.cpu.usage, container.cpu.utilization (deprecated) | ||
| select: max(container.cpu.usage or container.cpu.utilization) AS 'CPU Utilization (%)' |
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.
Is max this way correct, in some places you use the or inside and in others outside?
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.
There are two fields that a customer can set up in their OTel configuration, enabling the ingestion of either or both simultaneously, specifically for the dockerstats receiver. The other queries listed here pertain to metrics from the Kubernetes receiver, which are separate and not currently within scope. Just as a note, this change has already been merged into the main branch.
eca62cb to
937d018
Compare
entityBot
left a comment
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.
beep boop bop.
I have found some errors processing these changes:
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT rate(sum(container.cpu.usage.total)/1E9, 1 second) AS 'CPU usage (cores)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause uses a different value function than the rest.
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT rate(sum(container.network.rx_bytes), 1 second) + rate(sum(container.network.tx_bytes), 1 second) AS 'Network traffic (bytes per second)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause uses a different value function than the rest.
Let's validate those changes.
entityBot
left a comment
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.
beep boop bop.
I have found some errors processing these changes:
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT rate(sum(container.cpu.usage.total)/1E9, 1 second) AS 'CPU usage (cores)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause uses a different value function than the rest.
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT rate(sum(container.network.rx_bytes), 1 second) + rate(sum(container.network.tx_bytes), 1 second) AS 'Network traffic (bytes per second)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause uses a different value function than the rest.
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
1 similar comment
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
entityBot
left a comment
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.
beep boop bop.
I have found some errors processing these changes:
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT (sum(container.cpu.usage.total) / 1E9) / sum((endTimestamp - timestamp) / 1000) AS 'CPU usage (cores)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause uses a different value function than the rest.
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT (sum(container.network.rx_bytes) / sum((endTimestamp - timestamp) / 1000)) + (sum(container.network.tx_bytes) / sum((endTimestamp - timestamp) / 1000)) AS 'Network traffic (bytes per second)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause cannot be used to produce a valid golden metric.
|
👋 This PR has seen no activity in over 4 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
2 similar comments
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
entityBot
left a comment
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.
beep boop bop.
I have found some errors processing these changes:
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT (sum(container.cpu.usage.total) / 1E9) / sum((endTimestamp - timestamp) / 1000) AS 'CPU usage (cores)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause uses a different value function than the rest.
* Error VALIDATING domainType: 'INFRA-CONTAINER' on Golden metrics for key 'SELECT (sum(container.network.rx_bytes) / sum((endTimestamp - timestamp) / 1000)) + (sum(container.network.tx_bytes) / sum((endTimestamp - timestamp) / 1000)) AS 'Network traffic (bytes per second)' FROM Metric WHERE entity.guid IN ('guid') FACET entity.name TIMESERIES' - Exception: The select clause cannot be used to produce a valid golden metric.
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
|
Please fix the issues |
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
1 similar comment
|
👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP Please review or provide an update. |
Relevant information
This fixes the golden signal metrics for the dockerstats otel
Checklist
identifierwill be unique and valid.