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

[DOCS] Alerting PagerDuty benefits #63652

Merged
merged 9 commits into from
Apr 20, 2020
Merged

Conversation

gchaps
Copy link
Contributor

@gchaps gchaps commented Apr 16, 2020

Summary

This PR adds information about integrating PagerDuty with Elastic Alerts.

Preview
http://kibana_63652.docs-preview.app.elstc.co/diff

Copy link

@alexfrancoeur alexfrancoeur left a comment

Choose a reason for hiding this comment

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

Just ran through these instructions in Kibana and added a two comments. With those changes, this LGTM! Any chance we can get this merged by end of week? cc: @arisonl

* Alternatively, create a connector by navigating to Management from the {kib} navbar and selecting
*Alerts and Actions*. Then select the *Connectors* tab, click on the *Create connector* button, and select the PagerDuty option.

. Enter a name for the connector, the API URL, and the Routing Key.

Choose a reason for hiding this comment

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

The API URL seems to be optional. Maybe we call that out what the default is here?

Copy link

@alexfrancoeur alexfrancoeur left a comment

Choose a reason for hiding this comment

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

The changes look good to me @gchaps. I dropped in a few suggestions as well. Given that the changes are a little larger than originally expected, I added @arisonl as a reviewer. Aris, let us know what you think.

[[pagerduty-how-it-works]]
==== How it works

{kib} allows you to create threshold alerts to notify you of a significant move

Choose a reason for hiding this comment

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

To avoid continual updates in the docs, I'd vote to keep a few of these sentences more generic than they are right now. {kib} allows you to create alerts to notify you of a significant move. What do you think @arisonl ?


{kib} allows you to create threshold alerts to notify you of a significant move
in your datasets. You can create alerts of a generic type that you can use for all your indices.
You can also create specific alerts for Observability and SIEM.

Choose a reason for hiding this comment

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

In a similar vein, what about replacing

in your datasets. You can create alerts of a generic type that you can use for all your indices.
You can also create specific alerts for Observability and SIEM.

with

You can create alerts for all your Observability, Security and Stack use cases

@gchaps @arisonl

Copy link
Contributor Author

Choose a reason for hiding this comment

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

++

* Alternatively, create a connector by navigating to *Management* from the {kib} navbar and selecting
*Alerts and Actions*. Then, select the *Connectors* tab, click the *Create connector* button, and select the PagerDuty option.

. Configure the connector by giving it a name and entering the API URL and Routing Key.

Choose a reason for hiding this comment

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

I've learned that the API URL is optional, so it might be worth calling out here.

Configure the connector by giving it a name, using the default or optional API URL and Routing Key.

@arisonl @gchaps probably not the best phrasing, but does it make sense to call out that it's optional without having to click into the config docs? I actually opened an issue for this to fix in product: #63768

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@alexfrancoeur are both the API URL and Routing Key optional, or just the API URL?

Copy link
Contributor

@arisonl arisonl left a comment

Choose a reason for hiding this comment

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

Only question for me at this point is whether we should replace "Elastic Alerts" references with "Kibana Alerts".

Copy link
Contributor

@mikecote mikecote left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for enhancing the docs!

docs/user/alerting/action-types/pagerduty.asciidoc Outdated Show resolved Hide resolved
Copy link

@alexfrancoeur alexfrancoeur left a comment

Choose a reason for hiding this comment

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

@gchaps @arisonl I added suggestions to replace "Elastic Alerts" with more generic alerting references. It feels more natural to have in our docs vs. explicitly calling out Elastic or Kibana. If you come from our documentation, you already have context that you're looking at Elastic and Kibana documentation. The title for alerting is "Alerting and actions", not "Kibana / Elastic alerting and actions". It felt redundant and I propose we simplify. If you're coming direction from PagerDuty's integration, you'll have full context as the integration will be called "Elastic Alerts" to describe the company and functionality. Taking this approach also aligns with the subscription page and "Kibana alerts" syntax as the alerting documentation is nested under Kibana.

[[pagerduty-benefits]]
=== PagerDuty + Elastic integration benefits

By integrating PagerDuty with Elastic Alerts, you can:

Choose a reason for hiding this comment

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

"By integrating PagerDuty with alerting, you can"


By integrating PagerDuty with Elastic Alerts, you can:

* Route your Elastic Alerts to the right PagerDuty responder within your team, based on your structure, escalation policies, and workflows.
Copy link

@alexfrancoeur alexfrancoeur Apr 18, 2020

Choose a reason for hiding this comment

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

"Route your alerts to the right PagerDuty responder within your team, based on your "

{kib} allows you to create alerts to notify you of a significant move
in your dataset.
You can create alerts for all your Observability, Security, and Elastic Stack use cases.
Elastic Alerts triggers a new incident on the corresponding PagerDuty service.

Choose a reason for hiding this comment

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

"Alerts will trigger a new incident on the corresponding PagerDuty service"

application privileges to access Metrics, APM, Uptime, or SIEM.
* If you are using a self-managed deployment with security, you must have
Transport Security Layer (TLS) enabled for communication <<configuring-tls-kib-es, between Elasticsearch and Kibana>>.
Elastic Alerts uses API keys to secure background alert checks and actions,

Choose a reason for hiding this comment

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

"Alerting uses API keys to secure background alert checks and actions,"

@gchaps gchaps marked this pull request as ready for review April 20, 2020 15:37
@gchaps gchaps added release_note:skip Skip the PR/issue when compiling release notes Team:Docs v7.7.0 v7.8.0 v8.0.0 labels Apr 20, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

@gchaps gchaps merged commit d45d780 into elastic:master Apr 20, 2020
@gchaps gchaps deleted the docs/pagerduty branch April 20, 2020 16:09
gchaps added a commit to gchaps/kibana that referenced this pull request Apr 20, 2020
* [DOCS] Alerting PagerDuty benefits

* [DOCS] Fixes broken link

* [DOCS] Organization changes

* [DOCS] Changes to meet template and incorporate review comments

* [DOCS] Fixed formatting of bulleted list

* [DOCS] Incorporates review comments

* Update docs/user/alerting/action-types/pagerduty.asciidoc

Co-Authored-By: Mike Côté <[email protected]>

* [DOCS] Fixes naming and other formatting issues

Co-authored-by: Mike Côté <[email protected]>
gchaps added a commit to gchaps/kibana that referenced this pull request Apr 20, 2020
* [DOCS] Alerting PagerDuty benefits

* [DOCS] Fixes broken link

* [DOCS] Organization changes

* [DOCS] Changes to meet template and incorporate review comments

* [DOCS] Fixed formatting of bulleted list

* [DOCS] Incorporates review comments

* Update docs/user/alerting/action-types/pagerduty.asciidoc

Co-Authored-By: Mike Côté <[email protected]>

* [DOCS] Fixes naming and other formatting issues

Co-authored-by: Mike Côté <[email protected]>
gchaps added a commit that referenced this pull request Apr 20, 2020
* [DOCS] Alerting PagerDuty benefits

* [DOCS] Fixes broken link

* [DOCS] Organization changes

* [DOCS] Changes to meet template and incorporate review comments

* [DOCS] Fixed formatting of bulleted list

* [DOCS] Incorporates review comments

* Update docs/user/alerting/action-types/pagerduty.asciidoc

Co-Authored-By: Mike Côté <[email protected]>

* [DOCS] Fixes naming and other formatting issues

Co-authored-by: Mike Côté <[email protected]>

Co-authored-by: Mike Côté <[email protected]>
gchaps added a commit that referenced this pull request Apr 20, 2020
* [DOCS] Alerting PagerDuty benefits

* [DOCS] Fixes broken link

* [DOCS] Organization changes

* [DOCS] Changes to meet template and incorporate review comments

* [DOCS] Fixed formatting of bulleted list

* [DOCS] Incorporates review comments

* Update docs/user/alerting/action-types/pagerduty.asciidoc

Co-Authored-By: Mike Côté <[email protected]>

* [DOCS] Fixes naming and other formatting issues

Co-authored-by: Mike Côté <[email protected]>

Co-authored-by: Mike Côté <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Docs v7.7.0 v7.8.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants