Skip to content

Conversation

@ptodev
Copy link
Contributor

@ptodev ptodev commented May 8, 2024

PR Description

In OTel collector v0.99.0, there was a change to how metrics are displayed.

Since Alloy v1.0.0 shows all grpc-level metrics, we need to use the "detailed" logging level by default for backwards compatibility.

Which issue(s) this PR fixes

Fixes #776

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

@ptodev ptodev marked this pull request as ready for review May 8, 2024 10:09
@ptodev ptodev requested a review from clayton-cornell as a code owner May 8, 2024 10:09
@ptodev
Copy link
Contributor Author

ptodev commented May 8, 2024

I opened #792 for this to be made more configurable.

@ptodev
Copy link
Contributor Author

ptodev commented May 8, 2024

I tested this change locally using the config file in our docs. I see two extra batch processor metrics than what I saw with v1.0.0, but all those metrics have been around for a while and I think I can assume that they must have been in v1.0.0. This is why I'm not adding the metrics which I'm documenting now to the changelog. I'm not sure why I don't see them locally though.

@ptodev ptodev requested a review from rfratto May 8, 2024 10:37
Copy link
Member

@rfratto rfratto left a comment

Choose a reason for hiding this comment

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

Thanks!

@rfratto rfratto added backport-to-agent PR should be backported to the agent repo. backport release/v1.1 labels May 8, 2024
@ptodev ptodev merged commit bcb1d29 into main May 8, 2024
@ptodev ptodev deleted the ptodev/set-otel-tel-level branch May 8, 2024 11:19
github-actions bot pushed a commit that referenced this pull request May 8, 2024
ptodev added a commit that referenced this pull request May 8, 2024
(cherry picked from commit bcb1d29)

Co-authored-by: Paulin Todev <[email protected]>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport release/v1.1 backport-to-agent PR should be backported to the agent repo. frozen-due-to-age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Alloy / OpenTelemetry" dashboard has unexpected blank panels

2 participants