support control loop transaction metrics
Issue-ID: POLICY-3033
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
Change-Id: Ic13275607a56b36506a0c79d1880603ff374b0c2
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
diff --git a/controlloop/common/feature-controlloop-trans/src/main/java/org/onap/policy/drools/apps/controlloop/feature/trans/CacheBasedControlLoopMetricsManager.java b/controlloop/common/feature-controlloop-trans/src/main/java/org/onap/policy/drools/apps/controlloop/feature/trans/CacheBasedControlLoopMetricsManager.java
index 956d28e..468c2ea 100644
--- a/controlloop/common/feature-controlloop-trans/src/main/java/org/onap/policy/drools/apps/controlloop/feature/trans/CacheBasedControlLoopMetricsManager.java
+++ b/controlloop/common/feature-controlloop-trans/src/main/java/org/onap/policy/drools/apps/controlloop/feature/trans/CacheBasedControlLoopMetricsManager.java
@@ -2,7 +2,7 @@
* ============LICENSE_START=======================================================
* ONAP
* ================================================================================
- * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2021 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -29,6 +29,7 @@
import java.util.ArrayList;
import java.util.List;
import java.util.Map;
+import java.util.Objects;
import java.util.Properties;
import java.util.UUID;
import java.util.concurrent.TimeUnit;
@@ -38,6 +39,7 @@
import org.onap.policy.controlloop.VirtualControlLoopNotification;
import org.onap.policy.drools.persistence.SystemPersistenceConstants;
import org.onap.policy.drools.system.PolicyController;
+import org.onap.policy.drools.system.PolicyEngineConstants;
import org.onap.policy.drools.utils.logging.MdcTransaction;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -163,7 +165,7 @@
case FINAL_FAILURE:
case FINAL_SUCCESS:
case FINAL_OPENLOOP:
- endTransaction(notification);
+ endTransaction(controller, notification);
break;
case ACTIVE:
case OPERATION:
@@ -224,18 +226,15 @@
/**
* End of a control loop transaction.
*
+ * @param controller controller
* @param notification control loop notification
*/
- protected void endTransaction(VirtualControlLoopNotification notification) {
+ protected void endTransaction(PolicyController controller, VirtualControlLoopNotification notification) {
ZonedDateTime startTime;
VirtualControlLoopNotification startNotification = cache.getIfPresent(notification.getRequestId());
- if (startNotification != null) {
- startTime = startNotification.getNotificationTime();
- } else {
- startTime = notification.getNotificationTime();
- }
+ startTime = Objects.requireNonNullElse(startNotification, notification).getNotificationTime();
- this.transaction(notification, startTime);
+ this.transaction(controller, notification, startTime);
if (startNotification != null) {
removeTransaction(startNotification.getRequestId());
}
@@ -349,7 +348,8 @@
return trans;
}
- protected void transaction(VirtualControlLoopNotification notification, ZonedDateTime startTime) {
+ protected void transaction(PolicyController controller,
+ VirtualControlLoopNotification notification, ZonedDateTime startTime) {
MdcTransaction trans = getMdcTransaction(notification)
.setStartTime(startTime.toInstant())
.setEndTime(notification.getNotificationTime().toInstant());
@@ -372,6 +372,13 @@
break;
}
+ try {
+ PolicyEngineConstants.getManager().transaction(controller.getName(),
+ notification.getClosedLoopControlName(), trans.getMetric());
+ } catch (RuntimeException rex) {
+ logger.info("error pegging control loop transaction: {}", trans.getMetric(), rex);
+ }
+
trans.transaction().resetTransaction();
}
@@ -387,10 +394,6 @@
private String notificationTypeToResponseCode(String notificationType) {
String code = note2code.get(notificationType);
- if (code != null) {
- return code;
- } else {
- return UNKNOWN_RESPONSE_CODE;
- }
+ return Objects.requireNonNullElse(code, UNKNOWN_RESPONSE_CODE);
}
}
diff --git a/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java b/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java
index a19006e..480c8d4 100644
--- a/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java
+++ b/controlloop/common/feature-controlloop-trans/src/test/java/org/onap/policy/drools/apps/controlloop/feature/trans/ControlLoopMetricsFeatureTest.java
@@ -2,7 +2,7 @@
* ============LICENSE_START=======================================================
* ONAP
* ================================================================================
- * Copyright (C) 2018-2020 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2018-2021 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -32,6 +32,7 @@
import java.util.UUID;
import java.util.concurrent.TimeUnit;
import org.junit.AfterClass;
+import org.junit.Before;
import org.junit.BeforeClass;
import org.junit.Test;
import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
@@ -52,6 +53,17 @@
private static final Path configPath = SystemPersistenceConstants.getManager().getConfigurationPath();
private static PolicyController testController;
+ private static void resetStats() {
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setAverageExecutionTime(0d);
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setLastExecutionTime(0L);
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setLastStart(0L);
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setAverageExecutionTime(0d);
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setPolicyExecutedCount(0L);
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setPolicyExecutedFailCount(0L);
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setPolicyExecutedSuccessCount(0L);
+ PolicyEngineConstants.getManager().getStats().getGroupStat().setTotalElapsedTime(0d);
+ }
+
/**
* Setup method.
*/
@@ -65,6 +77,12 @@
@AfterClass
public static void tearDown() {
SystemPersistenceConstants.getManager().setConfigurationDir(configPath.toString());
+ resetStats();
+ }
+
+ @Before
+ public void beforeTest() {
+ resetStats();
}
@Test
@@ -231,6 +249,19 @@
Serialization.gsonPretty.fromJson(finalSuccessNotification, VirtualControlLoopNotification.class);
feature.beforeDeliver(testController, CommInfrastructure.DMAAP, POLICY_CL_MGT, finalSuccess);
assertEquals(0, ControlLoopMetricsManager.getManager().getTransactionIds().size());
+ assertEquals(1,
+ PolicyEngineConstants.getManager().getStats().getGroupStat().getPolicyExecutedSuccessCount());
+ assertEquals(0,
+ PolicyEngineConstants.getManager().getStats().getGroupStat().getPolicyExecutedFailCount());
+ assertEquals(1, PolicyEngineConstants.getManager().getStats().getGroupStat().getPolicyExecutedCount());
+ assertEquals(1587409937684L,
+ PolicyEngineConstants.getManager().getStats().getGroupStat().getLastExecutionTime());
+ assertEquals(461d,
+ PolicyEngineConstants.getManager().getStats().getGroupStat().getAverageExecutionTime(), 0.0d);
+ assertEquals(1587409937223L,
+ PolicyEngineConstants.getManager().getStats().getGroupStat().getLastStart());
+ assertEquals(461d,
+ PolicyEngineConstants.getManager().getStats().getGroupStat().getTotalElapsedTime(), 0.0d);
}
@Test
@@ -255,4 +286,5 @@
Thread.sleep((ControlLoopMetricsManager.getManager().getTransactionTimeout() + 1) * 1000L); // NOSONAR
assertEquals(0, ControlLoopMetricsManager.getManager().getTransactionIds().size());
}
+
}