Skip to content
This repository was archived by the owner on Sep 17, 2021. It is now read-only.

Conversation

@LukeKennedy
Copy link
Contributor

The AWS IAM watcher attempts to list attachments for all policies within an account for users, groups, and roles. This causes a minimum of three calls to iam:ListEntitiesForPolicy for every policy even if the attachment count for a policy is zero. This change reduces the number of calls by checking the attachment count of a policy before listing the attached entities. For accounts with a large number of unused policies or additional tools calling IAM, this prevents unnecessary volume and helps prevent throttling by AWS.

The AWS IAM watcher attempts to list attachments for all policies within an account for users, groups, and roles. This causes a minimum
of three calls to iam:ListEntitiesForPolicy for every policy even if the attachment count for a policy is zero. This change reduces the
number of calls by checking the attachment count of a policy before listing the attached entities. For accounts with a large number of
unused policies or additional tools calling IAM, this prevents unnecessary volume and helps prevent throttling by AWS.
@scriptsrc
Copy link
Contributor

Nice.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 60.411% when pulling 6df4f03 on LukeKennedy:fewer-iam-calls into baddf1a on Netflix:develop.

@scriptsrc scriptsrc merged commit 370f40e into Netflix:develop Apr 25, 2017
@scriptsrc scriptsrc mentioned this pull request May 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants