Skip to content

feat(stg): Add otel kafka broker to conatiner rels#2738

Merged
apoloa merged 5 commits intonewrelic:mainfrom
shashank-reddy-nr:otel-ka-broker-to-contain
Apr 9, 2026
Merged

feat(stg): Add otel kafka broker to conatiner rels#2738
apoloa merged 5 commits intonewrelic:mainfrom
shashank-reddy-nr:otel-ka-broker-to-contain

Conversation

@shashank-reddy-nr
Copy link
Copy Markdown
Contributor

@shashank-reddy-nr shashank-reddy-nr commented Apr 6, 2026

Changes

  • Adds 4 new OTel relationship rules to INFRA-CONTAINER-to-INFRA-KAFKABROKER.stg.yml to link OTel-instrumented Kafka brokers to their host containers.
    relationships/synthesis/INFRA-CONTAINER-to-INFRA-KAFKABROKER.stg.yml

Why

Previously, the only container → broker relationship was for NR OnHost Integration (KafkaBrokerSample). OTel Kafka brokers had no container relationship, leaving a gap in the entity topology for OTel users.

Rule Source container Kafka broker target
OtelKafkaBrokerToOtelDockerContainer OTel container (container.id) extractGuid: entity.guid
OtelKafkaBrokerToNrDockerContainer NR native ContainerSample (docker:{container.id}) extractGuid: entity.guid
OtelKafkaBrokerToOtelK8sContainer OTel k8s container (cluster:namespace:pod:container) extractGuid: entity.guid
OtelKafkaBrokerToNrK8sContainer NR native K8sContainerSample (k8s:{cluster}:…) extractGuid: entity.guid

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-546406

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

Copy link
Copy Markdown
Collaborator

@entityBot entityBot left a 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 PARSING domainType: 'INFRA-CONTAINER-to-INFRA-KAFKABROKER' on entitydefinitionemitter.models.relationshipsynthesis.ListOfRelationshipSynthesisDefinitionPublic - jackson.databind.exc.ValueInstantiationException: Cannot construct instance of entitydefinitionsclientrelationshipsynthesis.models.definitions.Condition, problem: At least one of the operators must be defined
    at [Source: (StringReader); line: 60, column: 7] (through reference chain: entitydefinitionemitter.models.relationshipsynthesis.ListOfRelationshipSynthesisDefinitionPublic["relationships"]->ArrayList[1]->entitydefinitionemitter.models.relationshipsynthesis.RelationshipSynthesisDefinitionPublic["conditions"]->ArrayList[1])
  • ARB tickets not approved: NR-546406 (Status: Backlog).
    The review process is not handled by Entity Platform, reach #api-review-board channel in case of doubts

@entityBot entityBot requested a review from a team April 6, 2026 10:17
@shashank-reddy-nr shashank-reddy-nr marked this pull request as draft April 6, 2026 10:22
@entityBot entityBot dismissed their stale review April 6, 2026 10:22

Let's validate those changes.

Copy link
Copy Markdown
Collaborator

@entityBot entityBot left a 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:

  • ARB tickets not approved: NR-546406 (Status: Backlog).
    The review process is not handled by Entity Platform, reach #api-review-board channel in case of doubts

@shashank-reddy-nr shashank-reddy-nr changed the title fix: Add otel kafka broker to conatiner rels feat(stg): Add otel kafka broker to conatiner rels Apr 6, 2026
@shashank-reddy-nr shashank-reddy-nr marked this pull request as ready for review April 6, 2026 11:04
@entityBot entityBot dismissed their stale review April 6, 2026 11:04

Let's validate those changes.

Copy link
Copy Markdown
Collaborator

@entityBot entityBot left a 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:

  • ARB tickets not approved: NR-546406 (Status: Backlog).
    The review process is not handled by Entity Platform, reach #api-review-board channel in case of doubts

@entityBot entityBot dismissed their stale review April 6, 2026 13:03

Let's validate those changes.

Copy link
Copy Markdown
Collaborator

@entityBot entityBot left a 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:

  • ARB tickets not approved: NR-546406 (Status: Backlog).
    The review process is not handled by Entity Platform, reach #api-review-board channel in case of doubts

Copy link
Copy Markdown
Contributor

@vagrawal-newrelic vagrawal-newrelic left a comment

Choose a reason for hiding this comment

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

LGTM

@entityBot entityBot dismissed vagrawal-newrelic’s stale review April 9, 2026 09:13

Dismissing reviews

@apoloa apoloa merged commit a9b3d97 into newrelic:main Apr 9, 2026
3 checks passed
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

Successfully merging this pull request may close these issues.

5 participants