Add account id env variable to lambdas - Githubissues

Optum / dce

Disposable Cloud Environment

Apache License 2.0

315 stars 82 forks source link

Add account id env variable to lambdas #476

Closed bjfish25 closed 9 months ago

bjfish25 commented 9 months ago

Proposed changes

update_principal_policy lambda among others were using default 111111.. account id as the local account id wasn't being passed in
add a bit more basic error logging for roles that fail to create

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (changes to code, which do not change application behavior)

Checklist

[ ] I have filled out this PR template
[ ] I have read the CONTRIBUTING doc
[ ] I have added automated tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (README.md, inline comments, etc.)
[ ] I have updated the CHANGELOG.md under a ## next release, with a short summary of my changes

Relevant Links

Further comments