-
Notifications
You must be signed in to change notification settings - Fork 47
feat(TPG >=4.23)!: Add function_docker_registry variables and use them in terraform-google-event-function module #120
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
Conversation
Hi again @apeabody ! I'd then extend this module as well with the 3 variables used now in https://github.com/terraform-google-modules/terraform-google-event-function. Could you please review it + gcbrun? |
/gcbrun |
@tardigrde Hey, can you update the branch so that @apeabody can have a look? If you're busy, I can work on it. |
Updated! |
Updated the branch! @apeabody can you please move this forward somehow? |
/gcbrun |
Co-authored-by: Andrew Peabody <andrewpeabody@google.com>
No update since.. |
/gcbrun |
Hi @tardigrde - From the lint it needs a
|
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days |
@tardigrde @apeabody Any issues if I take over and open a new PR? I can build off the current commits. I've been waiting quite a while for this to get merged in. |
@alexrohv please take over. :) |
Closing since @alexrohv will take over. Thanks so far! |
This is a follow-up PR to terraform-google-modules/terraform-google-event-function#164. It depends on that PR, so it will be in draft until the other one is merged.
The scope of this PR is the same as the dependency PR. Variables added:
This enables storing Cloud Build container in Artifact Registry.
See https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/cloudfunctions_function#docker_registry