provider/aws: Add arn attribute for DynamoDB tables

[calvinfo]

This commit exports the arn as well as the id, since IAM
roles require the full resource name rather than just the table
name. I'd even be in favor or having arn as the id since the
<region, tablename> pair is the uniqueness constraint, but this
will keep backwards compatibility:

http://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_CreateTable.html

Feedback appreciated :)

[catsby]

I'm curious, does this work without specifying arn in the schema? I think arn needs to be defined above, like the Kinesis resource (though you can omit the optional and just leave it as computed).

[catsby]

Thanks for the contribution! I noted in-line, but I think we need two things here. First, we need the arn in the schema:

Schema: map[string]*schema.Schema{
    "arn": &schema.Schema{
        Type:     schema.TypeString,
        Computed: true,
    },
    "name": &schema.Schema{
    [...]

Second, let's move the d.Set into the read method, so that existing Dynamo resources will pick up the ARN on the next refresh. Remove your current d.Set("arn") and add this line just before the return nil in resourceAwsDynamoDbTableRead:

d.Set("arn", table.TableARN)

Then we're good to go!

[catsby]

Going to go ahead and pull this in, and make those modifications myself. Thanks again!

[catsby]

Said changes: a1a78bd

[calvinfo]

Whoop, good catch! Thanks for making the changes!

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