Skip to content

feat : Add Consul OpenTelemetry production definition support#3058

Open
mbolla22 wants to merge 1 commit into
newrelic:mainfrom
mbolla22:feat/consul-changes
Open

feat : Add Consul OpenTelemetry production definition support#3058
mbolla22 wants to merge 1 commit into
newrelic:mainfrom
mbolla22:feat/consul-changes

Conversation

@mbolla22

Copy link
Copy Markdown
Contributor

Enable OpenTelemetry synthesis and dashboard template for the Consul production definition, and remove the redundant OTel dashboard coverage widget.

Relevant information

Api Review Board (ARB)

Any pull request with changes to this repository has to be linked with an ARB ticket, which has to be approved before merging.

If you are an external contributor please contact New Relic Support.

If you don't know about the ARB process check this

ARB Jira ticket:
https://new-relic.atlassian.net/browse/NR-581271

Test

Screenshot 2026-07-20 at 3 50 35 PM

Checklist

  • I've read the guidelines and understand the acceptance criteria.
  • The value of the attribute marked as identifier will be unique and valid.
  • I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.
  • I've linked an ARB ticket & received approval from API Review Board in order to make these changes

Enable OpenTelemetry synthesis and dashboard template for the Consul production definition, and remove the redundant OTel dashboard coverage widget.
@entityBot
entityBot requested a review from a team July 20, 2026 10:23
@mbolla22
mbolla22 marked this pull request as ready for review July 20, 2026 10:24
@mbolla22 mbolla22 changed the title Add Consul OpenTelemetry production definition support feat : Add Consul OpenTelemetry production definition support Jul 20, 2026

@sairaj18 sairaj18 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for working on this changes. Can you add telemetry tests for new synthesis rule that is being added as part of this PR?

identifier: service.instance.id
name: service.instance.id
encodeIdentifierInGUID: true
conditions:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why are we not using otel.library.name for filtering the telemetry?

- attribute: otel.library.name
  value: "otelcol/prometheusreceiver"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants