-
Notifications
You must be signed in to change notification settings - Fork 66
Add support for using secret manager environment variables in Cloud Function #88
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
Add support for using secret manager environment variables in Cloud Function #88
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). For more information, open the CLA check for this pull request. |
93c8735 to
3554484
Compare
3554484 to
45274ec
Compare
bharathkkb
left a comment
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.
Thanks for the PR @AronllStone
Could we also showcase this in one of the examples?
1ec5908 to
b1e17d0
Compare
b1e17d0 to
3a6fc71
Compare
|
@bharathkkb, could you please help with logs from |
|
@AronllStone The function seems to be failing to become ready I have kicked off another build and will try to see if anything shows up in the logs |
|
@AronllStone |
|
Hey, @bharathkkb & @morgante! |
Hello All,
I would like to add support for using secret manager environment variables to this module.
New variable:
secret_environment_variablesSupported by Google provider since version 4.11.0
Available for Terraform >= v0.13.0
Example of usage: