fix: Update tags for default resources to correct spurious plan diffs

[bryantbiggs]

Description

• default_xxx_name default values changed to null; still works the same due to use of coalesce(...)
• Name tags for default resources changed to use coalesce(var.default_xxx_name, "${var.name}-default") except for default_vpc_name because its not tied to this module (aka - created by this module)
• Add default_route_table_name to match other default resource conventions (tag)
• Change default_security_group_ingress and default_security_group_egress values to []; using null will fail - updated and tested in simple-vpc - does not affect users because in order for this to work for them they will have had to set the rules so its a no-op for users. This also matches the default route table routes values as well
• Clean up usage of format(...)
• Fix complete example to correct generic endpoint policy to not use DynamoDB endpoint but VPC CIDR and for DynamoDB endpoint to use correct data source to get the VPC endpoint ID; closes DynamoDB VPC Endpoint Policy in the complete example denies all DynamoDB requests #680
• Update example outputs to include full breadth of outputs to aid in testing - more broader coverage of any potential changes
• Update local variable vpc_id to use try(); replace nearly all instances of aws_vpc.this[0].id with local.vpc_id except for the CIDR block association
• Clean up local.nat_gateway_ips hack now that issue is resolved and we are requiring min version of 0.13.1

Motivation and Context

• Closes default_security_group_name is not set on managed default security group (only set as tag 'Name') #541
• Closes fix: remove incorrect logic from complete example #671
• Closes DynamoDB VPC Endpoint Policy in the complete example denies all DynamoDB requests #680
• Closes manage_default_network_acl flag when set to true always prompts for plan changes even if there are none #684

Breaking Changes

• No; these are all backwards compatible

How Has This Been Tested?

• I have tested and validated these changes using one or more of the provided examples/* projects
  • Tested with complete-vpc and simple-vpc

[bryantbiggs]

one of these days I'll get the title right on the first go 😅

[bryantbiggs]

should be ready for review now

[antonbabenko]

LGTM. Now, when we don't have format("%s", ...) let's hope that nobody calls their VPC true which will be interpreted as a boolean in tags :)

[bryantbiggs]

@antonbabenko should be all set now if you get some time to take another 👀

[antonbabenko]

This PR is included in version 3.11.3 🎉

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.