Provider update 3.3.0

[bharathkkb]

Fixes #338

Change List for 3.3.0 upgrade:

• Remove internal references in locals to google_container_cluster.primary.region
• Default cluster_ipv4_cidr to null. cluster_ipv4_cidr is used for routes-based clusters and ip_allocation_policy is used for VPC-native clusters
• ${random_string.suffix.result} where networking 2.0 is used due to inability to use computed values with the new for_each implementation in subnets module

[morgante]

@bharathkkb What's the latest on this? Do you need anything to complete it?

[bharathkkb]

@morgante working on this currently. 3.3.0 was released today so I am making sure int tests are passing.

[bharathkkb]

@morgante actually we havent released the network module with the commits for relaxing the version constraint. If you could release it that would be awesome.

[morgante]

@bharathkkb Released - https://github.com/terraform-google-modules/terraform-google-network/blob/master/CHANGELOG.md#201---2019-12-18

[bharathkkb]

@morgante @aaron-lane this should be ready for review

[aaron-lane]

@bharathkkb can you add an upgrade guide since this branch includes breaking changes?

[aaron-lane]

Also, can the fix for #384 be split to a separate pull request so we can get it merged quickly and unblock other pull requests?

[bharathkkb]

@aaron-lane sg I will separate the PRs and add upgrade guide

[bharathkkb]

@aaron-lane @morgante I dont think there are any breaking changes here as I am now defaulting cluster_ipv4_cidr to null. Previously I had mistakenly removed this but cluster_ipv4_cidr is still used for routes-based clusters.

Old