Add support for CloudWatch metric streams through firehose endpoint#6380
Add support for CloudWatch metric streams through firehose endpoint#6380kaiyan-sheng merged 23 commits intoelastic:masterfrom
Conversation
|
This pull request does not have a backport label. Could you fix it @kaiyan-sheng? 🙏
NOTE: |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
|
BTW: I think the dataset value you're setting looks appropriate. I'd suggest we copy that to |
axw
left a comment
There was a problem hiding this comment.
Looks good, thanks for adding summary metric support! Can you please extend this test to include a summary metric type?
apm-server/model/metricset_test.go
Lines 73 to 115 in 411206b
Also, the ingest pipeline still needs to be updated:
|
Thanks for the review @axw ! I see we also have histogram in ingest pipeline here: https://github.com/elastic/apm-server/blob/master/ingest/pipeline/definition.yml#L130. Do we need to add |
axw
left a comment
There was a problem hiding this comment.
Thanks for the review @axw ! I see we also have histogram in ingest pipeline here: https://github.com/elastic/apm-server/blob/master/ingest/pipeline/definition.yml#L130. Do we need to add summary also here?
No. That file will be deleted soon, and we'll rely 100% on Fleet for managing the ingest pipeline.
apmpackage/apm/data_stream/app_metrics/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
…6380) * Process CloudWatch metric streams using firehose endpoint * Add data_stream.dataset separately for logs and metrics * remove unit from metricset * Add summray metric type * update unit test for cloudwatch metric * update go.sum * make update * add test for summary metric type * fix manifest.yml * fix manifest.yml * Fix comments in summary fields * remove summary metric type * undo some changes * add changelog * remove extra line (cherry picked from commit 85be4e0)
…6380) (#6448) * Process CloudWatch metric streams using firehose endpoint * Add data_stream.dataset separately for logs and metrics * remove unit from metricset * Add summray metric type * update unit test for cloudwatch metric * update go.sum * make update * add test for summary metric type * fix manifest.yml * fix manifest.yml * Fix comments in summary fields * remove summary metric type * undo some changes * add changelog * remove extra line (cherry picked from commit 85be4e0) Co-authored-by: kaiyan-sheng <kaiyan.sheng@elastic.co>
Motivation/summary
This PR is to add support for CloudWatch metric streams through Firehose using the same
/firehoseendpoint.Checklist
For functional changes, consider:
How to test these changes
This can be tested either hosted in an EC2 instance or locally. Here is how to test it locally:
makelocalhost:8200/firehosewith body:Also make sure the API key is passed into the request using
X-Amz-Firehose-Access-Keyheader.5. You should be able to see a 200 response code and an EC2 CPU utilization metric posted into Elasticsearch.
Related issues
closes elastic/integrations#956
Sample Output Event