New module - s3_logging

[wimnat]

New module to manage logging option of s3 buckets

[gregdek]

cc @ansible/aws

[ghost]

According to http://docs.ansible.com/developing_modules.html#module-checklist, default is not needed when required is true.

[ghost]

@wimnat I did a some quick testing with this one and it works as advertised. Just a couple of documentation updates needed. The module is requiring target_bucket on absent so I think it should be added to the example.

[wimnat]

@RickMendes I've removed the target_bucket requirement and updated the doc to mention it is required for state=present. I didn't want to have state=absent dependent on target_bucket.

[ghost]

@wimnat That works for me. It makes more sense to me to not have it required for state-absent. I give this a +1.

[bcoca]

i'm guessing the default here is null or is 'no' the prefix that gets added?

[wimnat]

Should be an empty string rather than null.