From f74bbc97aefb9c7e550b8e123581f1e4fa78f84d Mon Sep 17 00:00:00 2001 From: Jorge Hernandez Date: Mon, 7 May 2018 20:26:19 -0500 Subject: [PATCH] fix PE DB lock timeout problems see companion docker review. The failOverReadOnly .. flags should only be used when a DB high availability environment is used, such as a master/slave configuration. As they are the jdbc parameters have caused bad interactions with the underlying persistence libraries rendering the PAP useless. It is thought that this may perhaps solve POLICY-773, POLICY-775, POLICY-779, and perhaps POLICY-776. Change-Id: Icb69a3afc0a19969a34af40d960fa8e1b94aabc0 Issue-ID: POLICY-777 Signed-off-by: Jorge Hernandez --- .../test/java/org/onap/policy/pap/xacml/rest/PAPRestConfigTest.java | 2 +- ONAP-PAP-REST/xacml.pap.properties | 4 ++-- ONAP-SDK-APP/xacml.admin.properties | 4 ++-- POLICY-SDK-APP/src/test/resources/xacml.admin.properties | 4 ++-- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/ONAP-PAP-REST/src/test/java/org/onap/policy/pap/xacml/rest/PAPRestConfigTest.java b/ONAP-PAP-REST/src/test/java/org/onap/policy/pap/xacml/rest/PAPRestConfigTest.java index 35540b3a2..b57c187c7 100644 --- a/ONAP-PAP-REST/src/test/java/org/onap/policy/pap/xacml/rest/PAPRestConfigTest.java +++ b/ONAP-PAP-REST/src/test/java/org/onap/policy/pap/xacml/rest/PAPRestConfigTest.java @@ -45,7 +45,7 @@ public class PAPRestConfigTest { @Test public void testInit() { String driver = "org.mariadb.jdbc.Driver"; - String url = "jdbc:mariadb://localhost:3306/onap_sdk?failOverReadOnly=false&autoReconnect=true"; + String url = "jdbc:mariadb://localhost:3306/onap_sdk?connectTimeout=30000&socketTimeout=60000&log=true&sessionVariables=max_statement_time=30"; String user = "policy_user"; String password = "policy_user"; diff --git a/ONAP-PAP-REST/xacml.pap.properties b/ONAP-PAP-REST/xacml.pap.properties index dc8300d92..bd0fed40c 100644 --- a/ONAP-PAP-REST/xacml.pap.properties +++ b/ONAP-PAP-REST/xacml.pap.properties @@ -89,7 +89,7 @@ xacml.rest.pdp.idfile=test.properties #Properties for db access #properties for MySql xacml database: PLEASE DO NOT REMOVE... NEEDED FOR APIs javax.persistence.jdbc.driver=org.mariadb.jdbc.Driver -javax.persistence.jdbc.url=jdbc:mariadb://localhost:3306/onap_sdk?failOverReadOnly=false&autoReconnect=true +javax.persistence.jdbc.url=jdbc:mariadb://localhost:3306/onap_sdk?connectTimeout=30000&socketTimeout=60000&log=true&sessionVariables=max_statement_time=30 javax.persistence.jdbc.user=policy_user javax.persistence.jdbc.password=policy_user @@ -158,4 +158,4 @@ CLIENT_KEY= #Micro Service Model Properties xacml.policy.msOnapName=http://org.onap -xacml.policy.msPolicyName=http://org.onap.policy \ No newline at end of file +xacml.policy.msPolicyName=http://org.onap.policy diff --git a/ONAP-SDK-APP/xacml.admin.properties b/ONAP-SDK-APP/xacml.admin.properties index 6d6570fe9..8e6852805 100644 --- a/ONAP-SDK-APP/xacml.admin.properties +++ b/ONAP-SDK-APP/xacml.admin.properties @@ -81,7 +81,7 @@ xacml.testEnvironment.loginId = #Log DB properties xacml.log.db.driver=org.mariadb.jdbc.Driver -xacml.log.db.url=jdbc:mariadb://localhost:3306/log?failOverReadOnly=false&autoReconnect=true +xacml.log.db.url=jdbc:mariadb://localhost:3306/log?connectTimeout=30000&socketTimeout=60000&log=true&sessionVariables=max_statement_time=30 xacml.log.db.user=policy_user xacml.log.db.password=policy_user @@ -204,4 +204,4 @@ xacml.policy.msOnapName=http://org.onap xacml.policy.msPolicyName=http://org.onap.policy #Size limit (in bytes) for file uploads -file.size.limit=30000000 \ No newline at end of file +file.size.limit=30000000 diff --git a/POLICY-SDK-APP/src/test/resources/xacml.admin.properties b/POLICY-SDK-APP/src/test/resources/xacml.admin.properties index 333da49a5..363ab621f 100644 --- a/POLICY-SDK-APP/src/test/resources/xacml.admin.properties +++ b/POLICY-SDK-APP/src/test/resources/xacml.admin.properties @@ -81,7 +81,7 @@ xacml.testEnvironment.loginId = #Log DB properties xacml.log.db.driver=org.mariadb.jdbc.Driver -xacml.log.db.url=jdbc:mariadb://localhost:3306/log?failOverReadOnly=false&autoReconnect=true +xacml.log.db.url=jdbc:mariadb://localhost:3306/log?connectTimeout=30000&socketTimeout=60000&log=true&sessionVariables=max_statement_time=30 xacml.log.db.user=policy_user xacml.log.db.password=policy_user @@ -200,4 +200,4 @@ policyAdapter.impl.className = org.onap.policy.admin.PolicyAdapter #Micro Service Model Properties xacml.policy.msOnapName=http://org.onap -xacml.policy.msPolicyName=http://org.onap.policy \ No newline at end of file +xacml.policy.msPolicyName=http://org.onap.policy -- 2.16.6