-
Notifications
You must be signed in to change notification settings - Fork 221
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
Improves the dynamic providers docs page examples to reflect how to pass creds via env variables and configuration. #13372
base: master
Are you sure you want to change the base?
Improves the dynamic providers docs page examples to reflect how to pass creds via env variables and configuration. #13372
Conversation
In pulumi/pulumi#17673 and pulumi/pulumi#17697 we introduced a new optional configure method that has access to the stack configuration. Wait for the next Pulumi release before merging this
Co-authored-by: Will Jones <will@sacharissa.co.uk>
Your site preview for commit e83d864 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-13372-e83d8649.s3-website.us-west-2.amazonaws.com. |
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.
Nice!
Your site preview for commit 49f0cda is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-13372-49f0cdaa.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit b71e675 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-13372-b71e6758.s3-website.us-west-2.amazonaws.com. |
Proposed changes
The dynamic providers page has examples that do not show best practices for passing in secrets. This update modifies the github example to use the (new)
configure()
interface.It also removes the simple
configure()
example that was added with the docs update to add the interface.Unreleased product version (optional)
Related issues (optional)