Skip to content

Update ownership for uninstrumented entity definitions#2699

Open
bgalmesnr wants to merge 1 commit intomainfrom
add-ownership-for-uninstrumented-definitions
Open

Update ownership for uninstrumented entity definitions#2699
bgalmesnr wants to merge 1 commit intomainfrom
add-ownership-for-uninstrumented-definitions

Conversation

@bgalmesnr
Copy link
Copy Markdown
Contributor

Relevant information

Updates entity ownership for UNINSTRUMENTED definitions that had no owner assigned. It matches the owner on the regular entity types.

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 previously approved these changes Mar 16, 2026
@entityBot entityBot requested review from a team March 16, 2026 10:02
naxhh
naxhh previously approved these changes Mar 16, 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/BEYOND, @newrelic/OHAI, @newrelic/K8S-AGENTS

Please review or provide an update.

nr-ksteinbach
nr-ksteinbach previously approved these changes Mar 21, 2026
Copy link
Copy Markdown
Contributor

@nr-ksteinbach nr-ksteinbach 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 stale reviews from naxhh, nr-ksteinbach, and themself March 23, 2026 08:10

Dismissing reviews

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 VALIDATING domainType: 'INFRA-OCIAUTONOMOUSDATABASE' on synthesis - Exception: No entity matches 'INFRA-OCIAUTONOMOUSDATABASE' synthesis rules for the given test data: 'MetricRaw.json file - #1' data point. Please review test data and synthesis rules.

@entityBot entityBot requested a review from a team March 23, 2026 08:10
@bgalmesnr bgalmesnr force-pushed the add-ownership-for-uninstrumented-definitions branch from 27813f3 to c82709c Compare March 23, 2026 13:48
entityBot
entityBot previously approved these changes Mar 23, 2026
@entityBot entityBot requested a review from a team March 23, 2026 13:48
@bgalmesnr bgalmesnr force-pushed the add-ownership-for-uninstrumented-definitions branch from c82709c to d1fd4ab Compare March 25, 2026 13:59
@entityBot entityBot removed the request for review from a team March 25, 2026 13:59
@bgalmesnr bgalmesnr marked this pull request as draft March 25, 2026 14:02
@bgalmesnr bgalmesnr marked this pull request as ready for review March 25, 2026 14:02
ownership:
primaryOwner:
teamName: "no owner"
teamName: "K8s Agents"
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.

I'm not sure I understand the point of this entity, or how we can own a definition that's not coming from our agents.

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.

This Uninstrumented Entity is created when a Candidate Relationship can not be resolved as per KUBERNETESCLUSTER candidate definition.

Our proposal is assigning the Uninstrumented Entities to the owners of their "regular Entity" counterpart.
In this case, your team owns INFRA-KUBERNETESCLUSTER

@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/BEYOND, @newrelic/OHAI, @newrelic/K8S-AGENTS

Please review or provide an update.

@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/BEYOND, @newrelic/OHAI

Please review or provide an update.

1 similar comment
@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/BEYOND, @newrelic/OHAI

Please review or provide an update.

Copy link
Copy Markdown
Contributor

@nr-ksteinbach nr-ksteinbach left a comment

Choose a reason for hiding this comment

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

LGTM for OHAI.

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.

7 participants