aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2017-09-27Refinements to coreGaurav Agrawal4-326/+324
Changes includes: 1. Simplifying the test cases. 2. Critical Sonar issue fix for MdsalHelper 3. Critical sonar issue fix for SvcLogicServiceImpl 4. Use logger in-built formatting. https://sonar.onap.org/component_issues/index?id=org.onap.ccsdk.sli.core%3Accsdk-sli-core#resolved=false|severities=CRITICAL Change-Id: Ib85637fcd2351970f8d96bf6b855db05cf7d9834 Issue-Id: CCSDK-67 Signed-off-by: Gaurav Agrawal <gaurav.agrawal@huawei.com>
2017-09-27Merge "Use Long.toString instead"Dan Timoney1-1/+1
2017-09-27Merge "Some minor refinements in CCSDK SLI Core module"Dan Timoney2-2/+9
2017-09-27Merge "Remove hardcoded dependency versions"Dan Timoney9-9/+10
2017-09-27Merge "Fixes for SvcLogicActivator"Dan Timoney1-150/+141
2017-09-27Use Long.toString insteadsurya-huawei1-1/+1
Major sonar issue in sli/core module *Using static method to reduce memory and cycles Issue-Id: CCSDK-87 Change-Id: Iaf255892a4bd698934f865a707e54f8e740370ab Signed-off-by: surya-huawei <a.u.surya@huawei.com>
2017-09-26Updated DBLIB svc definitionRich Tabedzki1-1/+2
Changes made: 1. updated DBLibService definition by adding javax.sql.DataSource Change-Id: I5dd63a81420f451b0ce9bbbc878d351d62ccd819 Issue-ID: CCSDK-102 Signed-off-by: Rich Tabedzki <richard.tabedzki@att.com>
2017-09-25Some minor refinements in CCSDK SLI Core moduleramu.n2-2/+9
*Remove useless braces while logging exception stack trace *Register all DG nodes in SvcLogicParser Test file Change-Id: I7916bec20e36e1def15c24806da2f7e01e79bf4b Issue-Id: CCSDK-106 Signed-off-by: Ramu N <ramu.n@huawei.com>
2017-09-25Remove hardcoded dependency versionsGaurav Agrawal9-9/+10
Remove the hardcoded junit & common-lang version and rather take from parent pom. Change-Id: Ibc71ff4b86a2fefc89365017f06333776cb2fed3 Issue-Id: CCSDK-105 Signed-off-by: Gaurav Agrawal <gaurav.agrawal@huawei.com>
2017-09-25Fixes for SvcLogicActivatorGaurav Agrawal1-150/+141
1. Convert tabs to 4 spaces. 2. Break lines to be less than 120 characters 3. Remove extraneous spaces/parenthesis & redundant assignments. 4. Use logger built-in string format. 5. Fix critical sonar issues. https://sonar.onap.org/component_issues/index?id=org.onap.ccsdk.sli.core%3Accsdk-sli-core#resolved=false|severities=CRITICAL Change-Id: Idb348f125e92f0e9be30847c41b385411220fd96 Issue-Id: CCSDK-67 Signed-off-by: Gaurav Agrawal <gaurav.agrawal@huawei.com>
2017-09-23Some minor refinements in CCSDK SLI Core moduleramu.n1-16/+15
Fix few Critical/Major sonar issues in CCSDK SLI Core module * Define and throw a dedicated exception instead of using a generic one * Either log or rethrow the exception * Change "try" to a try-with-resources * Remove useless assignment to local variable * Use built-in logger method string formatting instead of string concatination * code indentation Change-Id: I83dc1fb931b8021118e259c722f7dfbc1621dce8 Issue-Id: CCSDK-67 Signed-off-by: Ramu N <ramu.n@huawei.com>
2017-09-22Changes to fix failing utils featurePatrick Brady2-3/+3
Change-Id: I88d30aaf49da78008c62352d398323fbc0d4dd5b Signed-off-by: Patrick Brady <pb071s@att.com> Issue-id: CCSDK-102
2017-09-22Merge "Add "@Override" annotation"Dan Timoney1-0/+2
2017-09-22Merge "Resolved sonar issue."Dan Timoney2-347/+345
2017-09-22Merge "Sql query blocker issue fixed."Dan Timoney2-729/+627
2017-09-22Add "@Override" annotationsurya-huawei1-0/+2
Added @Override annotation before method signature *This is to improve the readability of code Issue-Id: CCSDK-87 Change-Id: I979eb03ec81b9868012e5c16f48b595c8e53e26e Signed-off-by: surya-huawei <a.u.surya@huawei.com>
2017-09-21Install sdnc-core utils as karaf packageDan Timoney21-43/+501
New utils jar was added in another change, but wasn't bundled as a karaf feature and was not being installed in container. This broke odlsli docker container. This change wraps that util package as a new feature (sdnc-core-utils). Change-Id: Ie8c1d608e2c67aa1635837bc400cec942761c81c Issue-ID: CCSDK-54 Signed-off-by: Dan Timoney <dtimoney@att.com>
2017-09-21Sql query blocker issue fixed.Bharat saraswal2-729/+627
Rdundant code and complextity of methods reduced. @override annotaiton added. Exception handling and logging impoved. Issue-ID:CCSDK-87 Change-Id: I3907be73a2f6ccdfadeaed79c87705ae7afdb4ab Signed-off-by: Bharat saraswal <bharat.saraswal@huawei.com>
2017-09-21Resolved sonar issue.Bharat saraswal2-347/+345
Related to exception handling. @Override method Redundant code and resulable parameters Issue-Id:CCSDK-87 Change-Id: I82e2aad453edf5cd647c87322fc75b2a56c37566 Signed-off-by: Bharat saraswal <bharat.saraswal@huawei.com>
2017-09-21Merge "Improve code coverage for MdsalHelper"Dan Timoney3-78/+309
2017-09-21Merge "Remove unnecessary parentheses"Dan Timoney1-18/+18
2017-09-21Improve code coverage for MdsalHelperGaurav Agrawal3-78/+309
Changes includes 1. Modified pom to take JUNIT version from parent pom. 2. Added basic testcases for MdsalHelper utilities with test YANG. 3. Made necessary pom additions. Change-Id: Ib46bd9c8a87db33c90aaaf36736d709a7c17cbdd Issue-Id: CCSDK-101 Signed-off-by: Gaurav Agrawal <gaurav.agrawal@huawei.com>
2017-09-21Remove unnecessary parenthesessurya-huawei1-18/+18
*Useless parentheses can sometimes be misleading and hence removed Issue-Id: CCSDK-87 Change-Id: I018d791500a42356296d07d04de45517d061bb7f Signed-off-by: surya-huawei <a.u.surya@huawei.com>
2017-09-21Fix Sonar Issues in sli/core moduleBharat saraswal4-262/+249
Few major issues *Remove unused variables To increase code readability *Replace the type specification with the diamond operator *Add the "@Override" annotation above this method signature To increase code readability" Issue-Id: CCSDK-87 Change-Id: I2f0c16a83ab10a0a3e39a2a1d4db3c195ec2bba8 Signed-off-by: Bharat saraswal <bharat.saraswal@huawei.com>
2017-09-20Merge "Abstract utility classes for re-use"Dan Timoney15-59/+154
2017-09-20Merge "Fix Sonar Issues in sli/core module"Dan Timoney1-19/+20
2017-09-20Merge "Fixes for MdsalHelper util"Dan Timoney1-1002/+1062
2017-09-20Merge "Fix Sonar Issue"Dan Timoney1-9/+1
2017-09-20Abstract utility classes for re-useRyan Goulding15-59/+154
When looking at the existing uses of BundleActivator(s), it became immediately clear that other parts of the code have a strategy based properties file resolution very similar to those abstracted by dblib. This change aggregates a separate utils bundle for this functionality, which is further abstracted for potential reuse with other use cases. The next use case is in the sli bundle, which will be handled in a follow-up patch. Issue-Id: SDNC-54 Change-Id: Ie4d4bb679617474b1983e6044c08cca1742b9893 Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
2017-09-20Fix Sonar Issues in sli/core modulesurya-huawei1-19/+20
Few major issues *Remove unused imports To increase code readability *Replace the type specification with the diamond operator To reduce the verbosity of generics code *Add the "@Override" annotation above this method signature To increase code readability *Remove this unnecessary cast to "String" Unnecessary casting expressions pollutes code Issue-Id: CCSDK-87 Change-Id: I38952f5026a5c61af73ac16706e8ddc278566b13 Signed-off-by: surya-huawei <a.u.surya@huawei.com>
2017-09-20Fixes for MdsalHelper utilGaurav Agrawal1-1002/+1062
1. Convert tabs to 4 spaces. 2. Break lines to be less than 120 characters 3. Remove extraneous spaces and parenthesis 4. Fixed blocker/critical sonar issues. https://sonar.onap.org/component_issues/index?id=org.onap.ccsdk.sli.core%3Accsdk-sli-core#resolved=false|severities=BLOCKER%2CCRITICAL Change-Id: I8b40784ecb178409a6fe9a00d28ccd03873044c5 Issue-Id: CCSDK-67 Signed-off-by: Gaurav Agrawal <gaurav.agrawal@huawei.com>
2017-09-20Fix Sonar Issuesurya-huawei1-9/+1
One blocker issue in sli/core module *Check for null dereferencing Issue-Id: CCSDK-87 Change-Id: I5ebaef24b3fd6534b5e818c3087e3d8213ed770c Signed-off-by: surya-huawei <a.u.surya@huawei.com>
2017-09-20Fix few Major sonar issuesramu.n1-45/+44
Fix few Major sonar issues in CCSDK SLI Core module * Remove useless assignment to local variable * Use built-in logger method string formatting instead of string concatination * code indentation Change-Id: I20fc9d8f4c510f6169e6f989c6f5595371e5f2dd Issue-Id: CCSDK-67 Signed-off-by: Ramu N <ramu.n@huawei.com>
2017-09-19Merge "Use Junit4 instead of Junit3 in sli/common"Dan Timoney2-54/+85
2017-09-19Remove mysql dependencies in sli/coreDan Timoney7-251/+27
Remove lingering mysql dependencies in sli/core, which prevented odlsli container from starting up clean. Note: tested this using bareodl container and verified that these changes install clean AFTER changing incorrect driver property name in dblib.properties. This patch is fine, but a second patch to ccsdk/distribution is needed to fix that property file. Change-Id: I6a900d1c29998867bfbae29d5e113c4e392a8bb9 Issue-ID: CCSDK-95 Signed-off-by: Dan Timoney <dtimoney@att.com>
2017-09-20Use Junit4 instead of Junit3 in sli/commonramu.n2-54/+85
Modified sli/common pom.xml file to take Junit version from parent pom file and modified one Testfile as per Junit4 Change-Id: Iee4397125d9873def60c77ba8be5d541e0df62fa Issue-Id: CCSDK-98 Signed-off-by: Ramu N <ramu.n@huawei.com>
2017-09-19Merge "Fix few Major sonar issues"Dan Timoney1-297/+314
2017-09-19Merge "Fix few Major sonar issues"Dan Timoney3-56/+56
2017-09-19Fix mariadb-connector-java referencesDan Timoney1-1/+1
Remove mariadb-connector-java reference in dblib features.xml Change-Id: I29f7466a82b25e35b180eb48911dd553c86460fa Issue-ID: CCSDK-95 Signed-off-by: Dan Timoney <dtimoney@att.com>
2017-09-18Fix mariadb-connector-java referencesDan Timoney2-2/+2
Correct mariadb JDBC connector artifactId should be mariadb-java-client, not mariadb-connector-java Change-Id: If58a33a1ef22d47e1d5c138bad164b0ad89219eb Issue-ID: CCSDK-95 Signed-off-by: Dan Timoney <dtimoney@att.com>
2017-09-18Update sli/core to use Mariadb connectoriDan Timoney6-15/+14
Update sli/core (including dblib, notably) to use mariadb-connector for any jUnit tests. Update Import-Package in dblib provider pom.xml to import org.mariadb.jdbc so that org.mariadb.jdbc.Driver will be resolved Change-Id: I39dfe2cafb955ba1bf05b0ecdb0212d766fcc96c Issue-ID: CCSDK-95 Signed-off-by: Dan Timoney <dtimoney@att.com>
2017-09-18Merge "Fix few Major sonar issues"Dan Timoney2-181/+128
2017-09-18Explicitly import com.mysql.jdbc packageDan Timoney1-0/+1
Need to explicitly add com.mysql.jdbc to Import-Package in dblib/provider pom.xml, so that Class.forName() can resolve com.mysql.jdbc.Driver. Will also need to add org.mariadb.jdbc,but will do that in a separate commit Change-Id: I6a4be5d4e1841690919484940af75a2cb15c357b Issue-ID: CCSDK-94 Signed-off-by: Dan Timoney <dtimoney@att.com>
2017-09-18Fix few Major sonar issuesramu.n1-297/+314
Fix few Major sonar issues in CCSDK SLI Core module * Use logger method's built-in string formatting instead of string concatination * code formatting * Use Integer.toString() instead of concatenating empty string "" to a primitive Change-Id: I5a15c9b0049b71d7b0fa06c6791abe766a035c46 Issue-Id: CCSDK-67 Signed-off-by: Ramu N <ramu.n@huawei.com>
2017-09-18Fix few Major sonar issuesramu.n3-56/+56
Fix few Major sonar issues in CCSDK SLI Core module * Remove useless parentheses * Remove useless assignment to local variable * Use logger method instead of string formatting Change-Id: Ie9958d0bc4e754c36c127d465a1953160c2c3e2b Issue-Id: CCSDK-67 Signed-off-by: Ramu N <ramu.n@huawei.com>
2017-09-18Fix few Major sonar issuesramu.n2-181/+128
Fix few Major sonar issues in CCSDK SLI Core module * Remove useless parentheses * Either log or rethrow the exception * Add the "@Override" annotation * Change "try" to a try-with-resources * Replace duplicate string literals with constants Change-Id: If52163220d7231ae42ef42b0313d56435f6745e1 Issue-Id: CCSDK-67 Signed-off-by: Ramu N <ramu.n@huawei.com>
2017-09-17Fix Sonar issues in sli/core modulesurya-huawei1-8/+2
One major issue * Use try with resources instead of try Issue-Id: CCSDK-87 Change-Id: Ic30016f70ebc355f80a73c9e4632b7971d2b87be Signed-off-by: surya-huawei <a.u.surya@huawei.com>
2017-09-16Merge "Fix few Major sonar issues"Dan Timoney1-94/+41
2017-09-15Fix service tag in sli/recording blueprintDan Timoney1-1/+1
One of the service tags in sli/recording blueprint uses attribute 'class' instead of 'interface' to advertise the service. This is preventing SDNC container from loading sdnc-sli-recording feature. Change-Id: I4f6863d74dd6fb8babd91fde53b9680594aa7a31 Issue-ID: SDNC-54 Signed-off-by: Dan Timoney <dtimoney@att.com>
2017-09-15Merge "Log current DG & NodeId using MDC"Dan Timoney1-216/+221