provider/aws: Increase AMI destroy timeout

[grubernaut]

Acceptance tests were timing out on AMI destroy, should alleviate the problem.

[grubernaut]

$ make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSAMI'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/03/21 23:30:22 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSAMI -timeout 120m
=== RUN   TestAccAWSAMICopy
--- PASS: TestAccAWSAMICopy (471.51s)
=== RUN   TestAccAWSAMIFromInstance
--- PASS: TestAccAWSAMIFromInstance (111.38s)
=== RUN   TestAccAWSAMILaunchPermission_Basic
--- PASS: TestAccAWSAMILaunchPermission_Basic (400.12s)
=== RUN   TestAccAWSAMI_basic
--- PASS: TestAccAWSAMI_basic (60.72s)
=== RUN   TestAccAWSAMI_snapshotSize
--- PASS: TestAccAWSAMI_snapshotSize (59.70s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    1103.466s

[radeksimko]

I usually wrap these in a function, merely to avoid test failures and confusions caused by reusing such configs and forgetting to pass the integer. Function with argument can enforce this at compilation time, which is nice.

With all that said it's probably a nitpick. :)

[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.