[OpenNMS/opennms] d0c28c: NMS-20046: Handle Poller/Collect RPC executor shut...

  Branch: refs/heads/merge-foundation/foundation-2023-to-foundation-2024
  Home:   https://github.com/OpenNMS/opennms
  Commit: d0c28cc52d7d7a94d9148d54b82040c3f136220d
      https://github.com/OpenNMS/opennms/commit/d0c28cc52d7d7a94d9148d54b82040c3f136220d
  Author: Chandra Gorantla <[email protected]>
  Date:   2026-07-22 (Wed, 22 Jul 2026)

  Changed paths:
    M features/collection/client-rpc/src/main/resources/META-INF/opennms/applicationContext-rpc-collector.xml
    A features/collection/client-rpc/src/test/java/org/opennms/netmgt/collection/client/rpc/CollectorExecutorLifecycleTest.java
    M features/poller/client-rpc/src/main/resources/META-INF/opennms/applicationContext-rpc-poller.xml
    A features/poller/client-rpc/src/test/java/org/opennms/netmgt/poller/client/rpc/PollerExecutorLifecycleTest.java

  Log Message:
  -----------
  NMS-20046: Handle Poller/Collect RPC executor shutdown (#8646)

Currently daemon shutdown is taking longer as orderly shutdown waits
for all the current jobs to execute properly. We need force shutdown
for RPCs as they have longer timeout


  Commit: 93da0275a4ecdfcaca502dc295107f7c7bfde1c7
      https://github.com/OpenNMS/opennms/commit/93da0275a4ecdfcaca502dc295107f7c7bfde1c7
  Author: Christian Pape <[email protected]>
  Date:   2026-07-23 (Thu, 23 Jul 2026)

  Changed paths:
    M features/minion/core/repository/src/assembly/repo.xml
    M features/minion/repository/src/assembly/repo.xml

  Log Message:
  -----------
  NMS-19961: Prevent old bcprov-jdk18on version from being included


  Commit: 4dc92b1cb234004f692fa057f82e8ad6b25a9d4f
      https://github.com/OpenNMS/opennms/commit/4dc92b1cb234004f692fa057f82e8ad6b25a9d4f
  Author: Dino <[email protected]>
  Date:   2026-07-23 (Thu, 23 Jul 2026)

  Changed paths:
    M core/grpc/osgi/pom.xml
    M core/ipc/grpc/itests/pom.xml
    M core/ipc/twin/grpc/itests/pom.xml
    M core/ipc/twin/kafka/itests/pom.xml
    M features/scv/dominion-grpc-impl/pom.xml
    M features/telemetry/protocols/openconfig/itests/pom.xml
    M pom.xml

  Log Message:
  -----------
  NMS-20047: newts delete() fix (#8655)

* NMS-20047: tick pom to newts 3.0.1 to pick up newts fix

* Newts 3.0.1 depends on newer Guava, so bump it to 33.3.1 and consolidate the guava*Version tags down


  Commit: 15ffd2fcca77cf7a9a6fd59b9f91495c4c181302
      https://github.com/OpenNMS/opennms/commit/15ffd2fcca77cf7a9a6fd59b9f91495c4c181302
  Author: Morteza E <[email protected]>
  Date:   2026-07-23 (Thu, 23 Jul 2026)

  Changed paths:
    M opennms-container/common.mk
    M opennms-container/core/Dockerfile
    M opennms-container/minion/Dockerfile
    M opennms-container/sentinel/Dockerfile
    M smoke-test/src/main/java/org/opennms/smoketest/containers/MockCloudContainer.java

  Log Message:
  -----------
  Update base images


  Commit: bc96994fa50f76789e35c9cae96cd56cf4951e8b
      https://github.com/OpenNMS/opennms/commit/bc96994fa50f76789e35c9cae96cd56cf4951e8b
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-24 (Fri, 24 Jul 2026)

  Changed paths:
    M opennms-provision/opennms-provisiond/src/main/java/org/opennms/netmgt/provision/service/NodeScan.java
    A opennms-provision/opennms-provisiond/src/test/java/org/opennms/netmgt/provision/service/NodeScanTest.java

  Log Message:
  -----------
  NMS-19096: Refresh NodeScan scan stamp on each scheduled run

A NodeScan scheduled with scheduleWithFixedDelay is reused for every
periodic run, but m_scanStamp was set once in the constructor and never
advanced. Obsolete-interface deletion compares each interface's
ipLastCapsdPoll against this stamp, so a long-lived scheduled scan kept
comparing against a stale stamp and could not reap non-primary interfaces
that had gone away; only a manual rescan, which builds a fresh NodeScan,
removed them. Advance the stamp in reset(), which already runs at the
start of every scan.


  Commit: 98c2793aca24e4c036aaf2ecbd7816ce4cf14a0f
      https://github.com/OpenNMS/opennms/commit/98c2793aca24e4c036aaf2ecbd7816ce4cf14a0f
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-24 (Fri, 24 Jul 2026)

  Changed paths:
    R opennms-provision/opennms-provisiond/src/test/java/org/opennms/netmgt/provision/service/NodeScanTest.java
    M opennms-provision/opennms-provisiond/src/test/java/org/opennms/netmgt/provision/service/ProvisionerIT.java

  Log Message:
  -----------
  NMS-19096: Replace scan-stamp unit test with a behavioral IT

The NodeScanTest only reflected reset() and asserted it reassigned
m_scanStamp, which restates the implementation rather than testing
behavior. Replace it with ProvisionerIT#testReusedNodeScanReapsVanishedInterfaceNms19096,
which reuses a single NodeScan instance across two runs (as the scheduler
does) and asserts that an SNMP-discovered interface which disappears
between runs is reaped. This fails against the frozen-stamp behavior and
passes with the fix.


  Commit: 5933d811cd2366504f08bfa595b8eb06f2f4488f
      https://github.com/OpenNMS/opennms/commit/5933d811cd2366504f08bfa595b8eb06f2f4488f
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-24 (Fri, 24 Jul 2026)

  Changed paths:
    M opennms-provision/opennms-provisiond/src/test/java/org/opennms/netmgt/provision/service/ProvisionerIT.java

  Log Message:
  -----------
  NMS-19096: Strengthen the NodeScan reap IT assertions

Address review feedback: assert the reused NodeScan advances its scan
stamp between runs (via getScanStamp()), and assert specifically that
198.51.100.204 is present after the first scan and removed after the
second while the primary 198.51.100.201 is retained.


  Commit: e5cdcd9e059d5a0a7654fee654a737a65a78be09
      https://github.com/OpenNMS/opennms/commit/e5cdcd9e059d5a0a7654fee654a737a65a78be09
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-24 (Fri, 24 Jul 2026)

  Changed paths:
    M docs/modules/deployment/pages/core/getting-started.adoc

  Log Message:
  -----------
  docs: Fix malformed system requirements xref on the core install page

The requirements bullet ended with a bare `xref:system-requirements.adoc`
with no target brackets. Without them it is not a macro at all, so
AsciiDoc rendered the raw text instead of a link.

Use the module-relative resource ID form already used elsewhere in the
deployment module (nav.adoc, air-gap/air-gap-install.adoc) and leave the
brackets empty so Antora supplies the page title.


  Commit: 27f33084d9303815c0e6aefdbe693124e8bbbd82
      https://github.com/OpenNMS/opennms/commit/27f33084d9303815c0e6aefdbe693124e8bbbd82
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-24 (Fri, 24 Jul 2026)

  Changed paths:
    M docs/modules/deployment/pages/core/getting-started.adoc

  Log Message:
  -----------
  Merge pull request #8662 from marshallmassengill/mm/docs-install-xref-f2024

docs: Fix malformed system requirements xref on the core install page


  Commit: 192f1702f6722b4e083120dd00db3c940fc80a8b
      https://github.com/OpenNMS/opennms/commit/192f1702f6722b4e083120dd00db3c940fc80a8b
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-24 (Fri, 24 Jul 2026)

  Changed paths:
    M opennms-provision/opennms-provisiond/src/main/java/org/opennms/netmgt/provision/service/NodeScan.java
    M opennms-provision/opennms-provisiond/src/test/java/org/opennms/netmgt/provision/service/ProvisionerIT.java

  Log Message:
  -----------
  Merge pull request #8661 from OpenNMS/mm/NMS-19096-smoke

NMS-19096: Refresh NodeScan scan stamp on each scheduled run


  Commit: 6f238fef7ab87a5d2299a4f35eab3008ed36a438
      https://github.com/OpenNMS/opennms/commit/6f238fef7ab87a5d2299a4f35eab3008ed36a438
  Author: Chandra Gorantla <[email protected]>
  Date:   2026-07-27 (Mon, 27 Jul 2026)

  Changed paths:
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/DiscoveryRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/DiscoveryRestServiceIT.java
    M opennms-webapp/src/main/webapp/WEB-INF/applicationContext-spring-security.xml

  Log Message:
  -----------
  NMS-20056: Fix discovery Rest to restrict endpoints to Admins (#8658)


  Commit: 951f984d5d113c20f27d4e6f8dacd56013c8eca3
      https://github.com/OpenNMS/opennms/commit/951f984d5d113c20f27d4e6f8dacd56013c8eca3
  Author: CI/CD System <[email protected]>
  Date:   2026-07-27 (Mon, 27 Jul 2026)

  Changed paths:
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/DiscoveryRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/DiscoveryRestServiceIT.java
    M opennms-webapp/src/main/webapp/WEB-INF/applicationContext-spring-security.xml

  Log Message:
  -----------
  Merge remote-tracking branch 'origin/foundation-2023' into foundation-2024


Compare: https://github.com/OpenNMS/opennms/compare/d7f5e3c385ea...951f984d5d11

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