Add support for EC2 detailed monitoring in nodeGroup

[mmasaki]

Description

This change adds support for EC2 detailed monitoring in node group and managed node group.

Checklist

• Added tests that cover your change (if possible)
• Added/modified documentation as required (such as the README.md, or the userdocs directory)
• Manually tested
• Made sure the title of the PR is a good description that can go into the release notes
• (Core team) Added labels for change area (e.g. area/nodegroup) and kind (e.g. kind/improvement)

BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯

• Backfilled missing tests for code in same general area 🎉
• Refactored something and made the world a better place 🌟

[Callisto13]

👍 Thanks for the PR @mmasaki .

Can we change the field name to detailedMonitoring? Simply monitoring is quite misleading for users, as monitoring: false does not mean that it is disabled, just that it is "basic".

[cPu1]

Thanks for the PR @mmasaki .

Can we change the field name to detailedMonitoring? Simply monitoring is quite misleading for users, as monitoring: false does not mean that it is disabled, just that it is "basic".

I second this. The PR LGTM, however, I think enableDetailedMonitoring would be clearer.

[mmasaki]

@Callisto13 @cPu1
Thanks for your review.
It's a good idea to change the field name.
I changed it to enableDetailedMonitoring.

[cPu1]

LGTM. Thanks for the PR!