Fix Duplicate Scheduling of Index Rollover in ADIndexManagement

ADIndexManagement inherits from IndexManagement, which implements LocalNodeClusterManagerListener, causing it to listen for master node changes. This listener schedules index rollover and deletion. However, ADIndexManagement was inadvertently listening to the cluster manager event again, leading to duplicate scheduling of these tasks.

This commit removes the redundant listener call in ADIndexManagement, ensuring that index rollover and deletion are scheduled only once per interval.

Testing:

Verified that index rollover and deletion now execute once per interval as expected.

Signed-off-by: Kaituo Li <[email protected]>

src/main/java/org/opensearch/ad/indices/ADIndexManagement.java

@@ -96,7 +96,6 @@ public ADIndexManagement(
             AnomalyDetector::parse,
             ADCommonName.CUSTOM_RESULT_INDEX_PREFIX + "*"
         );
-        this.clusterService.addLocalNodeClusterManagerListener(this);
 
         this.indexStates = new EnumMap<ADIndex, IndexState>(ADIndex.class);