Skip to content

Commit

Permalink
Merge pull request #50 from aws-ia/ram-share-fix
Browse files Browse the repository at this point in the history
fix regression in ram share pool names
  • Loading branch information
drewmullen authored Jan 24, 2023
2 parents 3e84a3c + 0bcfa23 commit b7ef5c7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion modules/sub_pool/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,8 @@ resource "aws_vpc_ipam_pool_cidr" "sub" {
resource "aws_ram_resource_share" "sub" {
count = local.ram_share_enabled ? 1 : 0

name = local.description
# if a user specifies a var.pool.description must validate there is no / which is invalid for RAM names
name = replace(local.description, "/", "-")

tags = local.tags
}
Expand Down

0 comments on commit b7ef5c7

Please sign in to comment.