使用for_each创建EC2实例时遇Terraform子网ID错误求助

错误原因 #

• subnet_id字段写法错误：你把each.value.id用引号包裹，Terraform会把它当作字面量字符串"each.value.id"而非循环变量引用。而且toset(...)的每个元素本身就是子网ID字符串，根本不需要加.id后缀。
• 冗余的数据源依赖：你已经通过aws_subnet.public资源创建了子网，直接引用该资源的输出比用data.aws_subnets更可靠，还能避免额外的查询开销和同步问题。

修正后的代码 #

方案1：直接关联子网资源ID（推荐） #

利用子网资源的ids属性作为for_each的数据源，直接绑定每个子网ID：

resource "aws_instance" "EC2" {
  ami                    = data.aws_ami.amazon_linux.id
  instance_type          = var.instance_type
  key_name               = var.key
  # 直接用子网资源生成的ID集合
  for_each               = toset(aws_subnet.public.ids)
  # each.value就是子网ID，直接赋值
  subnet_id              = each.value
  vpc_security_group_ids = [aws_security_group.ssh.id]
}

方案2：保持count模式同步创建 #

如果想保留子网的count写法，EC2实例也可以用count和子网索引一一对应：

resource "aws_instance" "EC2" {
  count                  = var.subnet_count
  ami                    = data.aws_ami.amazon_linux.id
  instance_type          = var.instance_type
  key_name               = var.key
  # 通过索引关联对应子网的ID
  subnet_id              = aws_subnet.public[count.index].id
  vpc_security_group_ids = [aws_security_group.ssh.id]
}

额外提示 #

• 优先用for_each替代count：当资源需要和外部标识符（比如子网ID）绑定，for_each更稳定，能避免因资源顺序变化导致的不必要重建。
• 资源/变量引用别加引号：Terraform里引用变量或资源属性时不需要引号，加了引号会被解析成普通字符串，这是新手容易踩的坑。

内容的提问来源于stack exchange，提问作者Surfingjoe