[OpenNMS/opennms] 8f35e8: NMS-20006: bound local RPC execution to a fixed, c...

  Branch: refs/heads/foundation-2024
  Home:   https://github.com/OpenNMS/opennms
  Commit: 8f35e86a6fa16f1a45ba33d151bace5414aebdf5
      https://github.com/OpenNMS/opennms/commit/8f35e86a6fa16f1a45ba33d151bace5414aebdf5
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-21 (Tue, 21 Jul 2026)

  Changed paths:
    M core/ipc/rpc/api/src/main/java/org/opennms/core/rpc/api/RpcExceptionUtils.java
    M core/ipc/rpc/camel/src/main/java/org/opennms/core/rpc/camel/CamelRpcClientFactory.java
    A core/ipc/rpc/camel/src/test/java/org/opennms/core/rpc/camel/LocalRpcTimeoutTest.java
    M docs/modules/reference/pages/configuration/ttl-rpc.adoc

  Log Message:
  -----------
  NMS-20006: bound local RPC execution to a fixed, configurable timeout

Local (same-location) RPC execution in CamelRpcClientFactory is bounded
again so a module future that never completes cannot wedge its caller
(the NMS-19951 goal), but the bound is no longer the request TTL: it is
a fixed timeout, 30 minutes by default, configurable via the
org.opennms.core.ipc.rpc.local.timeout system property (0 disables).
TTL values resolved from snmp-config, service parameters, or the
collection interval no longer affect local execution, so
slow-but-completing local operations cannot be failed by configuration
intended for the Minion path.

RpcExceptionUtils now maps java.util.concurrent.TimeoutException to
onTimedOut, so an expired local execution is reported as a timeout
(e.g. CollectionTimedOut) rather than as an unexpected exception.


  Commit: 572db79e320b4841376b5eb58dc9b921a68aaac2
      https://github.com/OpenNMS/opennms/commit/572db79e320b4841376b5eb58dc9b921a68aaac2
  Author: Marshall Massengill <[email protected]>
  Date:   2026-07-22 (Wed, 22 Jul 2026)

  Changed paths:
    M core/ipc/rpc/api/src/main/java/org/opennms/core/rpc/api/RpcExceptionUtils.java
    M core/ipc/rpc/camel/src/main/java/org/opennms/core/rpc/camel/CamelRpcClientFactory.java
    A core/ipc/rpc/camel/src/test/java/org/opennms/core/rpc/camel/LocalRpcTimeoutTest.java
    M docs/modules/reference/pages/configuration/ttl-rpc.adoc

  Log Message:
  -----------
  Merge pull request #8641 from OpenNMS/mm/NMS-20006-smoke

NMS-20006: bound local RPC execution with a timeout decoupled from the TTL


Compare: https://github.com/OpenNMS/opennms/compare/ffb62839abb6...572db79e320b

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