terraform-aws-iam-system-user icon indicating copy to clipboard operation
terraform-aws-iam-system-user copied to clipboard

feat: add ses smpt password to ssm, add ssm base path option

Open Gowiem opened this issue 3 years ago • 6 comments

what

  • Add option to change the SSM base path under which credentials are stored
  • Store ses_smtp_password_v4 in SSM

why

  • Even if the IAM users are global per account often times there is a need to separeate them per environment (dev, staging). By adding the ssm_base_path parameter we can have a better SSM hierarchy. Example: /dev/system_user/tc-dev-s3 and /staging/system_user/tc-staging-ses
  • Also having the ses_smtp_password_v4 stored in SSM will allow better integration with CI systems when deploying applications (rather than having a script to generate the password or read the terraform state)

references

  • Link to any supporting github issues or helpful documentation to add some context (e.g. stackoverflow).
  • closes #60
  • Copy of #61, Closes #61

Gowiem avatar May 03 '22 23:05 Gowiem

/test all

Gowiem avatar May 03 '22 23:05 Gowiem

This pull request is now in conflict. Could you fix it @Gowiem? 🙏

mergify[bot] avatar May 03 '22 23:05 mergify[bot]

@Gowiem conflict

nitrocode avatar May 03 '22 23:05 nitrocode

@nitrocode have you seen those auto-readme conflicts before? Not sure what is up there... I believe I want to accept all the incoming changes, but wanted to get another set of eyes before I undo some work if I'm wrong.

Gowiem avatar May 03 '22 23:05 Gowiem

/test all

Gowiem avatar May 04 '22 17:05 Gowiem

/test all

Gowiem avatar May 04 '22 17:05 Gowiem