fix: Switch to name_prefix for iam roles

terraform-aws-modules / terraform-aws-dms

Terraform module to create AWS DMS (Database Migration Service) resources 🇺🇦

https://registry.terraform.io/modules/terraform-aws-modules/dms/aws

Apache License 2.0

60 stars 93 forks source link

fix: Switch to name_prefix for iam roles #39

Closed Alexander-Babichuk closed 1 year ago

Alexander-Babichuk commented 1 year ago

A hotfix that prevents IAM role already exists error.

Description

changed name to name_prefix for IAM roles

Motivation and Context

This change was made to prevent 'IAM role already exists' error.

Breaking Changes

It shouldn't break backward compatibility with the current major version.

How Has This Been Tested?

The changes were tested with two DMS instances.

bryantbiggs commented 1 year ago

please read the docs - this is not possible

github-actions[bot] commented 1 year ago

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.