Skip to content

feat(telemetry): Mail Telemetry Alerts from a template - #41

Merged
s-aga-r merged 2 commits into
developfrom
feat/telemetry-alert-email
Oct 28, 2026
Merged

s-aga-r merged 2 commits into
developfrom
feat/telemetry-alert-email

Conversation

@s-aga-r

@s-aga-r s-aga-r commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Telemetry Alert mails use the layout of the Suite emails. Before, a mail was the alert details and a bare link.
  • One template, templates/operator_emails/telemetry_alert.html, extends tenant_emails/base.html. It sets the heading and message for each alert type (Silent Host, Error Rate) and state (opened, resolved).
  • A details box shows the node (hostname and IP), the opened and resolved times in the system timezone, and the alert details. A button opens the alert.
  • An open Silent Host mail tells the operator to resolve the alert by hand for a node taken out of service.
  • Alerts.notify renders the template and embeds the Suite icon with get_inline_images. The subject does not change.
  • The template is in operator_emails/ because these mails go to the Suite Cloud operators, not to a tenant.

Screenshots

Silent Host, opened Silent Host, resolved
silent-opened silent-resolved
Error Rate, opened Error Rate, resolved (node without a hostname)
error-opened error-resolved

Test plan

  • test_alerts.py checks the heading, details, link and icon of the opened mails, and the heading and Resolved row of the resolved mail.
  • Ruff check and format --check pass on suite_cloud/telemetry. The 10 alert tests pass locally on the parent branch, where alerts.py, test_alerts.py and email_types.py are the same as on develop.

An alert mail was its details text and a bare link. It now uses the
layout of the Suite emails: a heading for the alert type and state, a
box with the node, the opened and resolved times and the details, and a
button to the alert. An open Silent Host alert also says to resolve it
by hand for a node taken out of service.

The template is in templates/operator_emails because these mails go to
the Suite Cloud operators, not to a tenant.
@s-aga-r
s-aga-r merged commit 8479b51 into develop Oct 28, 2026
7 checks passed
@s-aga-r
s-aga-r deleted the feat/telemetry-alert-email branch October 28, 2026 07:15
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.

1 participant