Skip to content

Parser secrets under terraform management#119

Draft
chris-bridgett-nandos wants to merge 1 commit intopagerduty-supportfrom
pagerduty-support-tf-secrets
Draft

Parser secrets under terraform management#119
chris-bridgett-nandos wants to merge 1 commit intopagerduty-supportfrom
pagerduty-support-tf-secrets

Conversation

@chris-bridgett-nandos
Copy link
Copy Markdown

No description provided.

@davidstanke
Copy link
Copy Markdown

Hey! This looks good, except, I'd prefer to actually generate the secret itself as part of the Terraform. You can see an example here: https://github.com/GoogleCloudPlatform/fourkeys/blob/e4f660f4dbfad13f852459443ba3ddc83a82503e/setup/resource_event_handler.tf#L57-L64

@chris-bridgett-nandos
Copy link
Copy Markdown
Author

Hey! This looks good, except, I'd prefer to actually generate the secret itself as part of the Terraform.

Hi. We would have liked to do this too, but unfortunately the secret is something generated and provided by PagerDuty themselves (mentioned here, documented by PagerDuty here (Response -> body -> delivery_method -> secret).

Unfortunately the secret is not exposed by their pagerduty_webhook_subscription Terraform resource, either. ☹️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants