Skip to content

fix(otel): restore normal Kafka relationships on staging#2595

Merged
apoloa merged 2 commits into
newrelic:mainfrom
shashank-reddy-nr:fix-otel-kafka-broker-host
Jan 21, 2026
Merged

fix(otel): restore normal Kafka relationships on staging#2595
apoloa merged 2 commits into
newrelic:mainfrom
shashank-reddy-nr:fix-otel-kafka-broker-host

Conversation

@shashank-reddy-nr

@shashank-reddy-nr shashank-reddy-nr commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

Changes:

  • Moved back to normal synthetic relationships from candidate rels for OTel Kafka.

Relevant information

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.

@entityBot entityBot left a comment

Copy link
Copy Markdown
Collaborator

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-HOST-to-INFRA-KAFKABROKER' on entitydefinitionemitter.models.relationshipsynthesis.ListOfRelationshipSynthesisDefinitionPublic - jackson.module.kotlin.KotlinInvalidNullException: Instantiation of [simple type, class entitydefinitionemitter.models.relationshipsynthesis.LookupGuidResolverPublic] value failed for JSON property candidateCategory due to missing (therefore NULL) value for creator parameter candidateCategory which is a non-nullable type
at [Source: (StringReader); line: 266, column: 7] (through reference chain: entitydefinitionemitter.models.relationshipsynthesis.ListOfRelationshipSynthesisDefinitionPublic["relationships"]->ArrayList[6]->entitydefinitionemitter.models.relationshipsynthesis.RelationshipSynthesisDefinitionPublic["relationship"]->entitydefinitionemitter.models.relationshipsynthesis.RelationshipPublic["source"]->entitydefinitionemitter.models.relationshipsynthesis.ResolverPublic["lookupGuid"]->entitydefinitionemitter.models.relationshipsynthesis.LookupGuidResolverPublic["candidateCategory"])

@entityBot entityBot requested a review from a team January 21, 2026 11:48

@vagrawal-newrelic vagrawal-newrelic 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.

LGTM

entityType:
value: KAFKABROKER

# Local scenario: Kafka broker running on the same host with OpenTelemetry

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remove this, as this is duplication of above rules.

entityType:
value: KAFKABROKER

# Host measures Kafka broker with OpenTelemetry when broker.endpoint is not present

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remove this, as this is duplication of above rules.

entityType:
value: KAFKABROKER

# Remote scenario: Host monitors remote Kafka broker with OpenTelemetry when broker.endpoint is present

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remove this, as this is duplication of above rules.

@NSSPKrishna NSSPKrishna left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@apoloa apoloa merged commit 215e0b8 into newrelic:main Jan 21, 2026
4 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