-
Notifications
You must be signed in to change notification settings - Fork 389
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
New PagerDuty Integration Resource #135
Conversation
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.
* `service_name` - (Required) Your Service name in PagerDuty. | ||
* `service_key` - (Required) Your Service name associated service key in Pagerduty. | ||
* `schedules` - (Optional) Array of your schedule URLs. | ||
* `subdomain` - (Required) Your PagerDuty account’s personalized subdomain name. |
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.
The subdomain is only required for the "crawler" part of the integration right? If that's the case, it's is not more required than the API token is 🤔
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.
That is correct, the subdomain
in only required by the crawler. Here in the terraform provider context, its being used as a unique identifier for the resource schema.
|
||
### See also | ||
* [PagerDuty Integration Guide](https://www.pagerduty.com/docs/guides/datadog-integration-guide/) | ||
* [Datadog API Reference > Integrations > PagerDuty](https://docs.datadoghq.com/api/?lang=bash#pagerduty) |
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.
This documentation looks good to me 🙇
This looks like it supersedes #56 ? |
New PagerDuty Integration Resource
What does this PR do?
Creates a new PagerDuty Integration resource from commits here:
https://github.com/terraform-providers/terraform-provider-datadog/pull/56
Who will it impact?
No one is yet using this resource.
Motivation
Manage Datadog PagerDuty integration configurations from terraform.