Firehose metrics add subsystem, create override_default_tags

[ryantanjunming]

Description

• Added subsystem value to common attributes of firehose metrics
• Added override_default_tags to allow users to override the default tags we set

How Has This Been Tested?

Local testing with our AWS account

Checklist:

• I have updated the relevant example in the examples directory for the module.
• I have updated the relevant test file for the module.
• This change does not affect module (e.g. it's readme file change)

[macmiranda]

Hey @ryantanjunming just an idea, perhaps var.override_default_tags should be a boolean. var.user_supplied_tags remains the variable for user input. The former just changes the behavior of the merge. If true, the user-supplied tags come later in the merge, if false the other way around (which is the default).

Having two maps for user-supplied tags will be confusing.

[ryantanjunming]

@macmiranda i'll set it as var.use_default_tags boolean default to true. If false, it will remove the default tags and leave just the var.user_supplied_tags.

So it's possible to have it empty also.

[github-actions]

🎉 This PR is included in version 1.0.58 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