Skip to content

[processor/awsentity] Add documentation for entity processor #1741

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

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

Conversation

zhihonl
Copy link
Contributor

@zhihonl zhihonl commented Jun 23, 2025

Description of the issue

Describe the problem or feature in addition to a link to the issues.

Description of changes

How does this change address the problem?

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests

Describe what tests you have done.

Requirements

Before commiting your code, please do the following steps.

  1. Run make fmt and make fmt-sh
  2. Run make lint

Integration Tests

To run integration tests against this PR, add the ready for testing label.

@zhihonl zhihonl requested a review from a team as a code owner June 23, 2025 16:34
[amazon-cloudwatch-agent]: https://github.com/aws/amazon-cloudwatch-agent
<!-- end autogenerated section -->

The AWS Entity processor enriches telemetry data with AWS-specific entity information for improved observability and correlation in Amazon CloudWatch. It processes metrics to add entity attributes that help identify and categorize AWS resources and services. Please refer to [config.go](./config.go) for the config spec.
Copy link
Contributor

Choose a reason for hiding this comment

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

it would be good to include public documentation on what an entity is and what it can be used for: https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/API_Entity.html

Also, if we just add the processor -- it doesn't work natively to the cloudwatch exporter. You would need the exporters to parse these attributes to plug into the API. Worth calling out

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