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.

Sign up for GitHub

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

Jump to bottom

Add an output to export resource_id #112

Merged
merged 3 commits into from
Apr 2, 2021
Merged

Add an output to export resource_id #112

merged 3 commits into from
Apr 2, 2021

Conversation

jblackburn22
Copy link
Contributor

what

  • Exports the resource_id

why

  • The resource_id is needed in order to create a least-privileges IAM policy to provide using RDS IAM authentication.

references

  • Use closes #111

@jblackburn22 jblackburn22 requested review from a team as code owners April 2, 2021 19:46
@jblackburn22 jblackburn22 requested a review from a team as a code owner April 2, 2021 19:48
@jblackburn22 jblackburn22 requested review from Gowiem and 3h4x April 2, 2021 19:48
@Nuru
Copy link
Contributor

Nuru commented Apr 2, 2021

/test all

@Nuru Nuru added the patch A minor, backward compatible change label Apr 2, 2021
@Nuru Nuru merged commit 1da1851 into cloudposse:master Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nuru Nuru Nuru approved these changes

@Gowiem Gowiem Awaiting requested review from Gowiem

@3h4x 3h4x Awaiting requested review from 3h4x

Assignees
No one assigned
Labels
patch A minor, backward compatible change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jblackburn22 @Nuru @cloudpossebot