prometheus / prometheus/alertmanager

UI: add notification observability when NotificationService is defined

Open
#5,494 0 comments 0 reactions 1 assignee View on GitHub

@siavashs is already working on this.

Since Aug 29, 2026.

component/api component/notify component/ui kind/enhancement
Dominant language
Go
Stars
8.6k
Forks
2.5k
Avg merge
2d 6h
Merged PRs (30d)
61

Description

Parent: #5451
API design dependency: #5484

Summary

Optionally add notification observability to the Mantine UI after #5484 chooses and publishes a semantics-correct notification.v3alpha.NotificationService contract.

This child is not a legacy-parity requirement and does not block completion of #5451.

Design gate

Do not implement a public page until #5484 decides among:

  1. Retained delivery history.
  2. Latest notification/deduplication state.
  3. Explicit deferral.

The UI must use the language and guarantees of the selected model. It must not present nflog deduplication state as delivery history or expose internal event-recorder/receiver payloads.

If #5484 explicitly defers NotificationService, close this issue as not planned with the API rationale.

Conditional implementation scope

If a service is published:

  • Generate and consume its TypeScript bindings through the shared Connect data layer.
  • Implement list/detail views for the selected state/history model.
  • Support receiver, receiver-label, group, outcome, and time filters that the approved contract defines.
  • Support deterministic ordering and pagination.
  • If WatchNotifications exists, consume it through the shared direct-stream infrastructure with documented snapshot/replay or resnapshot behavior.
  • Explain retention, persistence, HA merge/failover, ordering, cursor, replay, and sanitization limitations in the UI.
  • Provide loading, empty, unavailable, partial-result, and structured-error states.

Acceptance criteria

  • Notification semantics and storage are approved before any UI implementation.
  • UI terminology precisely matches retained history or latest-state semantics.
  • Internal nflog/event-recorder/integration payloads do not leak through the UI contract.
  • List/detail/filter/order/pagination behavior is generated, typed, bounded, and tested.
  • Watch behavior, if available, documents and handles retention, HA, overflow, restart, and replay/resnapshot limits.
  • If the API decision is deferral, this issue is closed as not planned rather than implying a missing implementation.
  • This issue remains excluded from #5451 completion criteria.

Dependencies and related work

  • NotificationService design and implementation: #5484.
  • Shared contracts: #5479.
  • Connect data layer: #5487.
  • Stream controls/broker #5478/#5480 and UI stream integration #5492 if a watch method is published.
  • Historical UI requests: #1305 and #172.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.