-
Notifications
You must be signed in to change notification settings - Fork 448
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
[AWS Cloudwatch] Add metric_type #6999
Changes from 5 commits
21ce50f
1b9bc15
2de4334
015eda2
335d9b9
310b7cb
9bc5618
944a6aa
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,15 +1,15 @@ | ||
format_version: 1.0.0 | ||
name: aws | ||
title: AWS | ||
version: 1.47.0 | ||
version: 1.47.1 | ||
license: basic | ||
description: Collect logs and metrics from Amazon Web Services (AWS) with Elastic Agent. | ||
type: integration | ||
categories: | ||
- aws | ||
release: ga | ||
conditions: | ||
kibana.version: "^8.8.1" | ||
kibana.version: "^8.9.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry I probably missed something, why do we need to update the kibana version here? If the upgrade is required, then we have to wait till 8.9.0 is released before merging this PR then. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've reverted this change for this PR - It will be added with the TSDB enablement in next PR. |
||
screenshots: | ||
- src: /img/metricbeat-aws-overview.png | ||
title: metricbeat aws overview | ||
|
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.
@kaiyan-sheng could you please help here? usually
type: object
is used in combination withobject_type
, but here it is skipped, is it intentional?Is it fine to define it as a
double
?I've also checked beats - https://github.com/elastic/beats/blob/main/x-pack/metricbeat/module/aws/cloudwatch/_meta/fields.yml and can't find this field there
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.
In beats, it's defined under
aws
: https://github.com/elastic/beats/blob/main/x-pack/metricbeat/module/aws/_meta/fields.yml#L27You are right, we are missing the object type here, it should be something like:
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.
thanks, I didn't look into this file. So it actually should be correct to set it to
double
, before this change - the object type was defined dynamically.