Invalid Parameter for Block device with Snapshot ID -- Terraform v0.5.0

[randylyon]

Receiving this error when using Terraform 0.5.0 with a block device specifying a snapshot. I get this error if I add encryption = false or don't have encryption defined at all.

• Error launching source instance: InvalidParameter: Parameter encrypted is invalid. You cannot specify the encrypted flag if specifying a snapshot id in a block device mapping.

Instance defined as:

resource "aws_instance" "test" {
    ami = "${var.instance_ami}"
    instance_type = "t2.medium"
    key_name = "REDACTED"
    availability_zone = "us-east-1a"
    user_data = "${file(\"userData.txt\")}"
    tags {
        Name = "test"
    }
    associate_public_ip_address = true
    security_groups = ["Web-Application-Default"]
    ebs_block_device {
      device_name  = "/dev/sdf"
      volume_size = 10
      volume_type = "gp2"
      delete_on_termination = true
      snapshot_id = "REDACTED"
    }
    root_block_device {
      delete_on_termination = true
      volume_size = 20
      volume_type = "gp2"
    }
}

[phinze]

Thanks for the report!

You'd think that the API would be cool with encrypted: false wouldn't you? But nope, it needs nil. 😐

We'll get this fixed up.

[egarbi]

+1 for this one, it happened me today when I was trying to build a new "graphite" instance.

[amaczuga]

+1, occurs to me too. worked around by continuing apply with v0.4.2... (a disgusting way to do things)

[wkielas]

+1 from me, this effectively prevents from modifying root EBS volumes on instances

[netors]

+1 this happens to me if I try to use a snapshot of an encrypted volume

[sathiyas]

+1 from me, this effectively prevents from modifying root EBS volumes on instances

[catsby]

Patch for this in #2120

[catsby]

If anyone can build master and try #2120 out that would help me merge with confidence 😄

I took @randylyon config and was able to get through as expected, but other validation is always welcome

[armen]

I was going to send a patch for this and interestingly my patch is exactly same as yours, so I do confirm this works.

[catsby]

Closed by merging #2120 , let me know if this comes up again

[ghost]

I'm going to lock this issue 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 similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.