Need upgrade task for collection strategy classes

Description

During dependency refactoring in 16.0.0, the data collection strategy classes moved out of opennms-dao and opennms-services and into the features/collection/api bundle. This makes the dependencies much cleaner but also requires updates to all of the data collection config files to point to the new names for the classes.

The commit where this change was made was aa99aea5ed190b535e552eb297fec32e8e170c8e and the diff inside the data collection configs is like:

<resourceType resourceLabel="${clvLinkMonGrpName} (${index})" label="Link monitor groups" name="clvLinkMonGrpEntry"> - <persistenceSelectorStrategy class="org.opennms.netmgt.collectd.PersistAllSelectorStrategy"/> - <storageStrategy class="org.opennms.netmgt.dao.support.IndexStorageStrategy"/> + <persistenceSelectorStrategy class="org.opennms.netmgt.collection.support.PersistAllSelectorStrategy"/> + <storageStrategy class="org.opennms.netmgt.collection.support.IndexStorageStrategy"/> </resourceType>

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Alejandro Galue June 9, 2015 at 1:14 PM

In case someone missed my comment on https://opennms.atlassian.net/browse/NMS-7724#icft=NMS-7724:

I've created another JUnit test to verify it and it works for all the files inside /opt/opennms/etc/datacollection, but not on datacollection-config.xml. Even if no one should add content directly on that file, it is still possible. I mean, you can have resource-types defined directly on datacollection-config.xml even if it is not recommended.

Jesse White April 20, 2015 at 7:11 PM

Fixed in c0ad255d416d3534e1f41370859e51b9accc32c6.

Fixed

Details

Assignee

Reporter

Labels

Sprint

Fix versions

Affects versions

Priority

PagerDuty

Created April 14, 2015 at 10:47 AM
Updated October 12, 2023 at 4:30 PM
Resolved April 20, 2015 at 7:11 PM

Flag notifications