search

iann0036 / iam-dataset

A consolidated cloud IAM dataset
MIT License
232 stars 23 forks source link

Add redshiftdata clustername resource mapping #15

Closed dfangl closed 2 years ago

dfangl commented 2 years ago

As seen in iam_definition.json, the 5 affected resources need the cluster* resource. However, it is defined with a ClusterName, and not with ClusterIdentifier (as in the SDK request).

This PR adds the appropriate resource mappings to map.json. The request layout can be seen here: https://docs.aws.amazon.com/redshift-data/latest/APIReference/API_DescribeTable.html

I hope directly modifying this file is fine, I did not find any automation affecting this.

If there is anything still needed for this to go through, please let me know.

iann0036 commented 2 years ago

LGTM, thanks!

The map.json is indeed the source of truth, everything downstream consumes from here.