Skip to content
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

feat: Add support for configuring pubsub object notifications #48

Closed
wants to merge 1 commit into from

Conversation

morgante
Copy link
Contributor

Fixes #28

@morgante morgante requested review from averbuks and bharathkkb April 19, 2020 03:08
@morgante morgante force-pushed the feat/notifications branch from 41ffbf6 to 4ce1954 Compare April 19, 2020 03:09
# project_id = var.project_id
bucket = lookup(each.value, "bucket")
payload_format = "JSON_API_V1"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at the google_storage_notification resource docs I think object_name_prefix and custom_attributes seem useful to be user configurable.

*/

locals {
# [for o in var.list : o.interfaces[0].name]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I think you meant to remove this?

resource "google_storage_notification" "notifications" {
for_each = var.bucket_configs

# project_id = var.project_id
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I think you meant to remove this?

@@ -138,3 +138,9 @@ variable "lifecycle_rules" {
description = "List of lifecycle rules to configure. Format is the same as described in provider documentation https://www.terraform.io/docs/providers/google/r/storage_bucket.html#lifecycle_rule except condition.matches_storage_class should be a comma delimited string."
default = []
}

variable "pubsub_topics" {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
variable "pubsub_topics" {
variable "pubsub_notification_topics" {

@github-actions
Copy link

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 Jan 20, 2021
@github-actions github-actions bot closed this Jan 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add notification configuration
2 participants