Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

feat(ops): add IAM recommendation script #155

Closed
wants to merge 3 commits into from
Closed

feat(ops): add IAM recommendation script #155

wants to merge 3 commits into from

Conversation

ace-n
Copy link
Contributor

@ace-n ace-n commented Aug 26, 2021

Adds the IAM recommendation script requested in #131

Note: this does not add any enforcement capacity, since we need to figure out what format that should take. (CI checks? GitHub Issues? etc.)

@ace-n ace-n requested review from grayside and dinagraves August 26, 2021 00:06
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 26, 2021
ops/format_recommendations.py Outdated Show resolved Hide resolved
ops/format_recommendations.py Outdated Show resolved Hide resolved
ops/format_recommendations.py Outdated Show resolved Hide resolved
ops/format_recommendations.py Outdated Show resolved Hide resolved
ops/format_recommendations.py Outdated Show resolved Hide resolved
ops/format_recommendations.py Outdated Show resolved Hide resolved
ops/format_recommendations.py Outdated Show resolved Hide resolved
ops/format_recommendations.py Outdated Show resolved Hide resolved
@grayside
Copy link
Collaborator

grayside commented Sep 1, 2021

Will continue the discussion in #131. For now this PR is incomplete so converting to a draft.

@grayside grayside marked this pull request as draft September 1, 2021 17:39
@grayside grayside added do not merge Indicates a pull request not ready for merge, due to either quality or timing. needs work This is a pull request that needs a little love. labels Sep 1, 2021
@grayside
Copy link
Collaborator

This is a good idea but needs to be explored more as a standalone project.

@grayside grayside closed this Jan 14, 2022
@pattishin pattishin deleted the recs-script branch December 9, 2022 21:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement. do not merge Indicates a pull request not ready for merge, due to either quality or timing. needs work This is a pull request that needs a little love.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants