Skip to content

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

Closed
wants to merge 12 commits into from

Conversation

tardigrde
Copy link
Contributor

@tardigrde tardigrde commented Apr 27, 2023

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:

  • function_docker_registry
  • function_docker_repository
  • function_kms_key_name

This enables storing Cloud Build container in Artifact Registry.

See https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/cloudfunctions_function#docker_registry

@tardigrde tardigrde changed the title feat: Add function_docker_registry variables and use them in terraform-google-event-function module May 25, 2023
@tardigrde tardigrde marked this pull request as ready for review May 25, 2023 06:02
@tardigrde tardigrde requested a review from a team as a code owner May 25, 2023 06:02
@tardigrde
Copy link
Contributor Author

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?

@apeabody
Copy link
Contributor

/gcbrun

@alexrohv
Copy link
Contributor

@tardigrde Hey, can you update the branch so that @apeabody can have a look? If you're busy, I can work on it.

@tardigrde
Copy link
Contributor Author

Updated!

@tardigrde
Copy link
Contributor Author

Updated the branch! @apeabody can you please move this forward somehow?

@apeabody
Copy link
Contributor

/gcbrun

Co-authored-by: Andrew Peabody <andrewpeabody@google.com>
@tardigrde
Copy link
Contributor Author

No update since..

@apeabody
Copy link
Contributor

/gcbrun

@apeabody
Copy link
Contributor

Hi @tardigrde - From the lint it needs a make docker_generate_docs

Checking for documentation generation
diff -r '--exclude=.terraform' '--exclude=.kitchen' '--exclude=autogen' '--exclude=*.tfvars' '--exclude=*metadata.yaml' /workspace/README.md /tmp/tmp.h0veMwqh1U/generate_docs/workspace/README.md
73c73
< | vpc\_connector | The VPC Network Connector that this cloud function can connect to. It should be set up as fully-qualified URI. The format of this field is projects//locations//connectors/\*. | `string` | `null` | no |
---
> | vpc\_connector | The VPC Network Connector that this cloud function can connect to. It should be set up as fully-qualified URI. The format of this field is projects//locations//connectors/*. | `string` | `null` | no |
diff -r '--exclude=.terraform' '--exclude=.kitchen' '--exclude=autogen' '--exclude=*.tfvars' '--exclude=*metadata.yaml' /workspace/examples/logs-slack-alerts/README.md /tmp/tmp.h0veMwqh1U/generate_docs/workspace/examples/logs-slack-alerts/README.md
30c30
< No output.
---
> No outputs.
Error: Documentation generation has not been run, please run the
'make docker_generate_docs' command and commit the above changes.
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

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

@github-actions github-actions bot added the Stale label Oct 9, 2023
@alexrohv
Copy link
Contributor

@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.

@github-actions github-actions bot removed the Stale label Oct 10, 2023
@tardigrde
Copy link
Contributor Author

@alexrohv please take over. :)

@tardigrde
Copy link
Contributor Author

Closing since @alexrohv will take over.

Thanks so far!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants