summaryrefslogtreecommitdiffstats
path: root/controlloop
AgeCommit message (Collapse)AuthorFilesLines
2018-03-25Add timeout to SO to wait for successliamfallon8-454/+1168
The timeout implementation assumes that a finished request is a HTTP-200 together with either a request state of COMPLETE or FAILED in the embedded request in the response in the JSON. This is the same as what is done earlier in this class and in, for example, VFC. Unit test for timeout has also been added. Change-Id: I637dbecb8d230b8060f7ced76d92d11ec482503e Issue-ID: POLICY-448 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-03-23transaction logging through a featureJorge Hernandez10-0/+828
This is the first of 2 code inspections. This review contains though most of the work. It keeps tracks of all in progress transactions by intercepting all notifications. From that appropriate metrics can be obtained and logged. The actual logging of the metrics and enablement of the feature at install time will come in a subsequent review. Change-Id: Ie5c8adfbdeae72758da7da8a1b4133cd4818c6fb Issue-ID: POLICY-533 Signed-off-by: Jorge Hernandez <jh1730@att.com>
2018-03-22Removed checkstyle warningsmmis44-3467/+3936
Removed checkstyle warnings in: policy/controlloop/common/guard policy/controlloop/common/msb policy/controlloop/simulators including renaming of the following classes: MSBServiceFactory -> MsbServiceFactory MSBServiceManager -> MsbServiceManager MSBServiceException -> MsbServiceException Issue-ID: POLICY-705 Change-Id: I11d62a74fcb77e764121df119cb0bd9b844b485b Signed-off-by: mmis <michael.morris@ericsson.com>
2018-03-22Merge "Removed checkstyle warnings"Pamela Dragosh35-4798/+5151
2018-03-22Removed checkstyle warningsmmis35-4798/+5151
Removed checkstyle warnings in: policy/drools-applications/controlloop/common/actors policy/drools-applications/controlloop/common/eventmanager policy/drools-applications/controlloop/common/feature-controlloop-utils Issue-ID: POLICY-705 Change-Id: Iccf99b291bc62bc3ba2082ccdb4c1f9e12107896 Signed-off-by: mmis <michael.morris@ericsson.com>
2018-03-22Merge "Fix Serialized Names for Legacy APPC"Pamela Dragosh7-3/+49
2018-03-21Fix Serialized Names for Legacy APPCCRUZ7-3/+49
The legacy APPC request and response messages must be upper camel case. Issue-ID: POLICY-681 Change-Id: I439fa22b76dd2cb35a054bf93260705b8cf1af51 Signed-off-by: dc443y <dc443y@att.com>
2018-03-20Merge "Fix Serialized Names of A&AI Subtag"Pamela Dragosh2-0/+6
2018-03-19Fix Serialized Names of A&AI Subtagdc443y2-0/+6
Onset events received from DCAE contain an "AAI" subtag that is capitalized. The casing of the AAI variable was changed to lower case which was making the gson serializer unable to deserialze a simulated onset event that matches the casing DCAE uses, which resulted in Policy rejecting the events because they appear to be missing "AAI". A serialized name was added to preserve the lower cased variable as sonar wants but will also allow onsets with capitalized AAI subtags to be processed by the rules engine. Issue-ID: POLICY-681 Change-Id: Ia37694f606b3fb0898d6c4dadea7c48c5372e64e Signed-off-by: dc443y <dc443y@att.com>
2018-03-15Update oparent version to 1.1.0liamfallon1-1/+1
Directive from ONAP project that all repos should be updated. Change vierified with a build with .m2 /org/onap cleared. Changed dependency org.onap.msb.java-sdki.msb-java-sdk from version 1.1.0-SNAPSHOT to version 1.1.0 Change-Id: I46abc49019cb81b726e3f71ca8fe1beed621a426 Issue-ID: POLICY-693 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-03-14Merge "Changed SO request to iterate extra props"Pamela Dragosh1-7/+36
2018-03-14Changed SO request to iterate extra propsMagnusen, Drew (dm741q)1-7/+36
Modified SoActorServiceProvider.constructRequest() method so that it iterates over extra properties instead of hard coding array values. Issue-ID: POLICY-443 Change-Id: I37d6be9dc595bd8d5e9f9f3f63d6890ef1b2f8b3 Signed-off-by: Magnusen, Drew (dm741q) <dm741q@att.com>
2018-03-14Fix artifactid parameter in archetype POMliamfallon1-1/+1
The parameter ${artifactId} is depricated in POMs. This change replaces that parameter ${project.artifactId} Change-Id: I754fe11918707c599847471c438bedfd1285b0ed Issue-ID: POLICY-238 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-03-02Merge "Upgrade MSB client for security issues"Jorge Hernandez1-1/+1
2018-03-02Upgrade MSB client for security issuesPamela Dragosh1-1/+1
Via LiZi, the MSB team has made some changes to resolve their CLM security issues. We will upgrade to their release version when they have completed that work. Issue-ID: POLICY-615 Change-Id: I7ccc0e0f14d4d8ae8b6852275308b1aad62acaa3 Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
2018-03-02Merge "Reduce Technical Debt on drools-applications"Pamela Dragosh18-193/+199
2018-03-02Reduce Technical Debt on drools-applicationsliamfallon18-193/+199
This commit mops up the technical debt issues in drools-applications that are reaistic to fix witout major surgery on the code. note: warnings on non compliant Java class names that are used in yaml serializaiton and deserailization are ignored. note: warnings on complexity figures for equals() methods are ignored. Issue-ID: POLICY-455 Change-Id: I99ccda4bd9932ceb5ccbcfd2778730234b55d7a5 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-03-01Remove CLM issues with commons-collectionsPamela Dragosh4-0/+36
We know that we are not configuring an LDAP PIP in our use of the XACML open source. The LDAP implementation uses Apache Velocity, which uses a very old version of commons-collections that has security issues. So we can exclude commons-collections from the build. Issue-ID: POLICY-504 Change-Id: I6d90731e601f58c8edaca6fe02df30ee2a090c2f Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
2018-02-21Support vDNS scale out for multiple timesSwapnilPathak1-1/+34
Generation of Unique values for attribute vf-module-name for every scale out operation Change-Id: I5274361c2f5a4136020a384dbe4a4009607f589a Issue-ID: POLICY-610 Signed-off-by: SwapnilPathak <SP00494485@techmahindra.com>
2018-02-19Rename maven modules to mirror directory structureliamfallon84-5925/+5964
The naming of Maven modules in drools-applications was not aligned with the directory structure in the git repository of drools-applications. Therefore it was difficult to see the strucutre of the repository in Eclipse and other IDEs. This change amends the Maven module IDs to reflect the repository directory structure. This patch reset fixes the previos patch set, where many references to maven modules internally in drools-applciations were missed. See also changes in engine and docker repos. Updated to reflect repo directory structure in maven artifact groups. Issue-ID: POLICY-238 Change-Id: I8ab1a7ecdb664045222bbbfda269135e3e449109 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-12Merge "Fix technical debt/JUnit on policy-yaml"Jorge Hernandez31-686/+2359
2018-02-12Fix technical debt/JUnit on policy-yamlmmis31-686/+2359
Unit tests added and technical debt removed Issue-ID: POLICY-455 Change-Id: I31566f9386cb94d78e9af3d6261e31b9cad1cb53 Signed-off-by: mmis <michael.morris@ericsson.com>
2018-02-11Tidy dependencies and scopes in POMsliamfallon8-31/+6
There were some duplicate dependencies in the POMs that were causing warnings. These have been removed. In cases where the revision of the duplicate dependencies were different,the latest revision was used. The scope of the "junit" dependency was "provided" in some POMs. These have been corrected with the scope of "test" being used for junit dependencies. Issue-ID: POLICY-238 Change-Id: Ib1cc6f040b0c4d7f2b39ae02bc6fb4590d395809 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-09Merge "Upgrade XACML jar"Jorge Hernandez5-6/+6
2018-02-09Merge "Remove Technical Debt from MSB code"Pamela Dragosh2-113/+113
2018-02-09Upgrade XACML jarPamela Dragosh5-6/+6
XACML was released with minor fixes for XML files with comments. Mainly it contained upgrade to dependencies that had fixed security issues. One still remains but will not be able to be fixed. Issue-ID: POLICY-616 Change-Id: Ifaf4f92d8da878a1c49dff5fd028744d01999ff5 Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
2018-02-09Remove Technical Debt from MSB codeliamfallon2-113/+113
Some minor warnings showed up in Eclipse on the MSB module. This change removes those warnings. Issue-ID: POLICY-455 Change-Id: Ibdd8c563120178d73a55adfd0d8853b6bc82b820 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-08JUint/Technical Debt for eventmanagerliamfallon29-942/+2551
Unit test expanded for SO POJOs, technical debt removed in eventmanager. Change-Id: I7045f5af88fd2c15019befe29a0b4fcd69801c49 Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-07Merge "Resolve security issues SONATYPE-2017-0359"Jorge Hernandez3-3/+3
2018-02-07Resolve security issues SONATYPE-2017-0359Pamela Dragosh3-3/+3
Upgrade httpclient to 4.5.5 - the latest version. Issue-ID: POLICY-612 Change-Id: I5e79918bc447889b1dbcd1bf897c3324f6a0cc0d Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
2018-02-07Merge "Fix technical debt/JUnit on MSB"Jorge Hernandez7-12/+147
2018-02-07Fix technical debt/JUnit on MSBeeimmis7-12/+147
Unit tests added and technical debt removed Issue-ID: POLICY-455 Change-Id: Ib10613ff7cac64bb576046e038b2acbd66152648 Signed-off-by: mmis <michael.morris@ericsson.com>
2018-02-07Log thrown exception in PIPEngineGetHistoryliamfallon1-1/+1
Sonar Critical error highlighted, with thrown execption not logged. The exception is now logged. Issue-ID: POLICY-455 Change-Id: If2969a4fe118d22dbd1fb11879ba028487848eb8 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-06Add JUnit and Remove Technical Debt in Guardliamfallon25-762/+1788
Unit test for guard added and technical debt removed. Issue-ID: POLICY-455 Change-Id: I440766660ddae078013e21b1991ee49c8e488bb3 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-05Fix package directory naming to lower caseLiam Fallon5-0/+0
In macOS (and windows) directory names are case insensitive, but in Linux they are case sensitive. Therefore, when the "actorServiceProvider" directory name was renamed to "actorserviceprovider", the change did not propogate into git when a Macbook was used for development. This error was discovered when working on a Ubuntu machine and is now fixed. Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Change-Id: Ifebe3d37d42e79fff8da2370369967a25c371b11 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-31Merge "Fix technical debt/JUnit on SO/VFC/SO ACTOR"Jorge Hernandez15-235/+703
2018-01-31Merge "Fix technical debt/JUnit on actor service provider"Jorge Hernandez12-64/+140
2018-01-30Merge "Do not enforce hostname validation"Jorge Hernandez1-2/+11
2018-01-30Fix technical debt/JUnit on actor service providerliamfallon12-64/+140
Unit test for actor service provider added and technical debt removed. Change-Id: I9573f1c3ff0252b166e06caaa88eb679a1fc7347 Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-30Fix technical debt/JUnit on SO/VFC/SO ACTORliamfallon15-235/+703
Unit test expanded for SO POJOs, technical debt removed in SO actor and VFC pojos. Change-Id: I23b886c40c1ac6ac8dc2ebbaade315b71cca9dd0 Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455
2018-01-29Do not enforce hostname validationAlexis de Talhouët1-2/+11
Change-Id: I1c4e5d58126e35a64140c91c0c3afda5e9b3a5bd Issue-ID: POLICY-510 Signed-off-by: Alexis de Talhouët <adetalhouet89@gmail.com>
2018-01-28Fix Tech Debt/JUnit on SO actorliamfallon3-565/+338
Unit test for SO actor added. SO actor service provider class refactored to remove technical debt. Change-Id: I6186ede5661ce483917184f03ee44d57a124b6a4 Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-23Implement JUnit tests for actor.vfcliamfallon2-6/+108
JUnit tests for the VFC actor implemented. Change-Id: Ifcd433a4ce6f46cad46d50f4ce4c7c03e2abec67 Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-22Fix Tech Debt/JUnit on VFC actorliamfallon3-107/+126
Unit test for VFC actor added. Some minor chages to actor service provider class to remove technical debt. Change-Id: I54ee24bcfa1c82f4a18c0035bdff186d4d3ebdf7 Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-22Merge "Fix Tech Debt/JUnit on VFC POJOs"Jorge Hernandez20-196/+996
2018-01-21Fix Tech Debt/JUnit on APPC and APPCLCM actorsliamfallon5-321/+358
Unit test updated to increase coverage on getter methods. Constants added for some string literals to reduce technical debt warnings. Change-Id: I3b175de7c65aa9d342e71adc951d1ee08c9d781b Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-20Fix Tech Debt/JUnit on VFC POJOsliamfallon20-196/+996
Making fields private, fixing field and methods to follow Java guidelines, and adding getter and setter methods. Unit test added for all classes in org.onap.policy.vfc Issue-ID: POLICY-455 Change-Id: I262337a816706cd5243849cd51e57689275545bf Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-19Merge "Fix Technical Debt, add Unit tests for SO"Jorge Hernandez42-829/+2390
2018-01-18Merge "Fix Technical Debt, Unit Test for APPCLCM POJOs"Jorge Hernandez14-158/+835
2018-01-18Merge "Fix Tech Debt/JUnit on control loop event POJOs"Jorge Hernandez25-669/+989