Skip to content

NMS-20100: Path Outages tab - #8708

Closed
joseanesONMS wants to merge 2 commits into
OpenNMS:developfrom
joseanesONMS:jira/NMS-20100-path-outages
Closed

NMS-20100: Path Outages tab#8708
joseanesONMS wants to merge 2 commits into
OpenNMS:developfrom
joseanesONMS:jira/NMS-20100-path-outages

Conversation

@joseanesONMS

Copy link
Copy Markdown

Adds the Path Outages tab to the Configure Notifications dialog. Depends on #8693 (the base Notifications page PR); review the last commit only until that merges.

  • Tab with the list of configured critical paths and an apply/clear flow driven by a filter rule with a node-count preview.
  • Storage is and stays the pathoutage table; the endpoints mirror the legacy NotificationWizardServlet semantics, including clearing the path when the critical IP is blank.

Rewrites the Notifications page (notice queries, outstanding/acknowledged
notice lists with CSV export and printing, collapsible explanations) as a
PrimeVue page backed by REST, and adds a Configure Notifications dialog
holding a General tab with the system-wide notification on/off switch.

Adds the admin-only /rest/notification-config endpoints for the notifd
status and the read side of destination paths (the base API surface for the
dependent tab PRs), backed by the same file-backed factories the legacy
wizards use — the XML files stay the system of record. Browser delivery of
notices for /ui pages ships here as well. The Event Notifications,
Destination Paths and Path Outages tabs follow in dependent PRs.
Adds the Path Outages tab: the list of configured critical paths and the
apply/clear flow driven by a filter rule with a node-count preview, in
place of the legacy notification wizard pages. Storage is and stays the
pathoutage table; the endpoints mirror the legacy NotificationWizardServlet
semantics, including clearing the path when the critical IP is blank.
Depends on the Notifications page base PR.
@joseanesONMS

Copy link
Copy Markdown
Author

Replaced by #8712 (NMS-20120, in-repo stacked on #8709).

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