Skip to content

Docs: Update dashboard instructions for alert configs - #3327

Merged
jwhartley merged 1 commit into
masterfrom
emily/alert-config-ui-docs
Aug 12, 2026
Merged

Docs: Update dashboard instructions for alert configs#3327
jwhartley merged 1 commit into
masterfrom
emily/alert-config-ui-docs

Conversation

@aeluce

@aeluce aeluce commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Description:

Updates docs to note new alert configuration options in the dashboard and specifies which functionality is available through the dashboard vs. flowctl on the customization page, which previously was solely focused on flowctl instructions.

Documentation links affected:

Pages on notifications and customizing alerts.

Notes for reviewers:

Thanks for reviewing! Updates are fairly minimal; open to further changes if needed, though the dashboard instructions feel pretty self-explanatory.

@aeluce
aeluce requested review from danthelion and jwhartley August 7, 2026 22:00
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

@jwhartley
jwhartley requested a review from jonwihl August 9, 2026 23:28

@jonwihl jonwihl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. There should be a note that you can't create duplicate alert subscriptions for the same prefix. Ex. There can only be one alert subscription for AcmeCo/ and you should edit the existing rules or add new alerts for prefixes instead

@jwhartley

Copy link
Copy Markdown
Contributor

I have a customer asking for this doc so I'm going to merge it @aeluce , if you want to add the note Jonathan mentioned can you please do it in a new PR sorry

@jwhartley
jwhartley merged commit 55a598d into master Aug 12, 2026
8 checks passed
@jwhartley

jwhartley commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

also @jonwihl task failure alert parameters aren't configurable in the UI yet?

@aeluce

aeluce commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

Data movement seems to be the only global setting currently and the UI PR didn't return hits when searching for task/shard failures. Would love to be proven wrong if you find otherwise.

@aeluce
aeluce deleted the emily/alert-config-ui-docs branch August 12, 2026 16:30
@jonwihl

jonwihl commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Thats correct, task failure configuration isn't available yet

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants