You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In aws module we have several metricsets that are not lightweight modules. This issue is to track the work for converting them into lightweight modules. There are some problems we are facing:
Some metricsets are not only calling CloudWatch API. For example, billing metricset is also making CostExplorer API and ec2 metricset is also making DescribeInstances API.
How to keep backwards compatibility with metric names. For example, after using lightweight module, aws.ec2.cpu.total.pct will be aws.ec2. CPUUtilization.avg. @jsoriano suggested that fields can be renamed using processors. This will be a good option to keep both names instead of making a breaking change.
This is a meta issue to track the work for migrating all listed metricsets above to use lightweight modules.
Update: For ec2, rds and sqs, we are adding metadata from separate API calls into the events that are populated from CloudWatch API response. billing metricset in aws module is not the same as other metricsets because it is not adding metadata into the same events. Instead, billing is creating two sets of events: from CloudWatch and from Cost Explorer.
I will keep billing metricset as it is for now. Maybe we should think about separating billing into its separate module in the future.
In
aws
module we have several metricsets that are not lightweight modules. This issue is to track the work for converting them into lightweight modules. There are some problems we are facing:* billing: also uses CostExplorer APIaws.ec2.cpu.total.pct
will beaws.ec2. CPUUtilization.avg
. @jsoriano suggested that fields can be renamed using processors. This will be a good option to keep both names instead of making a breaking change.This is a meta issue to track the work for migrating all listed metricsets above to use lightweight modules.
cc: @exekias @jsoriano @sorantis
The text was updated successfully, but these errors were encountered: