policy/drools-applications.git
6 years agotransaction logging through a feature 75/37975/2
Jorge Hernandez [Fri, 23 Mar 2018 06:14:12 +0000 (01:14 -0500)]
transaction logging through a feature

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>
6 years agoRemoved checkstyle warnings 09/37809/1
mmis [Thu, 22 Mar 2018 14:20:46 +0000 (14:20 +0000)]
Removed checkstyle warnings

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>
6 years agoMerge "Removed checkstyle warnings"
Pamela Dragosh [Thu, 22 Mar 2018 14:35:27 +0000 (14:35 +0000)]
Merge "Removed checkstyle warnings"

6 years agoRemoved checkstyle warnings 51/37751/3
mmis [Wed, 21 Mar 2018 15:22:10 +0000 (15:22 +0000)]
Removed checkstyle warnings

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>
6 years agoMerge "Fix Serialized Names for Legacy APPC"
Pamela Dragosh [Thu, 22 Mar 2018 12:25:48 +0000 (12:25 +0000)]
Merge "Fix Serialized Names for Legacy APPC"

6 years agoFix Serialized Names for Legacy APPC 25/37625/2
CRUZ [Thu, 22 Mar 2018 00:49:32 +0000 (19:49 -0500)]
Fix Serialized Names for Legacy APPC

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>
6 years agoMerge "Fix Serialized Names of A&AI Subtag"
Pamela Dragosh [Tue, 20 Mar 2018 12:37:43 +0000 (12:37 +0000)]
Merge "Fix Serialized Names of A&AI Subtag"

6 years agoFix Serialized Names of A&AI Subtag 89/36889/1
dc443y [Tue, 20 Mar 2018 02:04:50 +0000 (21:04 -0500)]
Fix Serialized Names of A&AI Subtag

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>
6 years agoUpdate oparent version to 1.1.0 35/35835/2
liamfallon [Wed, 14 Mar 2018 21:39:34 +0000 (21:39 +0000)]
Update oparent version to 1.1.0

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>
6 years agoMerge "Changed SO request to iterate extra props"
Pamela Dragosh [Wed, 14 Mar 2018 15:45:21 +0000 (15:45 +0000)]
Merge "Changed SO request to iterate extra props"

6 years agoChanged SO request to iterate extra props 59/35759/3
Magnusen, Drew (dm741q) [Wed, 14 Mar 2018 13:36:25 +0000 (08:36 -0500)]
Changed SO request to iterate extra props

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>
6 years agoFix artifactid parameter in archetype POM 41/35741/1
liamfallon [Wed, 14 Mar 2018 11:07:28 +0000 (11:07 +0000)]
Fix artifactid parameter in archetype POM

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>
6 years agoMerge "Upgrade MSB client for security issues"
Jorge Hernandez [Fri, 2 Mar 2018 18:15:50 +0000 (18:15 +0000)]
Merge "Upgrade MSB client for security issues"

6 years agoUpgrade MSB client for security issues 21/33821/1
Pamela Dragosh [Fri, 2 Mar 2018 16:04:27 +0000 (11:04 -0500)]
Upgrade MSB client for security issues

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>
6 years agoMerge "Reduce Technical Debt on drools-applications"
Pamela Dragosh [Fri, 2 Mar 2018 16:02:28 +0000 (16:02 +0000)]
Merge "Reduce Technical Debt on drools-applications"

6 years agoReduce Technical Debt on drools-applications 97/33797/1
liamfallon [Fri, 2 Mar 2018 14:24:23 +0000 (14:24 +0000)]
Reduce Technical Debt on drools-applications

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>
6 years agoRemove CLM issues with commons-collections 87/33687/1
Pamela Dragosh [Thu, 1 Mar 2018 22:11:20 +0000 (17:11 -0500)]
Remove CLM issues with commons-collections

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>
6 years agoSupport vDNS scale out for multiple times 85/32385/1
SwapnilPathak [Wed, 21 Feb 2018 11:26:27 +0000 (16:56 +0530)]
Support vDNS scale out for multiple times

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>
6 years agoRename maven modules to mirror directory structure 93/31593/6
liamfallon [Tue, 13 Feb 2018 11:29:56 +0000 (11:29 +0000)]
Rename maven modules to mirror directory structure

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>
6 years agoModify sonar settings to include all sub-modules 83/31983/1
Pamela Dragosh [Fri, 16 Feb 2018 15:41:37 +0000 (10:41 -0500)]
Modify sonar settings to include all sub-modules

We are now including all the sub-modules coverage into one file.
This will now reflect correct statistics for any sub-module that
tests code located in another sub-module.

Issue-ID: POLICY-634
Change-Id: Id96eb9e9f637770932067bd803cb5ccdb571c05a
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
6 years agoMerge "Fix technical debt/JUnit on policy-yaml"
Jorge Hernandez [Mon, 12 Feb 2018 16:54:48 +0000 (16:54 +0000)]
Merge "Fix technical debt/JUnit on policy-yaml"

6 years agoFix technical debt/JUnit on policy-yaml 25/31425/1
mmis [Mon, 12 Feb 2018 11:54:51 +0000 (11:54 +0000)]
Fix technical debt/JUnit on policy-yaml

Unit tests added and technical debt removed

Issue-ID: POLICY-455
Change-Id: I31566f9386cb94d78e9af3d6261e31b9cad1cb53
Signed-off-by: mmis <michael.morris@ericsson.com>
6 years agoTidy dependencies and scopes in POMs 61/31261/1
liamfallon [Sun, 11 Feb 2018 12:42:11 +0000 (12:42 +0000)]
Tidy dependencies and scopes in POMs

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>
6 years agoMerge "Upgrade XACML jar"
Jorge Hernandez [Fri, 9 Feb 2018 15:52:57 +0000 (15:52 +0000)]
Merge "Upgrade XACML jar"

6 years agoMerge "Remove Technical Debt from MSB code"
Pamela Dragosh [Fri, 9 Feb 2018 15:08:42 +0000 (15:08 +0000)]
Merge "Remove Technical Debt from MSB code"

6 years agoUpgrade XACML jar 93/31093/1
Pamela Dragosh [Fri, 9 Feb 2018 14:52:40 +0000 (09:52 -0500)]
Upgrade XACML jar

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>
6 years agoRemove Technical Debt from MSB code 75/31075/1
liamfallon [Fri, 9 Feb 2018 13:17:41 +0000 (13:17 +0000)]
Remove Technical Debt from MSB code

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>
6 years agoJUint/Technical Debt for eventmanager 25/30925/2
liamfallon [Thu, 8 Feb 2018 21:54:16 +0000 (21:54 +0000)]
JUint/Technical Debt for eventmanager

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>
6 years agoMerge "Resolve security issues SONATYPE-2017-0359"
Jorge Hernandez [Wed, 7 Feb 2018 20:50:20 +0000 (20:50 +0000)]
Merge "Resolve security issues SONATYPE-2017-0359"

6 years agoResolve security issues SONATYPE-2017-0359 83/30683/1
Pamela Dragosh [Wed, 7 Feb 2018 16:01:19 +0000 (11:01 -0500)]
Resolve security issues SONATYPE-2017-0359

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>
6 years agoMerge "Fix technical debt/JUnit on MSB"
Jorge Hernandez [Wed, 7 Feb 2018 15:58:54 +0000 (15:58 +0000)]
Merge "Fix technical debt/JUnit on MSB"

6 years agoFix technical debt/JUnit on MSB 31/30431/3
eeimmis [Tue, 6 Feb 2018 09:43:40 +0000 (09:43 +0000)]
Fix technical debt/JUnit on MSB

Unit tests added and technical debt removed

Issue-ID: POLICY-455
Change-Id: Ib10613ff7cac64bb576046e038b2acbd66152648
Signed-off-by: mmis <michael.morris@ericsson.com>
6 years agoLog thrown exception in PIPEngineGetHistory 73/30673/1
liamfallon [Wed, 7 Feb 2018 14:14:46 +0000 (14:14 +0000)]
Log thrown exception in PIPEngineGetHistory

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>
6 years agoAdd JUnit and Remove Technical Debt in Guard 65/30465/1
liamfallon [Tue, 6 Feb 2018 14:17:36 +0000 (14:17 +0000)]
Add JUnit and Remove Technical Debt in Guard

Unit test for guard added and technical debt removed.

Issue-ID: POLICY-455
Change-Id: I440766660ddae078013e21b1991ee49c8e488bb3
Signed-off-by: liamfallon <liam.fallon@ericsson.com>
6 years agoFix package directory naming to lower case 59/30259/1
Liam Fallon [Mon, 5 Feb 2018 12:02:05 +0000 (12:02 +0000)]
Fix package directory naming to lower case

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>
6 years agoMerge "Fix technical debt/JUnit on SO/VFC/SO ACTOR"
Jorge Hernandez [Wed, 31 Jan 2018 13:42:53 +0000 (13:42 +0000)]
Merge "Fix technical debt/JUnit on SO/VFC/SO ACTOR"

6 years agoMerge "Fix technical debt/JUnit on actor service provider"
Jorge Hernandez [Wed, 31 Jan 2018 13:39:55 +0000 (13:39 +0000)]
Merge "Fix technical debt/JUnit on actor service provider"

6 years agoMerge "Do not enforce hostname validation"
Jorge Hernandez [Tue, 30 Jan 2018 18:47:43 +0000 (18:47 +0000)]
Merge "Do not enforce hostname validation"

6 years agoFix technical debt/JUnit on actor service provider 83/29383/4
liamfallon [Sun, 28 Jan 2018 22:24:23 +0000 (22:24 +0000)]
Fix technical debt/JUnit on actor service provider

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>
6 years agoFix technical debt/JUnit on SO/VFC/SO ACTOR 75/29675/1
liamfallon [Tue, 30 Jan 2018 17:06:20 +0000 (17:06 +0000)]
Fix technical debt/JUnit on SO/VFC/SO ACTOR

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

6 years agoDo not enforce hostname validation 29/29529/2
Alexis de Talhouët [Mon, 29 Jan 2018 15:38:40 +0000 (10:38 -0500)]
Do not enforce hostname validation

Change-Id: I1c4e5d58126e35a64140c91c0c3afda5e9b3a5bd
Issue-ID: POLICY-510
Signed-off-by: Alexis de Talhouët <adetalhouet89@gmail.com>
6 years agoFix Tech Debt/JUnit on SO actor 25/29325/2
liamfallon [Fri, 26 Jan 2018 14:31:09 +0000 (14:31 +0000)]
Fix Tech Debt/JUnit on SO actor

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>
6 years agoImplement JUnit tests for actor.vfc 23/28923/2
liamfallon [Tue, 23 Jan 2018 13:47:12 +0000 (13:47 +0000)]
Implement JUnit tests for actor.vfc

JUnit tests for the VFC actor implemented.

Change-Id: Ifcd433a4ce6f46cad46d50f4ce4c7c03e2abec67
Issue-ID: POLICY-455
Signed-off-by: liamfallon <liam.fallon@ericsson.com>
6 years agoFix Tech Debt/JUnit on VFC actor 87/28787/4
liamfallon [Mon, 22 Jan 2018 12:13:47 +0000 (12:13 +0000)]
Fix Tech Debt/JUnit on VFC actor

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>
6 years agoMerge "Fix Tech Debt/JUnit on VFC POJOs"
Jorge Hernandez [Mon, 22 Jan 2018 14:31:03 +0000 (14:31 +0000)]
Merge "Fix Tech Debt/JUnit on VFC POJOs"

6 years agoFix Tech Debt/JUnit on APPC and APPCLCM actors 39/28739/1
liamfallon [Sat, 20 Jan 2018 11:22:27 +0000 (11:22 +0000)]
Fix Tech Debt/JUnit on APPC and APPCLCM actors

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>
6 years agoFix Tech Debt/JUnit on VFC POJOs 89/28689/2
liamfallon [Fri, 19 Jan 2018 19:55:02 +0000 (19:55 +0000)]
Fix Tech Debt/JUnit on VFC POJOs

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>
6 years agoMerge "Fix Technical Debt, add Unit tests for SO"
Jorge Hernandez [Fri, 19 Jan 2018 18:56:33 +0000 (18:56 +0000)]
Merge "Fix Technical Debt, add Unit tests for SO"

