-
Notifications
You must be signed in to change notification settings - Fork 341
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
Adds the ability to use bypath to aws_secrets lookup #192
Adds the ability to use bypath to aws_secrets lookup #192
Conversation
@dlundgren thanks for the PR. We appreciate your help with this. cc @gundalow |
@dlundgren We would like to get this PR landed. Could you rebase this so I can review it soon? |
6cae0b0
to
4069fce
Compare
@gravesm I've rebased as requested, please let me know if there is any thing else I can do help get this reviewed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Personally not tested this. But code wise looks good to me.
@dlundgren Thanks for all your work on this. One last request and then I think we can get it merged. Could you add a changelog fragment for this PR? See: https://docs.ansible.com/ansible/latest/community/development_process.html#creating-a-changelog-fragment. |
(mainly looked at docs changes) |
SUMMARY
Adds
bypath
functionality to theaws_secrets
lookup. This allows it to work similar to theaws_ssm
lookup plugin.ISSUE TYPE
COMPONENT NAME
aws_secrets