[OpenNMS/opennms] f901d0: NMS-20119: fix destination-path reload, last-path ...

joseanesONMS via opennms-cvs <[email protected]>
Newsgroups gmane.network.opennms.cvs
Message-ID <OpenNMS/opennms/push/refs/heads/jira/NMS-20119-destination-paths/[email protected]>
  Branch: refs/heads/jira/NMS-20119-destination-paths
  Home:   https://github.com/OpenNMS/opennms
  Commit: f901d0300574bca245009bd87c5994aa845a8df0
      https://github.com/OpenNMS/opennms/commit/f901d0300574bca245009bd87c5994aa845a8df0
  Author: Jose Anes <[email protected]>
  Date:   2026-08-12 (Wed, 12 Aug 2026)

  Changed paths:
    M opennms-config/src/main/java/org/opennms/netmgt/config/DestinationPathManager.java
    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/services/notificationConfigService.ts

  Log Message:
  -----------
  NMS-20119: fix destination-path reload, last-path delete, and rename consistency

Make a reload replace the destination-path map instead of merging over it by
building a new TreeMap and swapping the field reference in one assignment (and
mark parseXML synchronized), so readers keep a consistent snapshot and
saveCurrent()'s iteration is never emptied mid-flight. Reject deleting the last
remaining path with a 400 (it would otherwise 500 on schema validation) and
surface the server's message in the delete UI. On a rename, roll back the
persisted path change if the follow-up notifications save fails so
destinationPaths.xml and notifications.xml cannot diverge. Adds IT coverage for
the last-path rejection.



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.