log based alerts docs#596
Conversation
There was a problem hiding this comment.
✅ Deploy preview for docs-pr-596 is ready!
| Name | Details |
|---|---|
| ⚡ Service | docs-pr-596 (5 / 5 ready) |
| ☁️ Cluster | plural (eks) |
| 8bf7436 | |
| https://console.mgmt.plural.sh/cd/clusters/446acdef-8524-425a-a49f-5336a... | |
| 🔍 Logs URL | https://console.mgmt.plural.sh/cd/clusters/446acdef-8524-425a-a49f-5336a... |
| 🌊 Flow URL | https://console.mgmt.plural.sh/flows/08a17af1-8578-449e-9ae1-28ccd97eafc... |
| Preview Link | Preview |
You can learn more about preview environments and Plural Flows here
Greptile SummaryThis PR adds a new "Log-Based Alerts" section to the logging documentation, covering how monitors work, a three-step UI walkthrough, Liquid alert templates, and routing/AI integration. Five supporting screenshots are included and all image alt texts are correctly set. Confidence Score: 5/5This PR is safe to merge — it adds documentation and static assets only, with no code changes. All changes are documentation and image assets. No logic, security, or runtime concerns. Previously flagged issues (alt text, page title) have either been resolved or are tracked in earlier review threads. No files require special attention.
|
| Filename | Overview |
|---|---|
| pages/plural-features/observability/logging.md | Adds a well-structured "Log-Based Alerts" section covering monitor setup, UI walkthrough, Liquid alert templates, and AI/routing integration; existing front-matter title still reads "ElasticSearch Configuration" (flagged in a previous review) |
| public/assets/observability/create-monitor.png | New screenshot asset for the "Create Monitor" wizard step |
| public/assets/observability/log-query.png | New screenshot asset for the log query configuration step |
| public/assets/observability/threshold-config.png | New screenshot asset for the threshold configuration step |
| public/assets/observability/monitor-description.png | New screenshot asset for the monitor description/schedule step |
| public/assets/observability/monitor-insight.png | New screenshot asset for the AI insights / monitor-insight view; alt text is correctly set to "monitor-insight" |
Reviews (2): Last reviewed commit: "fix typo" | Re-trigger Greptile
|
@greptile |
Plural Flow: docs
Plural Preview: docs