[OpenNMS/opennms] 43b8d6: NMS-20120: Path Outages tab

joseanesONMS via opennms-cvs <[email protected]>
Newsgroups gmane.network.opennms.cvs
Message-ID <OpenNMS/opennms/push/refs/heads/jira/NMS-20120-path-outages/[email protected]>
  Branch: refs/heads/jira/NMS-20120-path-outages
  Home:   https://github.com/OpenNMS/opennms
  Commit: 43b8d62e299d2c738fa895bed519535e50fc9ac5
      https://github.com/OpenNMS/opennms/commit/43b8d62e299d2c738fa895bed519535e50fc9ac5
  Author: Jose Anes <[email protected]>
  Date:   2026-08-05 (Wed, 05 Aug 2026)

  Changed paths:
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/NotificationConfigRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v1/NotificationConfigRestServiceIT.java
    M ui/src/components/AdminNotifications/ConfigureNotificationsDialog.vue
    A ui/src/components/AdminNotifications/PathOutagesTab.vue
    M ui/src/services/index.ts
    M ui/src/services/notificationConfigService.ts
    M ui/src/stores/notificationConfigStore.ts
    M ui/src/types/notificationConfig.ts
    M ui/tests/stores/notificationConfigStore.test.ts

  Log Message:
  -----------
  NMS-20120: Path Outages tab

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.


  Commit: 03c9188b0dd9d43f04cf93d0e4109d29bd8eb2cc
      https://github.com/OpenNMS/opennms/commit/03c9188b0dd9d43f04cf93d0e4109d29bd8eb2cc
  Author: Jose Anes <[email protected]>
  Date:   2026-08-05 (Wed, 05 Aug 2026)

  Changed paths:
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/NotificationConfigRestService.java
    M ui/src/components/AdminNotifications/ConfigureNotificationsDialog.vue
    M ui/src/components/AdminNotifications/PathOutagesTab.vue
    M ui/src/services/notificationConfigService.ts
    M ui/src/stores/notificationConfigStore.ts
    M ui/tests/stores/notificationConfigStore.test.ts

  Log Message:
  -----------
  NMS-20120: review pass on the Path Outages tab

The node-filter rule now starts empty instead of IPADDR IPLIKE *.*.*.*, so the
tab no longer opens pre-armed to clear the critical path for every node. The
clear confirmation counts only the nodes that currently have a critical path and
match the rule, not every node the rule selects, so it no longer claims thousands
when a handful of rows will change. The path-outages tab loader now propagates a
real success flag — getPathOutages returns null on error and the store reports
false — so a failed load retries instead of latching the tab empty.

Also drops the unused isBlank helper from NotificationConfigRestService.


  Commit: e19ee50f20322f00398374df51fad48652e484f0
      https://github.com/OpenNMS/opennms/commit/e19ee50f20322f00398374df51fad48652e484f0
  Author: Jose Anes <[email protected]>
  Date:   2026-08-05 (Wed, 05 Aug 2026)

  Changed paths:
    M ui/src/components/AdminNotifications/PathOutagesTab.vue

  Log Message:
  -----------
  NMS-20120: use @opennms/onms-ui wrappers instead of direct PrimeVue

Migrate the Path Outages tab onto the Onms-* seam wrappers (table,
chip, buttons, inputs, select) so the page passes the
no-direct-primevue lint rule; iftalabel stays on PrimeVue as it has no
wrapper. Markup and behaviour are unchanged.


Compare: https://github.com/OpenNMS/opennms/compare/2228c22efc63...e19ee50f2032

To unsubscribe from these emails, change your notification settings at https://github.com/OpenNMS/opennms/settings/notifications


_______________________________________________
Please read the OpenNMS Mailing List FAQ:
http://www.opennms.org/wiki/index.php?page=MailingListFaq
opennms-cvs mailing list

To *unsubscribe* or change your subscription options, see the bottom of this page:
https://lists.sourceforge.net/lists/listinfo/opennms-cvs
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.