Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CLI option to force-include AWS services #13

Closed
kmcquade opened this issue Jan 22, 2021 · 5 comments · Fixed by #20
Closed

Add CLI option to force-include AWS services #13

kmcquade opened this issue Jan 22, 2021 · 5 comments · Fixed by #20
Labels
discussion enhancement New feature or request

Comments

@kmcquade
Copy link
Collaborator

kmcquade commented Jan 22, 2021

Just in case someone needs to include an AWS service but the tooling has not caught up.

aws-allowlister generate --include serv1,serv2,serv3

Maybe also an exclude option:

aws-allowlister generate --exclude qldb

@jdyke thoughts?

@kmcquade kmcquade added the enhancement New feature or request label Jan 22, 2021
@jdyke
Copy link
Collaborator

jdyke commented Jan 22, 2021

I thought about this scenario. I like the idea of the self-service approach to allow custom policy creation but went back and forth on if that fits into the goal of the repo from a compliance pov. If the repo will expand to being a general SCP / policy creation tool then it makes sense.

I also thought about creating a custom "Issue" on this repo that is templated to add or remove services from compliance programs. For example, if the repo is out of date, we would have a dedicated issue template to request a service be added (if the user did not want to make the code changes themselves).

@kmcquade
Copy link
Collaborator Author

I also thought about creating a custom "Issue" on this repo that is templated to add or remove services from compliance programs. For example, if the repo is out of date, we would have a dedicated issue template to request a service be added (if the user did not want to make the code changes themselves).

I was thinking about using this for that:

https://github.com/salesforce/aws-allowlister/blob/main/aws_allowlister/data/overrides.yml#L168

@kmcquade
Copy link
Collaborator Author

I like the idea of the self-service approach to allow custom policy creation but went back and forth on if that fits into the goal of the repo from a compliance pov. If the repo will expand to being a general SCP / policy creation tool then it makes sense.

I definitely don't think we want it to be a general SCP / policy creation tool - I wrote it specifically for AllowLists lol. Perhaps we should settle for uniformity instead of an overload of customization opportunities, and not include this CLI option

@kmcquade
Copy link
Collaborator Author

I spoke with someone about this internally. He identified two very legit use cases for if someone would want these include and exclude options (for reasons other than doing things you shouldn't be doing)

  1. Include option - could be useful for non-prod situations where people need to muck around, or a service has not been certified yet but it is about to be
  2. Exclude option - if someone doesn't want to pay for QLDB even though it is compliant.

@jdyke
Copy link
Collaborator

jdyke commented Jan 23, 2021

Very good points. It doesn't hurt to include it and can only add benefits (not take away) for numerous situations, especially the exclude scenario like you mentioned.

kmcquade added a commit that referenced this issue Jan 23, 2021
…include-aws-services

Adds ability to exclude and include AWS services via command line
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants