PMM-12287 Document removal of Integrated Alerting #3435
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
ademidoff
commented
Jan 8, 2025
| @@ -41,6 +41,7 @@ To use SMTP with a PMM Docker installation: | |||
| - `GF_SMTP_FROM_NAME`: Name to be used when sending out emails. | |||
|
|
|||
| *NB: If you are using your Gmail’s SMTP credentials as shown above, you will have to generate an app password and fill it in as the value of your $GF_SMTP_PASSWORD variable.* | |||
|
|
|||
There was a problem hiding this comment.
This fixes the bullet point rendering.
ademidoff
commented
Jan 8, 2025
|
|
||
| !!! caution alert alert-warning "Important" | ||
| Time specified in mute timing must be in UTC and military format i.e. 14:00 not 2:00 PM. | ||
| Time specified in mute timing must be in UTC format, i.e. 14:00, not 2:00 PM. |
There was a problem hiding this comment.
Military is not a well-known term, so I removed it to avoid confusion.
ademidoff
commented
Jan 8, 2025
| @@ -13,5 +13,5 @@ Percona Alerting is powered by Grafana infrastructure. It leverages Grafana's ad | |||
| Depending on the datasources that you want to query, and the complexity of your required evaluation criteria, Percona Alerting enables you to create the following types of alerts: | |||
|
|
|||
| - **Percona templated alerts**: alerts based on a set of Percona-supplied templates with common events and expressions for alerting. | |||
| If you need custom expressions on which to base your alert rules, you can also create your own templates. | |||
| If you want to use custom expressions in alert rules, you can create your own alert rule templates. | |||
There was a problem hiding this comment.
This makes it a bit more clear.
ademidoff
commented
Jan 8, 2025
| @@ -269,9 +269,10 @@ nav: | |||
| - Percona Alerting: | |||
| - alert/index.md | |||
| - alert/alert_rules.md | |||
| - alert/alert_notifications.md | |||
There was a problem hiding this comment.
This was rendering as None for such page is absent.
catalinaadam
reviewed
Jan 8, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PMM-12287