[OpenNMS/opennms] 497936: Docs updates
Dino via opennms-cvs <[email protected]>
| Newsgroups | gmane.network.opennms.cvs |
|---|---|
| Message-ID | <OpenNMS/opennms/push/refs/heads/dcy/no-beanshell/[email protected]> |
Branch: refs/heads/dcy/no-beanshell
Home: https://github.com/OpenNMS/opennms
Commit: 497936b2b5736f8f8d52d5df04e06e2f75d8c548
https://github.com/OpenNMS/opennms/commit/497936b2b5736f8f8d52d5df04e06e2f75d8c548
Author: Dino <[email protected]>
Date: 2026-08-07 (Fri, 07 Aug 2026)
Changed paths:
M dependencies/spring-security/pom.xml
M docs/modules/operation/pages/deep-dive/events/scriptd.adoc
M docs/modules/reference/pages/provisioning/detectors/BsfDetector.adoc
M docs/modules/reference/pages/service-assurance/monitors/BSFMonitor.adoc
Log Message:
-----------
Docs updates
BSFMonitor.adoc (-52 lines net)
- Migration note near the top: Groovy is what ships, BeanShell is gone, rewrite .bsh scripts and update bsf-engine/lang-class.
- BeanShell dropped from the bsf-engine parameter list, the lang-class description, and the language-settings table.
- Entire "BeanShell example" section removed.
- Corrected a now-wrong install instruction: the Groovy section told users to "copy a compatible groovy-all.jar into $OPENNMS_HOME/lib and restart." Groovy ships with OpenNMS, and groovy-all hasn't been the right artifact since Groovy 2.5 split into modules. Now reads "ships with {page-component-title}, so no additional library
is required."
- Jython kept as-is; BSF still supports it and it genuinely does need a user-supplied jar.
BsfDetector.adoc (-43 lines net)
- Same migration note, adapted to the detector's bsfEngine/langClass parameter names.
- BeanShell removed from both parameter choice lists and the language table; "BeanShell example" section removed.
- The IMPORTANT about placing a library in lib now applies to Jython only, since it was previously scoped to "Groovy or Jython."
Two unrelated defects I fixed while in the files
- Three Groovy blocks in BSFMonitor.adoc were tagged [source, java], so they rendered with Java highlighting. Now [source, groovy].
- The generic response-time-tracking snippet used rrd-base-name of minimalbshbase — a BeanShell-flavored name in a language-neutral section. Renamed to myscriptbase.
Verification
All five Groovy blocks across the two pages extracted, callout markers stripped, and compiled with GroovyShell.parse against a classpath including both opennms-services and features/poller/monitors/core — all pass, so even the advanced example's reference to org.opennms.netmgt.poller.monitors.BSFMonitor resolves. Both pages
render through asciidoctor with exit 0 and no warnings, which also confirms the 4-column tables survived row removal.
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