Skip to content

feat: Adds otel docker container to host relationships#2582

Merged
vlopeziglesias merged 8 commits into
newrelic:mainfrom
NSSPKrishna:docker-otel-rel-logs
Feb 17, 2026
Merged

feat: Adds otel docker container to host relationships#2582
vlopeziglesias merged 8 commits into
newrelic:mainfrom
NSSPKrishna:docker-otel-rel-logs

Conversation

@NSSPKrishna

@NSSPKrishna NSSPKrishna commented Jan 16, 2026

Copy link
Copy Markdown
Member

Relevant information

Attaches the logs to the OTel docker container entities and adds the relationship of the OTel container and host

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-container' on entitydefinitionemitter.models.BaseEntityDefinition - jackson.databind.JsonMappingException: while parsing a block mapping
in 'reader', line 5, column 3:
     rules:
     ^
expected <block end>, but found '<block sequence start>'
in 'reader', line 75, column 4:
      - ruleName: infra_container_cont ... 
      ^

at [Source: (StringReader); line: 75, column: 4] (through reference chain: entitydefinitionemitter.models.BaseEntityDefinition["synthesis"])

@entityBot entityBot requested a review from a team January 16, 2026 13:02

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

Looks good!!

entityBot
entityBot previously approved these changes Jan 19, 2026
github-actions[bot]
github-actions Bot previously approved these changes Jan 19, 2026
@bgalmesnr

Copy link
Copy Markdown
Contributor

Hi @NSSPKrishna ! if your team is the owner of the entity you are modifying we would appreciate if you can add your team as the owner. You can find further information here. Thank you!

@NSSPKrishna NSSPKrishna changed the title feat: Adds otel docker logs and host container relations [DO NOT MERGE]feat: Adds otel docker logs and host container relations Jan 19, 2026
github-actions[bot]
github-actions Bot previously approved these changes Jan 20, 2026
@NSSPKrishna NSSPKrishna marked this pull request as draft January 20, 2026 07:30
@NSSPKrishna NSSPKrishna marked this pull request as ready for review February 6, 2026 15:46
entityBot
entityBot previously approved these changes Feb 6, 2026
entityBot
entityBot previously approved these changes Feb 9, 2026

@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

@NSSPKrishna NSSPKrishna changed the title [DO NOT MERGE]feat: Adds otel docker logs and host container relations feat: Adds otel docker logs and host container relations Feb 9, 2026
entityBot
entityBot previously approved these changes Feb 12, 2026
@NSSPKrishna NSSPKrishna changed the title feat: Adds otel docker logs and host container relations feat: Adds otel docker container to host relationships Feb 12, 2026
@entityBot

Copy link
Copy Markdown
Collaborator

⚠️ Review Required ⚠️

👋 This PR has seen no activity in over 3 days and is still awaiting approval from: @newrelic/EP

Please review or provide an update.

mborroni
mborroni previously approved these changes Feb 16, 2026
@vlopeziglesias

vlopeziglesias commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

Currently there are conflicts @NSSPKrishna, can you take a look and fix them?

@NSSPKrishna NSSPKrishna dismissed stale reviews from mborroni and entityBot via d872163 February 17, 2026 04:38
@entityBot entityBot requested a review from a team February 17, 2026 04:38
@vlopeziglesias vlopeziglesias merged commit 60f2ac1 into newrelic:main Feb 17, 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.

8 participants