From 345dbce77afd80fc8cab1d05df7d0a0358d7386d Mon Sep 17 00:00:00 2001 From: Srikalyan Swayampakula Date: Sat, 13 Feb 2016 12:15:29 -0800 Subject: [PATCH] Made the necessary changes to ensure the variable name represents right value. --- .../providers/aws/resource_aws_sns_topic_subscription.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/providers/aws/resource_aws_sns_topic_subscription.go b/builtin/providers/aws/resource_aws_sns_topic_subscription.go index 75909fefbe0f..cece62b8a40a 100644 --- a/builtin/providers/aws/resource_aws_sns_topic_subscription.go +++ b/builtin/providers/aws/resource_aws_sns_topic_subscription.go @@ -5,11 +5,11 @@ import ( "log" "strings" + "github.com/hashicorp/terraform/helper/resource" "github.com/hashicorp/terraform/helper/schema" "github.com/aws/aws-sdk-go/aws" "github.com/aws/aws-sdk-go/service/sns" - "github.com/hashicorp/terraform/helper/resource" "time" ) @@ -188,7 +188,7 @@ func subscribeToSNSTopic(d *schema.ResourceData, snsconn *sns.SNS) (output *sns. endpoint := d.Get("endpoint").(string) topic_arn := d.Get("topic_arn").(string) endpoint_auto_confirms := d.Get("endpoint_auto_confirms").(bool) - confirmation_timeout_in_minutes := time.Duration(d.Get("confirmation_timeout_in_minutes").(int)) + confirmation_timeout_in_minutes := d.Get("confirmation_timeout_in_minutes").(int) if strings.Contains(protocol, "http") && !endpoint_auto_confirms { return nil, fmt.Errorf("Protocol http/https is only supported for endpoints which auto confirms!") @@ -213,7 +213,7 @@ func subscribeToSNSTopic(d *schema.ResourceData, snsconn *sns.SNS) (output *sns. log.Printf("[DEBUG] SNS create topic subscription is pending so fetching the subscription list for topic : %s (%s) @ '%s'", endpoint, protocol, topic_arn) - err = resource.Retry(time.Minute*confirmation_timeout_in_minutes, func() error { + err = resource.Retry(time.Duration(int(time.Minute)*confirmation_timeout_in_minutes), func() error { subscription, err := findSubscriptionByNonID(d, snsconn)