6 years agoMerge "Fix Technical Debt, Unit Test for APPCLCM POJOs"
Jorge Hernandez [Thu, 18 Jan 2018 20:02:58 +0000 (20:02 +0000)]
Merge "Fix Technical Debt, Unit Test for APPCLCM POJOs"

6 years agoMerge "Fix Tech Debt/JUnit on control loop event POJOs"
Jorge Hernandez [Thu, 18 Jan 2018 19:49:37 +0000 (19:49 +0000)]
Merge "Fix Tech Debt/JUnit on control loop event POJOs"

6 years agoFix Technical Debt, Unit Test for APPCLCM POJOs 03/28403/3
Liam Fallon [Wed, 17 Jan 2018 12:03:50 +0000 (12:03 +0000)]
Fix Technical Debt, Unit Test for APPCLCM POJOs

Mainly 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.appclcm

Issue-ID: POLICY-455
Change-Id: Iae873f94a236cc9faeac5542a9633ca840b696ed
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
6 years agoMerge "Fix Technical Debt, basic JUnit tests for Rest"
Jorge Hernandez [Thu, 18 Jan 2018 19:44:09 +0000 (19:44 +0000)]
Merge "Fix Technical Debt, basic JUnit tests for Rest"

6 years agoFix Technical Debt, add Unit tests for SO 19/28519/2
vdmeer [Thu, 18 Jan 2018 12:50:32 +0000 (12:50 +0000)]
Fix Technical Debt, add Unit tests for SO

Converted to POJOs, fixed sona lint errors, added JUnit tests
POJOs now have only private members and getters/setters
This implicated actors, simulators, and existing tests

Issue-ID: POLICY-455
Change-Id: I4b80f729565e8675822a890892b15676908f73b4
Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
6 years agoMerge "Fix Technical Debt for TrafficGenerator"
Pamela Dragosh [Thu, 18 Jan 2018 12:53:09 +0000 (12:53 +0000)]
Merge "Fix Technical Debt for TrafficGenerator"

6 years agoFix Tech Debt/JUnit on control loop event POJOs 11/28511/1
Liam Fallon [Thu, 18 Jan 2018 10:55:53 +0000 (10:55 +0000)]
Fix Tech Debt/JUnit on control loop event POJOs

Mainly 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.controlloop

Issue-ID: POLICY-455
Change-Id: I445b7cfaf9eb921a230bbb72b06ff4455fe003ce
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
6 years agoFix Technical Debt, basic JUnit tests for Rest 59/28459/2
vdmeer [Wed, 17 Jan 2018 23:13:14 +0000 (23:13 +0000)]
Fix Technical Debt, basic JUnit tests for Rest

Basic tests for get and post
(not testing full user/pw/body)

Issue-ID: POLICY-455
Change-Id: I186ebf969abc239e5056199c2fd8da9f3253d2fc
Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
6 years agoFix Technical Debt for TrafficGenerator 39/28439/3
vdmeer [Wed, 17 Jan 2018 18:52:14 +0000 (18:52 +0000)]
Fix Technical Debt for TrafficGenerator

Changed formatting for checkstyle, changed sequence of statements
Left empty constructors as requested

Issue-ID: POLICY-455
Change-Id: I0e443bba69cede4c006a841d8d34c82391ff1040
Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
6 years agoAdded all J Unit tests SDC 29/28429/1
vdmeer [Wed, 17 Jan 2018 17:30:04 +0000 (17:30 +0000)]
Added all J Unit tests SDC

Issue-ID: POLICY-455
Change-Id: I8192e8a595642e0d36608591d95f21f3ee574d13
Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
6 years agoFix Sonar Technical Debt, Unit Test for APPC POJOs 41/28341/1
Liam Fallon [Tue, 16 Jan 2018 18:42:01 +0000 (18:42 +0000)]
Fix Sonar Technical Debt, Unit Test for APPC POJOs

Mainly 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.appc

Change-Id: If3db740bc146a09f8f7387f02c12b048ad00b201
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
Issue-ID: POLICY-455
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
6 years agoRestore getters and setters to AAINQResponse class 37/28237/1
Liam Fallon [Mon, 15 Jan 2018 22:43:07 +0000 (22:43 +0000)]
Restore getters and setters to AAINQResponse class

Some get and set methods were deleted from this class in error. This patch restores them.

Change-Id: I4dfaf1215e56f30a1d62d19624fe4da570e99171
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
Issue-ID: POLICY-455

6 years agoMerge "Cleanup of A&AI message model module."
Jorge Hernandez [Mon, 15 Jan 2018 19:02:19 +0000 (19:02 +0000)]
Merge "Cleanup of A&AI message model module."

6 years agoAdd IntelliJ files to .gitignore 27/27827/1
Alexis de Talhouët [Fri, 5 Jan 2018 18:10:39 +0000 (13:10 -0500)]
Add IntelliJ files to .gitignore

Change-Id: I45dd03d4295322ecc0f10c4673848c1fe85d5e98
Issue-ID: POLICY-509
Signed-off-by: Alexis de Talhouët <adetalhouet89@gmail.com>
6 years agoAdd guard junits 77/26377/1
Temoc Rodriguez [Thu, 14 Dec 2017 23:58:32 +0000 (15:58 -0800)]
Add guard junits

Added a few junit files which test most functionality in the guard
submodule. Some minor bugs were found by the junits and fixed those
newly found bugs. Added persistence.xml for use by junits in guard.

Issue-ID: POLICY-490
Change-Id: Ied7e276cfd417e1f396b5a6685a2e9acc9efd109
Signed-off-by: Temoc Rodriguez <cr056n@att.com>
6 years agoCleanup of A&AI message model module. 31/26331/2
Liam Fallon [Thu, 14 Dec 2017 15:40:18 +0000 (07:40 -0800)]
Cleanup of A&AI message model module.

Fields have been made fields private in POJOs, and wrote JUnit unit tests to bring up code coverage. AAIManager also changed to use non-static methods so that it can be stubbed with Mockito.

Change-Id: I871427ced5db8a1dfd6495fef4e6d84adf264e65
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
Issue-ID: POLICY-455
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
6 years agoMerge "Technical Debt in AAI."
Pamela Dragosh [Mon, 11 Dec 2017 04:21:33 +0000 (04:21 +0000)]
Merge "Technical Debt in AAI."

6 years agoremove synchronized calls to avoid race conditions 29/25929/1
Jorge Hernandez [Wed, 6 Dec 2017 21:46:08 +0000 (15:46 -0600)]
remove synchronized calls to avoid race conditions

This is to prevent potential problems where multiple jar versions are
updated one after the other.

Change-Id: I78835b485e57125efb213cfdefba60351d815916
Issue-ID: POLICY-487
Signed-off-by: Jorge Hernandez <jh1730@att.com>
(cherry picked from commit f2715bc0f272e5e657ed8a82c0f4e520a7c7706d)

6 years agoRemove guard dependency from simulators 97/25797/1
Charles Cole [Thu, 7 Dec 2017 18:17:17 +0000 (12:17 -0600)]
Remove guard dependency from simulators

Removed the deendency on guard from the guard simulator tests so that
the guard simulator can be used to test guard functionality.

Issue-ID: POLICY-490
Change-Id: Ie67abb85e9844c412c483b1bbf3fb4aa21f3530a
Signed-off-by: Charles Cole <cc847m@att.com>
6 years agoTechnical Debt in AAI. 51/25751/1
Liam Fallon [Wed, 6 Dec 2017 22:06:31 +0000 (23:06 +0100)]
Technical Debt in AAI.

Eliminate the technical debt identified in the drools-applications/controlloop/common/model-impl/aai  module by Sonar

Change-Id: I72125baacc217386a4984be04bfc023439eddd13
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
Issue-ID: POLICY-455
Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
6 years agoActor service provider for VFC and Actor Unit Test 97/25697/1
EEILFN [Wed, 6 Dec 2017 17:19:07 +0000 (18:19 +0100)]
Actor service provider for VFC and Actor Unit Test

The service provider in the META-INF/services file for the VFC actor pointed at the APPC actor rather than the VFC actor, this is fixed to now point at the correct actor implementation. Alsothe Actor test POM is missing the actors 'appclcm' and 'vfc', thus excluding them from the unit test

Change-Id: I60bfd160952b7663aab355ae9c8b0aa362484874
Signed-off-by: EEILFN <liam.fallon@ericsson.com>
Issue-ID: POLICY-455
Signed-off-by: EEILFN <liam.fallon@ericsson.com>
6 years agoBump minor version 85/24285/2
Jessica Wagantall [Sun, 19 Nov 2017 02:36:44 +0000 (18:36 -0800)]
Bump minor version

Bump minor version in preparation for Amsterdam
branching.

Change-Id: Ia7e97a72053a2d1cd10c0b9d5c179817c3ac7e23
Issue-ID: CIMAN-120
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
6 years agoUpdate version.properties 1.1.2 03/23903/1
Pamela Dragosh [Thu, 16 Nov 2017 19:09:02 +0000 (14:09 -0500)]
Update version.properties 1.1.2

Missed this in the last update.

Issue-ID: POLICY-436
Change-Id: I673f13d75c8676630a94f8706bf1ceea91608518
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
6 years agoUpgrade to 1.1.2-SNAPSHOT 45/23645/1
Pamela Dragosh [Wed, 15 Nov 2017 16:47:35 +0000 (11:47 -0500)]
Upgrade to 1.1.2-SNAPSHOT

Released 1.1.1, now must bump patch by 1

Issue-ID: POLICY-436
Change-Id: I3f5e4c369575f6fe1fa06cabb96a5bd43cb11087
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
6 years agoMerge "Update SO to use is-base-vf-module=false" 1.0.0-Amsterdam 1.0.0-ONAP v1.1.1
Pamela Dragosh [Tue, 14 Nov 2017 00:57:19 +0000 (00:57 +0000)]
Merge "Update SO to use is-base-vf-module=false"

6 years agoUpdate SO to use is-base-vf-module=false 15/23415/3
Hockla, Ali (ah999m) [Mon, 13 Nov 2017 23:12:28 +0000 (17:12 -0600)]
Update SO to use is-base-vf-module=false

Issue-ID: POLICY-447
Change-Id: I63cd5d20670f224aa4b9220b99ee40aad4f3e50c
Signed-off-by: Hockla, Ali (ah999m) <ah999m@att.com>
6 years agodefault archetype tooling to 1.1.1 85/23385/2
Jorge Hernandez [Mon, 13 Nov 2017 18:45:53 +0000 (12:45 -0600)]
default archetype tooling to 1.1.1

Change-Id: I84cf9fe9e954a9ee9bac8eaff56169e44154e9c5
Issue-ID: POLICY-446
Signed-off-by: Jorge Hernandez <jh1730@att.com>
6 years agoMerge "Fix release job"
Jorge Hernandez [Sat, 11 Nov 2017 03:54:23 +0000 (03:54 +0000)]
Merge "Fix release job"

6 years agoFix release job 05/23205/2
Pamela Dragosh [Sat, 11 Nov 2017 01:11:58 +0000 (20:11 -0500)]
Fix release job

Missing staging repo

Issue-ID: POLICY-441
Change-Id: Ib652d90ced62d8e96fd737ddd491c7a96d5dd45c
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
6 years agoFix SO Request mismatch and udpate AAI NQ response 45/23145/1
Hockla, Ali (ah999m) [Fri, 10 Nov 2017 17:27:15 +0000 (11:27 -0600)]
Fix SO Request mismatch and udpate AAI NQ response

Issue-ID: POLICY-438
Change-Id: I1d27d0e5f7d253aa3cba82f5f4b515c01be35d8b
Signed-off-by: Hockla, Ali (ah999m) <ah999m@att.com>
6 years agoFix SO Request structure 63/22963/1
Hockla, Ali (ah999m) [Thu, 9 Nov 2017 14:38:35 +0000 (08:38 -0600)]
Fix SO Request structure

Issue-ID: POLICY-438
Change-Id: I05e634782004fff5540b3e6fabb9e22846438cb1
Signed-off-by: Hockla, Ali (ah999m) <ah999m@att.com>
6 years agoMerge "Fix SO Request body"
Jorge Hernandez [Thu, 9 Nov 2017 04:03:10 +0000 (04:03 +0000)]
Merge "Fix SO Request body"

6 years agoMerge "Update SNAPSHOT version"
Jorge Hernandez [Wed, 8 Nov 2017 23:59:42 +0000 (23:59 +0000)]
Merge "Update SNAPSHOT version"

6 years agoFix SO Request body 57/22857/1
Hockla, Ali (ah999m) [Wed, 8 Nov 2017 23:27:16 +0000 (17:27 -0600)]
Fix SO Request body

- Renamed modelNameVersionId to modelVersionId in SOModelInfo
- Removed requestID from SO Request body
- As a result of this, an SOResponseWrapper was needed in order to
  attach the control loop event requestId to use in the SOResponse
  rule in drl (the requestID that is returned in the SO response
  is a newly generated requestId with no relation to the Policy
  control loop event requestId as per Arthur)
- Updated drl to reflect the above mentioned change
- Updated simulator and junits to reflect changes
- Added an additional null httpResponse check/logger statement in SOManager and RestManager

Issue-ID: POLICY-438
Change-Id: I5f414ba69b60b5565cca9073b47f4c4835e5abbf
Signed-off-by: Hockla, Ali (ah999m) <ah999m@att.com>
6 years agoFix SO url to remove suffix HTTPS/1.1 33/22833/1
Hockla, Ali (ah999m) [Wed, 8 Nov 2017 19:38:51 +0000 (13:38 -0600)]
Fix SO url to remove suffix HTTPS/1.1

Issue-ID: POLICY-437
Change-Id: I19679d9782f78ff73e3a90d0c8c95b2b6e9d319f
Signed-off-by: Hockla, Ali (ah999m) <ah999m@att.com>
6 years agoUpdate SNAPSHOT version 27/22627/2
Pamela Dragosh [Wed, 8 Nov 2017 02:13:02 +0000 (21:13 -0500)]
Update SNAPSHOT version

Releasing v1.1.0 so need to update version to 1.1.1

Issue-ID: POLICY-436
Change-Id: I8b85de39bb3a32f5f4faeeb4fcdfb5d95291ae31
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
6 years agoSNAPSHOT present in policy core and msb deps 35/22535/1 v1.1.0
Jorge Hernandez [Tue, 7 Nov 2017 15:04:10 +0000 (09:04 -0600)]
SNAPSHOT present in policy core and msb deps

Change-Id: I61c8d698f8cf984c648e510997498d41e7d9744a
Issue-ID: POLICY-433
Signed-off-by: Jorge Hernandez <jh1730@att.com>
6 years agoFix Generic VNF isClosedLoopDisabled 43/22043/1
daniel [Fri, 3 Nov 2017 15:39:23 +0000 (10:39 -0500)]
Fix Generic VNF isClosedLoopDisabled

No A&AI queries will be done for enriched events.
The event manager now understands prepended "generic-vnf."
in front of is-closed-loop-disabled from DCAE onsets.
A test case was revised to include sample data from an
enriched onset with vnf data based on the VES documentation.

Issue-Id: POLICY-350
Change-Id: Id7f59f8b860b9f147087067a2a9a7766f167f7e4
Signed-off-by: Daniel Cruz <dc443y@att.com>
6 years agoMerge "Made network.log output more verbose."
Pamela Dragosh [Fri, 3 Nov 2017 14:29:29 +0000 (14:29 +0000)]
Merge "Made network.log output more verbose."

6 years agoMade network.log output more verbose. 35/21935/3
Magnusen, Drew (dm741q) [Thu, 2 Nov 2017 20:41:48 +0000 (15:41 -0500)]
Made network.log output more verbose.

Incoming values were not expanded in network logs for several
drools-applications. Current patch makes them more verbose.

Issue-ID: POLICY-410
Change-Id: I64bf88a3b39ea58dceaf24de698c8ae6a9a3f8b2
Signed-off-by: Magnusen, Drew (dm741q) <dm741q@att.com>
6 years agoMerge "Write to DB on guard deny"
Pamela Dragosh [Fri, 3 Nov 2017 14:09:27 +0000 (14:09 +0000)]
Merge "Write to DB on guard deny"

6 years agoMerge "Add fix to catch exception"
Jorge Hernandez [Fri, 3 Nov 2017 03:36:11 +0000 (03:36 +0000)]
Merge "Add fix to catch exception"

