Modify iam resource template for terraform converter

[iyabchen]

Modify iam template to adapt to multiple fields in asset name template.

• Since most of the iam resource have no test cases, so only cloud run service is impacted
• For storage bucket, the url starts with b/{bucket}, hence disable the auto generation and use a handwritten one. The handwritten one is actually the same as the original auto generated file.
• Other handwritten iam resource need similar change too, but will put into another PR.

If this PR is for Terraform, I acknowledge that I have:

• Searched through the issue tracker for an open issue that this either resolves or contributes to, commented on it to claim it, and written "fixes {url}" or "part of {url}" in this PR description. If there were no relevant open issues, I opened one and commented that I would like to work on it (not necessary for very small changes).
• Generated Terraform, and ran make test and make lint to ensure it passes unit and linter tests.
• Ensured that all new fields I added that can be set by a user appear in at least one example (for generated resources) or third_party test (for handwritten resources or update tests).
• Ran relevant acceptance tests (If the acceptance tests do not yet pass or you are unable to run them, please let your reviewer know).
• Read the Release Notes Guide before writing my release note below.

Release Note Template for Downstream PRs (will be copied)

[modular-magician]

Hello! I am a robot who works on Magic Modules PRs.

I have detected that you are a community contributor, so your PR will be assigned to someone with a commit-bit on this repo for initial review.

Thanks for your contribution! A human will be with you soon.

@ndmckinley, please review this PR or find an appropriate assignee.

[modular-magician]

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform Beta: Diff ( 1 file changed, 22 insertions(+), 22 deletions(-))
TF Validator: Diff ( 27 files changed, 137 insertions(+), 125 deletions(-))

[modular-magician]

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

TF Validator: Diff ( 27 files changed, 137 insertions(+), 125 deletions(-))

[modular-magician]

I have triggered VCR tests in RECORDING mode for the following tests that failed during VCR: TestAccApigeeEnvironmentIamBindingGenerated|TestAccApigeeEnvironmentIamPolicyGenerated|TestAccContainerNodePool_withInvalidUpgradeSettings|TestAccOsConfigOsPolicyAssignment_FixedOsPolicyAssignment|TestAccOsConfigOsPolicyAssignment_PercentOsPolicyAssignment You can view the result here: https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=240655

[melinath]

LGTM. It looks like this handles most cases well! There are a couple of edge cases that are currently incorrect, but it might be easier to review fixes for those cases in a separate PR; they could even be considered separate bugs, potentially. This is already a huge improvement over the current state.

• Storage bucket IAM would be great to get generated again - handwritten files are a significant maintenance burden.
• Asset names that include segments with camelCase seem to end up being incorrect - for example, private_ca_pool_iam ends up with an asset name that includes ca_pools (should be caPools) EDIT: This seems to be the only example like this.

[melinath]

I'm going to go ahead and merge this since it is such an improvement over the current state.

[melinath]

b/211764514