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

Add example of referencing the Default Global Ruleset #239

Merged
merged 2 commits into from
Jun 9, 2020

Conversation

nathanielks
Copy link
Contributor

It wasn't exactly clear how to reference the default global ruleset, but I realized I could query the API to find the name of the ruleset:

curl -H "Authorization: Token token=$PAGERDUTY_TOKEN" https://api.pagerduty.com/rulesets

This showed me the name, which is "Default Global." In hindsight, this now makes sense looking at the PagerDuty dashboard, but it wasn't clear before I looked at the API response. I thought this would help others like me!

Copy link
Contributor

@stmcallister stmcallister left a comment

Choose a reason for hiding this comment

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

Thank you for this submission! Will you please make the changes requested? Thanks!

```
data "pagerduty_ruleset" "default_global" {
name = "Default Global"
}w
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you remove the w here?

@@ -46,6 +46,14 @@ resource "pagerduty_ruleset_rule" "foo" {
}
```

### Default Global Ruleset

```
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add an hcl here so that the syntax highlights correctly?

@nathanielks
Copy link
Contributor Author

@stmcallister updated!

Copy link
Contributor

@stmcallister stmcallister left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for making the changes!

@stmcallister stmcallister merged commit 158f3f4 into PagerDuty:master Jun 9, 2020
@nathanielks nathanielks deleted the docs-default-global branch June 9, 2020 22:20
@nathanielks
Copy link
Contributor Author

Most certainly!

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

Successfully merging this pull request may close these issues.

2 participants