6 years agoWrite to DB on guard deny 29/21929/2
Temoc Rodriguez [Thu, 2 Nov 2017 18:12:39 +0000 (11:12 -0700)]
Write to DB on guard deny

When an operations was denied by guard, it was not writing to the DB at
all. Only in the deny case we start the operation, then
setOperationHasGuardDeny, then finish the operation. Had to add start
operation just before finishing it. This allows the write to occur at
setOperationHasGuardDeny.

Issue-ID: POLICY-345
Change-Id: I665b9cfaa684493e39e731102d4d43ecb39fad7c
Signed-off-by: Temoc Rodriguez <cr056n@att.com>
6 years agoAdded fix to catch null pointer in SOManager 21/21921/3
Hockla, Ali (ah999m) [Thu, 2 Nov 2017 17:14:29 +0000 (12:14 -0500)]
Added fix to catch null pointer in SOManager

Issue-ID: POLICY-408
Change-Id: I9f6681b8cbafdd9639966d862e7b1df3b729297e
Signed-off-by: Hockla, Ali (ah999m) <ah999m@att.com>
6 years agoAdd fix to catch exception 37/21937/1
Ritu Sood [Thu, 2 Nov 2017 12:29:49 +0000 (05:29 -0700)]
Add fix to catch exception

Added fix to catch exception caused by
NULL url provided to Policy for VFC

Change-Id: Ifb8bdf41fbb212acf13d388d041d3cb44143caeb
Issue-ID: POLICY-408
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
6 years agoMerge "Fix Abatement Processing"
Pamela Dragosh [Tue, 31 Oct 2017 17:20:08 +0000 (17:20 +0000)]
Merge "Fix Abatement Processing"

6 years agoFix Abatement Processing 91/21491/5
daniel [Tue, 31 Oct 2017 12:37:41 +0000 (07:37 -0500)]
Fix Abatement Processing

The drools engine will now only fire the event rule if the
event is an onset. This prevents abatements from being processed
as onsets.

Issue-Id: POLICY-407
Change-Id: Ic74efa74fbb1dc0e01d7c36a06fdaf48fc8a3238
Signed-off-by: Daniel Cruz <dc443y@att.com>
6 years agoPassed classloader property to entitymanager. 15/21515/2
Magnusen, Drew (dm741q) [Tue, 31 Oct 2017 14:41:11 +0000 (09:41 -0500)]
Passed classloader property to entitymanager.

Passed property EntityManager that sets its class loader.

Issue-ID: POLICY-345
Change-Id: Ie2a94feb442243de18cc4de5aab0312a02565729
Signed-off-by: Magnusen, Drew (dm741q) <dm741q@att.com>
6 years agoclean up parent pom pre-release 93/21393/1
Jorge Hernandez [Mon, 30 Oct 2017 23:51:07 +0000 (18:51 -0500)]
clean up parent pom pre-release

remove also hardcoded references to snapshots that
won't work when released

Change-Id: If0c43699388e0bf700024b9428c8c21dccfe6797
Issue-ID: POLICY-404
Signed-off-by: Jorge Hernandez <jh1730@att.com>
6 years agoMerge "change parent pom for msb module"
Jorge Hernandez [Mon, 30 Oct 2017 18:05:45 +0000 (18:05 +0000)]
Merge "change parent pom for msb module"

6 years agochange parent pom for msb module 25/21325/1
Jorge Hernandez [Mon, 30 Oct 2017 17:06:36 +0000 (12:06 -0500)]
change parent pom for msb module

Change-Id: I7584c76ba8f7ba4c1d55d556c0d3f36eb1347ccc
Issue-ID: POLICY-402
Signed-off-by: Jorge Hernandez <jh1730@att.com>
6 years agoUpdate SO url from v2 to v5 09/21309/2
Hockla, Ali (ah999m) [Mon, 30 Oct 2017 16:14:54 +0000 (11:14 -0500)]
Update SO url from v2 to v5

Issue-ID: POLICY-401
Change-Id: Id1be63d2f61bc0d9d8f15df7573af2c7307c1c31
Signed-off-by: Hockla, Ali (ah999m) <ah999m@att.com>