[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