AWSX-1006 fix s3 listobject issue visible in CloudTrail

[tyrcho]

What does this PR do?

Fix the wrong path declared in the CFT so the retry mechanism has access to the proper s3 prefix

Motivation

Issues reported by a few customers

Testing Guidelines

Additional Notes

I think the intent was to give ListBucket permission here but I'm a bit confused why we have 2 similar blocks
with different actions an resource id here and here.

Also I see in this commit that the ListBucket was removed from the first block by an external contributor.

In the change which was introduced 3 months ago to store and retry, there might be a mistake. IIUC, the call to list_objects uses the prefix failed_events while the template gives permission to the s3 prefix retry

The other prefix log-group-cache seems to be used properly from the other file you pointed out.

Types of changes

• Bug fix
• New feature
• Breaking change
• Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

• This PR's description is comprehensive
• This PR contains breaking changes that are documented in the description
• This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
• This PR impacts documentation, and it has been updated (or a ticket has been logged)
• This PR's changes are covered by the automated tests
• This PR collects user input/sensitive content into Datadog
• This PR passes the integration tests (ask a Datadog member to run the tests)
• This PR passes the unit tests
• This PR passes the installation tests (ask a Datadog member to run the tests)