From 60449686fb38104c4232650f7e4423d9d348c221 Mon Sep 17 00:00:00 2001 From: Jim Hahn Date: Tue, 27 Aug 2019 14:37:50 -0400 Subject: Remove AlertSeverity from logback in drools-pdp Merged severity and AlertSeverity in the MDC, and updated the logback xml files accordingly. The logback patterns for the metrics log had a place for both Severity and AlertSeverity; the latter was replaced with an empty place-holder so-as to maintain the same position for the other fields. Change-Id: Ic0977e1133cc4219394f2d71ca8ebae080620e0b Issue-ID: POLICY-1989 Signed-off-by: Jim Hahn --- .../drools/utils/logging/MdcTransaction.java | 10 --------- .../utils/logging/MdcTransactionConstants.java | 5 ----- .../drools/utils/logging/MdcTransactionImpl.java | 25 ++++------------------ 3 files changed, 4 insertions(+), 36 deletions(-) (limited to 'policy-utils/src/main/java/org') diff --git a/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransaction.java b/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransaction.java index d56e63e6..a2cf07fb 100644 --- a/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransaction.java +++ b/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransaction.java @@ -186,11 +186,6 @@ public interface MdcTransaction { */ MdcTransaction setProcessKey(String processKey); - /** - * sets alert severity. - */ - MdcTransaction setAlertSeverity(String alertSeverity); - /** * sets client ip address. */ @@ -231,11 +226,6 @@ public interface MdcTransaction { */ String getClientIpAddress(); - /** - * get alert severity. - */ - String getAlertSeverity(); - /** * get process key. */ diff --git a/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionConstants.java b/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionConstants.java index ba335958..eb37d0c7 100644 --- a/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionConstants.java +++ b/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionConstants.java @@ -138,11 +138,6 @@ public class MdcTransactionConstants { */ public static final String REMOTE_HOST = "RemoteHost"; - /** - * Alert Severity Both transactions and subtransactions will have its own copy. - */ - public static final String ALERT_SEVERITY = "AlertSeverity"; - /** * Target Virtual Entity Both transactions and subtransactions will have its own copy. */ diff --git a/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionImpl.java b/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionImpl.java index ac90ab96..4e1690ba 100644 --- a/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionImpl.java +++ b/policy-utils/src/main/java/org/onap/policy/drools/utils/logging/MdcTransactionImpl.java @@ -20,7 +20,6 @@ package org.onap.policy.drools.utils.logging; -import static org.onap.policy.drools.utils.logging.MdcTransactionConstants.ALERT_SEVERITY; import static org.onap.policy.drools.utils.logging.MdcTransactionConstants.BEGIN_TIMESTAMP; import static org.onap.policy.drools.utils.logging.MdcTransactionConstants.CLIENT_IP_ADDRESS; import static org.onap.policy.drools.utils.logging.MdcTransactionConstants.DEFAULT_HOSTIP; @@ -94,7 +93,6 @@ class MdcTransactionImpl implements MdcTransaction { private String statusCode; private String responseCode; private String responseDescription; - private String theSeverity; private String alertSeverity; private String targetEntity; @@ -158,7 +156,6 @@ class MdcTransactionImpl implements MdcTransaction { */ public MdcTransactionImpl(MdcTransaction transaction) { MDC.clear(); - this.setAlertSeverity(transaction.getAlertSeverity()); this.setClientIpAddress(transaction.getClientIpAddress()); this.setElapsedTime(transaction.getElapsedTime()); this.setEndTime(transaction.getEndTime()); @@ -205,7 +202,6 @@ class MdcTransactionImpl implements MdcTransaction { MDC.remove(PROCESS_KEY); MDC.remove(CLIENT_IP_ADDRESS); MDC.remove(REMOTE_HOST); - MDC.remove(ALERT_SEVERITY); MDC.remove(TARGET_VIRTUAL_ENTITY); return this; @@ -246,8 +242,7 @@ class MdcTransactionImpl implements MdcTransaction { setMdc(STATUS_CODE, this.statusCode); setMdc(RESPONSE_CODE, this.responseCode); setMdc(RESPONSE_DESCRIPTION, this.responseDescription); - setMdc(SEVERITY, this.theSeverity); - setMdc(ALERT_SEVERITY, this.alertSeverity); + setMdc(SEVERITY, this.alertSeverity); setMdc(TARGET_ENTITY, this.targetEntity); setMdc(TARGET_SERVICE_NAME, this.targetServiceName); setMdc(TARGET_VIRTUAL_ENTITY, this.targetVirtualEntity); @@ -305,12 +300,6 @@ class MdcTransactionImpl implements MdcTransaction { return this; } - @Override - public MdcTransaction setAlertSeverity(String alertSeverity) { - this.alertSeverity = alertSeverity; - return this; - } - @Override public MdcTransaction setClientIpAddress(String clientIpAddress) { this.clientIpAddress = clientIpAddress; @@ -353,11 +342,6 @@ class MdcTransactionImpl implements MdcTransaction { return this.clientIpAddress; } - @Override - public String getAlertSeverity() { - return this.alertSeverity; - } - @Override public String getProcessKey() { return this.processKey; @@ -451,7 +435,7 @@ class MdcTransactionImpl implements MdcTransaction { @Override public MdcTransaction setSeverity(String severity) { - this.theSeverity = severity; + this.alertSeverity = severity; return this; } @@ -500,7 +484,7 @@ class MdcTransactionImpl implements MdcTransaction { @Override public String getSeverity() { - return theSeverity; + return alertSeverity; } @Override @@ -651,8 +635,7 @@ class MdcTransactionImpl implements MdcTransaction { sb.append(", statusCode='").append(statusCode).append('\''); sb.append(", responseCode='").append(responseCode).append('\''); sb.append(", responseDescription='").append(responseDescription).append('\''); - sb.append(", severity='").append(theSeverity).append('\''); - sb.append(", alertSeverity='").append(alertSeverity).append('\''); + sb.append(", severity='").append(alertSeverity).append('\''); sb.append(", targetEntity='").append(targetEntity).append('\''); sb.append(", targetServiceName='").append(targetServiceName).append('\''); sb.append(", targetVirtualEntity='").append(targetVirtualEntity).append('\''); -- cgit 1.2.3-korg