v1.4.0

[aidanmelen]

Fixes

• set minimum terraform version to 0.14.3 for anytrue support.
• Managed and Common rules are missing default description #23
• created null_repack_matrix_rules sub-module to simplify matrix.tf

Proposed Changes

• adding unpack variable Prevent Service Interruption during rule updates #22
  • implemented by null_unpack_rules sub-module.
• add variable validation for ingress, egress, matrix_ingress, matrix_egress, computed_ingress,
  computed_egress, computed_matrix_ingress, and computed_matrix_egress.
• added debug variable to better understand the looping local data structures.
• added count checks to terratest.

Service Interruption Warning

There are two scenarios that will cause security group rule replacement during the upgrade:

1. With the unpack enhancement, we now normalize the rules (i.e. looking up managed and common rules values) before creating the map keys. Keys will now have a predefined order. As a consequence, existing ingress, egress, matrix_ingress, and matrix_egress module rules, for versions v1.3.0 and earlier, will be replaced when upgrading to v1.4.0. This doesn't warrant a major release because the all module versions v1.3.0 and earlier constantly suffer from the side-effect caused by packing rule arguments. Therefore, the service interruption is common and accepted. Upgrading will prevent it from happening again after the upgrade.
2. Rules that use managed or common (aliased rules) and do not have a description override will see the rule with the managed by Terraform description be replaced with the new alias rule descriptions. For aliased rules that are overriding the default description will not be replaced.