Repository: ansible/ansible

Issue:
dynamodb_table module has wrong version_added information
##### ISSUE TYPE
 - Documentation Report

##### COMPONENT NAME
 - dynamodb_table

##### ANSIBLE VERSION
```
$ ansible --version
ansible 2.3.0.0
  config file = 
  configured module search path = Default w/o overrides
  python version = 3.6.1 (default, Mar 22 2017, 06:17:05) [GCC 6.3.0 20170321]

```

##### CONFIGURATION

##### OS / ENVIRONMENT
N/A

##### SUMMARY

The `dynamodb_table` module lists `tags` and `wait_for_active_timeout` as being added in version 2.3, but they are not present in my current version of `2.3.0.0`:

http://docs.ansible.com/ansible/dynamodb_table_module.html

It appears that this is due to this PR being merged after the deadline for inclusion in 2.3:

https://github.com/ansible/ansible/pull/20946

##### STEPS TO REPRODUCE
```yaml
- dynamodb_table:
    name: foo
    region: us-east-1
    hash_key_name: id
    hash_key_type: STRING
    read_capacity: 2
    write_capacity: 2
    tags:
      foo: bar
```

<!--- You can also paste gist.github.com links for larger files -->

##### EXPECTED RESULTS

The playbook to run successfully and a DynamoDB table to be created

##### ACTUAL RESULTS
```
fatal: [localhost]: FAILED! => {
    "changed": false,
    "failed": true,
    "invocation": {
        "module_args": {
            "hash_key_name": "id",
            "hash_key_type": "STRING",
            "name": "foo",
            "read_capacity": 2,
            "region": "us-east-1",
            "tags": {
                "foo": "bar"
            },
            "write_capacity": 2
        }
    },
    "msg": "Unsupported parameters for (dynamodb_table) module: tags. Supported parameters include: aws_access_key,aws_secret_key,ec2_url,hash_key_name,hash_key_type,indexes,name,profile,range_key_name,range_key_type,read_capacity,region,security_token,state,validate_certs,write_capacity"
}
```



Discussion:
cc @loia
[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)
<!--- boilerplate: notify --->
@mechaxl By "merged after the deadline" are you also saying that this is working in devel?

needs_info
Sorry, "merged after the deadline" was a guess at what happened. The `tags` and `wait_for_active_timeout` parameters are not present in 2.3.0.0 as stated above. I can confirm that they are present after doing `pip install git+https://github.com/ansible/ansible.git@devel`, however.

Resolving patch:
diff --git a/lib/ansible/modules/cloud/amazon/dynamodb_table.py b/lib/ansible/modules/cloud/amazon/dynamodb_table.py
--- a/lib/ansible/modules/cloud/amazon/dynamodb_table.py
+++ b/lib/ansible/modules/cloud/amazon/dynamodb_table.py
@@ -86,13 +86,13 @@
     default: []
     version_added: "2.1"
   tags:
-    version_added: "2.3"
+    version_added: "2.4"
     description:
       - a hash/dictionary of tags to add to the new instance or for starting/stopping instance by tag; '{"key":"value"}' and '{"key":"value","key":"value"}'
     required: false
     default: null
   wait_for_active_timeout:
-    version_added: "2.3"
+    version_added: "2.4"
     description:
       - how long before wait gives up, in seconds. only used when tags is set
     required: false


Test patch: