Skip to content

log based alerts docs#596

Open
JohnBlackwell wants to merge 3 commits intomainfrom
log-based-alerts-docs
Open

log based alerts docs#596
JohnBlackwell wants to merge 3 commits intomainfrom
log-based-alerts-docs

Conversation

@JohnBlackwell
Copy link
Copy Markdown
Contributor

Plural Flow: docs
Plural Preview: docs

@JohnBlackwell JohnBlackwell requested a review from a team as a code owner April 29, 2026 20:32
Copy link
Copy Markdown
Contributor

@plural-copilot plural-copilot Bot left a comment

Choose a reason for hiding this comment

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

✅ Deploy preview for docs-pr-596 is ready!

Name Details
⚡ Service docs-pr-596 (5 / 5 ready)
☁️ Cluster plural (eks)
:octocat: Commit sha 8bf7436
Plural URL 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-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 29, 2026

Greptile Summary

This 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/5

This 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.

Important Files Changed

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

Comment thread pages/plural-features/observability/logging.md Outdated
@JohnBlackwell
Copy link
Copy Markdown
Contributor Author

@greptile

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.

1 participant