-
Notifications
You must be signed in to change notification settings - Fork 30
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
Adding integ test for otel yaml merging #424
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you link the successful integration test run?
00d02f7
to
2aec239
Compare
nit: https://github.com/aws/amazon-cloudwatch-agent/actions/runs/11404848694/job/31735108673 Based on the test run, I see this test is by itself under |
Description of the issue
This pr adds otel yaml merging integ test. We want to make sure that no regression happens in the agent where the yaml merging for the agent stops working. Passing test run: https://github.com/aws/amazon-cloudwatch-agent/actions/runs/11404848694/job/31735108673
Passing test
s/11353738985/job/31579557585Amazon-cloudwatch-agent repo pr: aws/amazon-cloudwatch-agent#1391
Description of changes
Test workflow
sudo /opt/aws/amazon-cloudwatch-agent/bin/amazon-cloudwatch-agent-ctl -a append-config -s -m ec2 -c file:./resources/otel.yaml
curl -X POST -H "Content-Type: application/json" -d @metrics.json -i localhost:4318/v1/metrics
where otlp reciever listens to this and sends it over to cloudwatch using the awsemf exporter.
Agent json configuration
Agent Yaml:
Otel Yaml
Merged OTEL configuration:
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
Manually tested this on ec2 instance by running these command with the above configurations: