[PR #2286/118905b8 backport][stable-8] cloudwatchlogs_log_group_metric_filter: add support for unit and dimensions #2361
Conversation
![patchback[bot]](https://avatars.githubusercontent.com/in/21488?s=60&v=4){kind=small}
…nsions (#2286) SUMMARY This PR adds support for the options unit and dimensions in the cloudwatchlogs_log_group_metric_filter module. This enables configuring unit and dimensions in Cloudwatch Logs Metricfilters using ansible, which was previously not possible. The addition is pretty straigthforward since both unit and dimensions are part of the metric_transformation parameter. dimensions and default_value are mutually exclusive, however: The AWS API does not fail when both are present, instead the dimensions are simply ignored. Since they are not top-level parameters, the ansible module option mutually_exclusive was not possible. Instead a custom check was added to the module that throws an error when both parameters are present. An integration test has been added for this case, as well as for configuring metric_filters with units and/or dimensions. The function metricTransformationHandler has been rewritten slightly due to the addition of the two extra optional parameters, to make it a bit more readable. Happy to get your feedback! ISSUE TYPE Feature Pull Request COMPONENT NAME cloudwatchlogs_log_group_metric_filter ADDITIONAL INFORMATION Reviewed-by: GomathiselviS <[email protected]> Reviewed-by: Jasper Misset Reviewed-by: Alina Buzachis (cherry picked from commit 118905b)
Docs Build 📝Thank you for contribution!✨ This PR has been merged and your docs changes will be incorporated when they are next published. |
|
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 4m 38s |
|
Build succeeded (gate pipeline). ✔️ ansible-galaxy-importer SUCCESS in 4m 10s |
softwarefactory-project-zuul
bot
deleted the
patchback/backports/stable-8/118905b8004526bfc8289ba41e0ac3d053d347c6/pr-2286
branch
This is a backport of PR #2286 as merged into main (118905b).
SUMMARY
This PR adds support for the options
unitanddimensionsin thecloudwatchlogs_log_group_metric_filtermodule.This enables configuring
unitanddimensionsin Cloudwatch Logs Metricfilters using ansible, which was previously not possible.The addition is pretty straigthforward since both
unitanddimensionsare part of themetric_transformationparameter.dimensionsanddefault_valueare mutually exclusive, however:mutually_exclusivewas not possible. Instead a custom check was added to the module that throws an error when both parameters are present.An integration test has been added for this case, as well as for configuring metric_filters with units and/or dimensions.
The function metricTransformationHandler has been rewritten slightly due to the addition of the two extra optional parameters, to make it a bit more readable.
Happy to get your feedback!
ISSUE TYPE
COMPONENT NAME
cloudwatchlogs_log_group_metric_filter
ADDITIONAL INFORMATION