Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fix infra-host definition so it uses the EC2 instanceId as the identifier of the entity #1849

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jsubirat
Copy link
Contributor

@jsubirat jsubirat commented Jan 9, 2025

Relevant information

This PR modifies the synthesis rule of HOST entities only for Log datapoints (feature currently not being used). The identifier of HOST entities needs to be aligned with the identifier used by the Metric Streams integration, which also uses the instanceId as the identifier as opposed to the ARN.

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.

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.

2 participants