Update dublin .gitreview

Update default.

Issue-ID: POLICY-1747
Change-Id: I7de69711426b33fc5327f69f9b73f9d4f62c86b4
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Update INFO.yaml to add Jim and Ram

Issue-ID: POLICY-1775
Change-Id: Ia6df04e2ac7e44140bfa61282f894ee57df1395a
Signed-off-by: liamfallon <liam.fallon@est.tech>

Step snapshot version of drools-pdp

Issue-ID: POLICY-1630
Change-Id: Ia77856077ae89f839aa930101c2002ac25f79595
Signed-off-by: liamfallon <liam.fallon@est.tech>

Upgrade to release policy/models

Version 2.0.0

Issue-ID: POLICY-1630
Change-Id: I0a18d4a5c52321b69757c07f48d6dd747606f9f3
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Ensure passive policies are injected when active

Add vim package for testing support

Change-Id: Icefcf8a57eea80b4fed97d4c63080f6159d8167a
Issue-ID: POLICY-1748
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Upgrade policy/parent & policy/common in drools-pdp

Change-Id: I5caa80f14e642d7c539c5b9df7da1f6770c68147
Issue-ID: POLICY-1630
Signed-off-by: ramverma <ram.krishna.verma@est.tech>

Track policies when active.

Do not honor group/subgroup changes with status messages.

When no present group/subgroups in update messages, assume that
they have to be reset.

Keep the policy cache of non-installed policies tracking them
when in passive state besides active.   Wait until go active
to enable them.

Remove legacy PAP and PDP-X healtchecks as it was basically
testing reachability.   There are conflicts with maintaining
2 different sets of PAPs and PDPs, trying to avoid it.
Alternative healtchecks are being placed on the robot test
framework.  What to healtcheck should be considered more
carefully in the near future.

Change-Id: I574f30bb5899faf521123c79046793d16b4a46e0
Issue-ID: POLICY-1748
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

relax check in features for multiversion deps

print a warning instead, and the base install one wins.

Change-Id: I666481e270bc03a8a7164e0fa947822d1ac8bbb6
Issue-ID: POLICY-1726
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Enable noop source support in controller

Additional telemetry apis for testing.

Change-Id: I60a61ffbbe86bbb0474777905de922434deabbe2
Issue-ID: POLICY-1716
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Change config dir owner to policy instead of root

Change-Id: I3318df8eb31568fe1c9ca06d588d18b355d7095b
Issue-ID: INT-998
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

New BUILD_VERSION needed as fixed in parent image

BUILD_VERSION could not be used in ENV variable
as it is cooked in the base image itself.  Need to
create a new one to reflect the drools build to
get by.

Change-Id: I681726f76e0ccfe84406f1414cad85edbcae52a8
Issue-ID: INT-998
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Add back BUILD_VERSION to Dockerfile

It is needed by drools and drools applications images and
were empty after using new base alpine images.

Change-Id: I76c0e1b01b6edc49e9fc8f69fdef4fe3baa22e48
Issue-ID: INT-998
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Manually pull the base alpine images.

Undo directory creation of base image to preserve
original layout pre-use of alpine common/base images.

Change-Id: I0e95cc2d6d6da03be66e3b0c2413676539909bb0
Issue-ID: INT-998
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Merge "Add telemetry API interface for Lifecycle"

Add telemetry API interface for Lifecycle

Some cleanup consolidating code in cire telemetry apis.

Issue-ID: POLICY-1687
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
Change-Id: Ic5241d45eba06796192cca1e82cba5ae53dc5aaf
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Rebase image on policy-common-alpine

Issue-ID: INT-1000
Change-Id: Ibd575b57c794a107d8fc890ffbe7303030984892
Signed-off-by: Dmitry Puzikov <dmitry.puzikov@tieto.com>

Add response section to PAP status message

Change-Id: I26bf9f515c8639c030e8beb9e8b37a6a632da686
Issue-ID: POLICY-1655
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

One year cert with new component names

Change-Id: I50afac9fc880d4dd5619093f917be8e55ee80fad
Issue-ID: POLICY-1580
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Change json files from typeVersion to type_version

Change-Id: I8544977c47c3b428b2d2cea568122f8927d8a4db
Issue-ID: POLICY-1647
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Send policies in status

Change-Id: Ia1f02857922a81c5d94ffc33cee72ce9a822c876
Issue-ID: POLICY-1624
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Code Changes done for OOF SON use case

Introducing control loop response flow using DCAE_CL_RSP topic

Change-Id: I90e30c061221e3d0c213fee9341b8b09a490fcae
Issue-ID: POLICY-1463
Signed-off-by: Saravanan A <saravanan.a75@wipro.com>

deploy and undeploy as audits.

Change-Id: Iea1766d5d8075394e1dccc109f4477bd2c270c36
Issue-ID: POLICY-1624
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Initial support for deploy

This is a first pass, some functionality is
missing (undeploy).

Compile error fix from changes in policy/models repo.

Change-Id: If448492ab665c135bace99d4d684d403e2a6be03
Issue-ID: POLICY-1624
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Add lifecycle feature to install package

Change-Id: Ia9edd30131d4dde22c9d2559b4ad1428fdc13f89
Issue-ID: POLICY-1608
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Handle pap update messages

Addressing feedback from previos review.

Issue-ID: POLICY-1608
Change-Id: I8e36c99b13a3eb81c72515c039e86aa9e32e6f14
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Initial lifecycle state machine.

Basic state machine and initial functionality to
transition to the Active state via Engine commands and
StateUpdate messages.

Change-Id: I8212bd264dbbea446d4e682918f9360e3984887f
Issue-ID: POLICY-1608
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Merge "Add MDC Topic Filtering"

Add MDC Topic Filtering

A feature to provide configurable properties for network topics
to extrac fields from JSON strings and place them in a mapped
diagnostic context.

Issue-ID: POLICY-1499
Change-Id: Icfca0be3b263ccf1612b79ed617e2b1ffb0317e6
Signed-off-by: Daniel Cruz <dc443y@att.com>

Support topic properties files.

They will be loaded automatically at startup.
Remove POLICY-PAP-PDP topic as is moved to a feature.
Remove PDPD-CONFIGURATION topic as it is not used.

Change-Id: I55629f885d61ce1cc4e3f24bcae5279e65a96f22
Issue-ID: POLICY-1610
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Remove OrderedService references

Removed OrderedService and OrderedServiceImpl from the
drools-pdp repo to use OrderedService and OrderedServiceImpl
from the common repo. All feature interfaces were updated
occordingly.

Issue-ID: POLICY-1499
Change-Id: Icc80dd1cbe73a97d4832f95ad2dad52d948b711a
Signed-off-by: Daniel Cruz <dc443y@att.com>

Manually add NSS libraries to images.

Alpine OpenJDK was updated 2 days ago (3/14), this
upgrade seems to break java applications that depend on libnss*.so
OS libraries, which seem are no longer installed with the jdk.

Before:

openjdk version "1.8.0_191"
OpenJDK Runtime Environment (IcedTea 3.10.0) (Alpine 8.191.12-r0)
OpenJDK 64-Bit Server VM (build 25.191-b12, mixed mode

ls /usr/lib/libnss*
/usr/lib/libnss3.so /usr/lib/libnssckbi.so.41  /usr/lib/libnssdbm3.so.41
/usr/lib/libnss3.so.41  /usr/lib/libnssdbm3.chk    /usr/lib/libnssutil3.so
/usr/lib/libnssckbi.so  /usr/lib/libnssdbm3.so     /usr/lib/libnssutil3.so.41

Now:

openjdk version "1.8.0_201"
OpenJDK Runtime Environment (IcedTea 3.11.0) (Alpine 8.201.08-r0)
OpenJDK 64-Bit Server VM (build 25.201-b08, mixed mode)

> ls /usr/lib/libnss*"
ls: cannot access '/usr/lib/libnss*': No such file or directory

Change-Id: I371d1b3426ded294dba16d4217f6c9ce5a5acf04
Issue-ID: POLICY-1604
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Update INFO.yaml file

Update missing information blocks and
indentation to pass verify job.

Change-Id: I3f057c3d0fd02a39391439a12d22f7db0e507f4e
Issue-ID: CIMAN-134
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Add alpine packages for better diagnostics

Change-Id: I96e30e2ffb7dc35ffc1e377c1021ee871ab5a310
Issue-ID: POLICY-1560
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

reducing kill timer to better play with k8s ones

Change-Id: Ibd69b100468716b929d42c9145fb72153cf685ed
Issue-ID: POLICY-1560
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

fixing problems related to move to Alpine base OS

Three problems are solved: (a) no amsterdam application comes
up, (b) wait for mariadb not working, (c) timeout does not work
(different syntax).

Change-Id: Ib46312201e8be06816b1339e543711f293225ca1
Issue-ID: POLICY-1551
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Merge "Tweaking scripts to install for  pdpd-cl image"

Tweaking scripts to install for  pdpd-cl image

Change-Id: I5f7ab5de402a5412938f42ae76e21f112b106b3a
Issue-ID: POLICY-1560
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Use Alpine as base docker image in drools-pdp

Issue-ID: POLICY-1551
Change-Id: I565644e048c789c3c71546ae00c0beb84bb6358a
Signed-off-by: Hengye <yehui.wang@est.tech>

Fix JAVA_HOME in docker image

JAVA_HOME is known at image creation.

Change-Id: I7274c47b5e890870d61949109ed2e1292a049eb9
Issue-ID: POLICY-1560
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Merge "pdp-d remove volume + install simplifications"

Merge "Replace PropertyConfiguration"

pdp-d remove volume + install simplifications

volume is unused and hides config dir when
inherited from pdp-d-cl.

small simplification in pdp-d install

Change-Id: I50d343b8d289f2f883bc31a6c650584343d8cd97
Issue-ID: POLICY-1560
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Replace PropertyConfiguration

Modified code to use BeanConfigurator instead of PropertyConfiguration.

Change-Id: I296d9047262fbc87f04c3db92f23181f588bfeda
Issue-ID: POLICY-1444
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Add Controller Logging Feature"

Merge "move all hard install config to environment vars"

Add Controller Logging Feature

This features provides a mechanism to extend the logback.xml
properties to add controller specific loggers. The controller's
logger will log messages from topics that the controller listens
to in a controller specific network log. The original network
log is preserved and still logs every message from every controller.
Note that the way a logger is associated with a controller is by
having the logger name match the controller's name. Any configuration
file that has "logback-include-" prepended and a ".xml" extension
will be added to the logback.xml and logback-eelf.xml files as
extensions to the base configuration.

Issue-ID: POLICY-1427
Change-Id: Iaeb823421eadb7ee413b6b03ae3dfe862f230612
Signed-off-by: Daniel Cruz <dc443y@att.com>

move all hard install config to environment vars

+ support multiple system properties files with variable interpolation
  loaded at initialization
+ support of configurable JVM options (-X, etc ..).
+ rearrange aaf configuration to avoid {{}} installation variables
  and use dynamic enviroment variables.
+ miscellaneous clean up in areas touched and checkstyle.

Change-Id: I71ad839778e17eb57c098a2c5cc2bf96e468669a
Issue-ID: POLICY-1524
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Update INFO.yaml Liam's email

est.tech

Issue-ID: POLICY-1556
Change-Id: I810dcac34ec078a07dfb2374067ecc06018fa7fb
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Base POLICY-PDP-PAP channel configuration

With these changes the PDP should come up listening in
the new PAP-PDP topic for messages, and configured to
send.

Change-Id: I395f87a7401aeb6385d640e51f75d71c0def36b0
Issue-ID: POLICY-1548
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Add getEffectiveTopic method to pooling test

A new method, getEffectiveTopic(), was added to the Topic interface.
A test class in the pooling feature uses that interface, thus the
method had to be added to the implementation within that test class.

Note: until this is merged, drools-pdp will not build.

Change-Id: Ib82e7a0d23f5b8e8eda08e8fc570e96543497da2
Issue-ID: POLICY-1534
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Add Nested JSON Filtering"

Add Nested JSON Filtering

The refactoring of the JsonProtocolFilter allows for complex filter
queries for JSON events coming in on network topics. The underlying
library used is json-path, which supports path based searching and
ruby regex filtering at any level of the JSON document.

The JsonProtocolFilter no longer requires a FilterRule class as one
json-path filter can contain multiple constraints using "&&" or "||".
This eliminates the need to identify the field name as a key with a
regex associated with it in the controller properties file (see the
original FilterRule implementation). It also simplifies the parsing
needed in the DroolsControllerFactory when extracting the filter
property and creating the JsonProtocolFilter.

JUnit coverage is 100% and all sonar flags were addressed related to
the JsonProtocolFilter class. Tested and verified working with the
telemetry API in a locally deployed PDP-D.

Change-Id: I8bd63db4e497c1ba0b5044b5449ccb7a9e4dbdbc
Issue-ID: POLICY-1489
Signed-off-by: Daniel Cruz <dc443y@att.com>

Moving hard install config to environment vars

Change-Id: I01e0cf2e2399ec81337961e0a2b3a349c34f55c4
Issue-ID: POLICY-1517
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Merge "Remove unneeded "throws""

Remove unneeded "throws"

Fixed sonar issue:
- remove the declaration of thrown exception
'com.google.gson.JsonParseException' which is a runtime exception

Change-Id: Ice29a875891a06a55932fcf80107d4dec2b10d1f
Issue-ID: POLICY-1533
Signed-off-by: Jim Hahn <jrh3@att.com>

Removed jackson from top-level pom

Change-Id: Ic9cc018e918527d610676b34f05da718f09d97b9
Issue-ID: POLICY-1529
Signed-off-by: Jim Hahn <jrh3@att.com>

Change properties to use gson

Modified property files to use gson instead of jackson for REST calls,
for both clients and servers.

Change-Id: Ib836886df33cd585b4c2d227e9cc496cc165fc2c
Issue-ID: POLICY-1431
Signed-off-by: Jim Hahn <jrh3@att.com>

Test gson in policy-management

Added jackson-like Gson annotations to policy-management classes.
Also added tests to verify that the classes serialize the same
with gson as with jackson.
Removed some trailing spaces.
Replaced tabs and adjusted spacing in json files.
Removed trailing newlines.
Updated license data per review comment.
Fixed merge conflict.
Added files that were inadvertently deleted.

Change-Id: Ib546e70f9d9d83187a6a93ff5c634000c4d53da5
Issue-ID: POLICY-1431
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Decouple nexus reconfiguration from installation"

Remove jackson from policy coders

Coders came in pairs, one for GSON and one for Jackson.  Replaced the
pairs with a single coder for GSON.
Simplified list copy.
Removed multipleToolsetRetries flag.
Removed additional jackson code.  It had been removed, but the changes
were somehow lost.
Simplified another list copy.
Updated license dates.
Note: the date in PolicyEngineTest will be updated via a
different review (mgmt-gson).

Change-Id: Id3c7688c2389d194f66afcefa14303cf403ada16
Issue-ID: POLICY-1431
Signed-off-by: Jim Hahn <jrh3@att.com>

Decouple nexus reconfiguration from installation

This allows for nexus to be reconfigured dynamically.

Change-Id: I103707ab7525ddb29663d7785739a34dff538605
Issue-ID: POLICY-1518
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Merge "Support of environment variables in properties"

Support of environment variables in properties

Change-Id: I5028ad3b4fb63a96607915e8b424dd2e57696bf0
Issue-ID: POLICY-1514
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Replace PolicyAssert with assertj

PolicyAssert has a sonar issue and is no longer needed, as the
same functionality (and more) is provided by assertj.  Modified
drools-pdp test code to replace PolicyAssert with assertj.
Updated license dates.
Removed trailing spaces.
Remove assertj version from poms, as it's now included in parent pom.

Change-Id: Ica82a959613e082dc6ed6fc9e103a198c2f17537
Issue-ID: POLICY-1393
Signed-off-by: Jim Hahn <jrh3@att.com>

Report KieBase names in rules artifacts.

The rule writer at design time can embed the domain base model(s) supported
at design time.

This information can be extracted at runtime to match a controller with the
domain/model functionality that it can support.

Change-Id: I7f7f1320d7d601d50949d0cd09bc48982c222333
Issue-ID: POLICY-1500
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Remove jackson from feature-pooling-dmaap

Modified code to use gson instead of jackson.

Change-Id: I5a1b2dacdc1801b1110154ed7c3c81e0713ef369
Issue-ID: POLICY-1431
Signed-off-by: Jim Hahn <jrh3@att.com>

Change Junit to check for error code 406

This is to support some clean up changes done in
policy/common under commit 55f5c4dc9e130e48a25b048e1f3091b10c17e365.

Change-Id: I2961d7ba719d22f144af972dc3964c3ac402e75c
Issue-ID: POLICY-1397
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Replace expectException with AssertJ

Change-Id: I5c9c5e962a7e2a5617405d534c0719f4bdb6ff1c
Issue-ID: POLICY-1393
Signed-off-by: Jim Hahn <jrh3@att.com>

Add tooling for drools policy developers

This commit adds tooling to better support development activities
for drools developers.

1. deploy artifacts to nexus or to local file repository on demand.
   It also allows to test on a single drools instance without the
   dependency on a nexus repo.

2. make installation configuration files conform to same
   convention (all upper case).

Change-Id: Ia0a2fc25db2ef21b774a9eee4ed51d4b876a1993
Issue-ID: POLICY-1367
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Merge "Environment variable pointing to logback.xml"

Environment variable pointing to logback.xml

Per current logging spec, the location is recommended to be
under /etc/onap/<component>/subcomponent but is acceptable
to have an enviroment variable pointing to it instead.

Environment variable is preferred over fixed location so
the config files are not all over the place, and is not
placed under directories owned by root.

Change-Id: Id5679f49a726fa43147ffb3e141acec30ebfdaaf
Issue-ID: POLICY-1294
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Update to latest keystore from AAF tooling

It is generic with extended expiration, and support for
policy-distribution and policy-apex-pdp components.

Change-Id: I02157b063159ed6f8fc71f00cf0f890059b20c7b
Issue-ID: POLICY-1377
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Merge "Fix sonar issue in droolspdf"

Fix sonar issue in droolspdf

Modify protocol coder to fix sonar issue about method
params too many. Using existing builder. Address review
comments, replace modified with modification

Issue-ID: POLICY-1251
Change-Id: I67bd753f346108865404b733e7cf6008c2012766
Signed-off-by: krisjinka <kris.jinka@samsung.com>

Update eclipselink version in drools-pdp

Use eclipselink from policy parent.
Re-insert scope=provided.

Change-Id: I325d13b7249747469d93bf451f5753cfbc728568
Issue-ID: POLICY-1361
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Remove Factory from various drools-pdp classes"

Merge "Fix sonar issue in event protocol coder"

Fix sonar issue in event protocol coder

Add method in Eventprotocolcoder uses more params than
allowed causing sonar issue, use builder object instead
Modify javadoc and tostring methods based on review comments

Issue-ID: POLICY-1251
Change-Id: I6f283135560bfa61ca9d9512f5c971d118d7e071
Signed-off-by: kris.jinka <kris.jinka@samsung.com>

Remove Factory from various drools-pdp classes

Also merged changes from 'Rename test classes in drools-pdp'.

Change-Id: I4d0a3a1b0b5523eadf38d1edb5aa86c404b69d19
Issue-ID: POLICY-1288
Signed-off-by: Jim Hahn <jrh3@att.com>

Rename test classes in drools-pdp

Make test classes name consistence

Issue-ID: POLICY-1281
Change-Id: I9632831dd36b1d362aeadfb53fda600762c835ca
Signed-off-by: Parshad Patel <pars.patel@samsung.com>

Modify decoder to use param obj

Modify decoder to use param object for sonar issue fix

Issue-ID: POLICY-1251
Change-Id: Iae68ba276f02be51058037b56d901a131b25838e
Signed-off-by: kris.jinka <kris.jinka@samsung.com>

Modify event (en)coder to use param objs

Use builder object to send params to event protocol
coder and encoder methods to fix sonar issue

Issue-ID: POLICY-1251
Change-Id: I6ca5823e1aa35d9aa3a05eb23ac159947efdde23
Signed-off-by: kris.jinka <kris.jinka@samsung.com>

Fix for Test failure issue in drools-pdp

Replace JAVA_HOME with PATH env variable in assert statement

Issue-ID: POLICY-1241
Change-Id: I2441d7be44a9578a3137a7a0b5747ec0799f0869
Signed-off-by: Parshad Patel <pars.patel@samsung.com>

Set new master version for Dublin

Issue-ID: POLICY-1232
Change-Id: I768c9f4e0e1411854fe6aadb52abbc616654127d
Signed-off-by: liamfallon <liam.fallon@ericsson.com>

Remove openecomp repo from drools-pdp pom

Change-Id: I44437c34f79fe7f169f38715bfd88d5ebfab93e4
Issue-ID: POLICY-1231
Signed-off-by: Jim Hahn <jrh3@att.com>

Enable Authorization AAF Filter programmatically

The intent is to reduce configuration options and
automatically set up the authorization filter
(that will complement the authentication filter
that the policy-endpoints common framework automatically
provides).

Previously to enable the AAF authorization filter,
it will need the configuration had to kept track of
AAF flag, and the "filterClasses" property configuration,
make a bit more difficult to keep them sync, and easily
turn on/off AAF.

Change-Id: I3d59de6fa83986c5dcb6b3d38077278b8e75689a
Issue-ID: POLICY-1216
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Fix refering to snapshot in policy/drools-pdp

Issue-ID: POLICY-1213
Change-Id: I485c15044ba5cc0d3daceaf0876f543cc5c8af82
Signed-off-by: ramverma <ram.krishna.verma@ericsson.com>

Updated the keystore to include pdp wildcard

This allows to talk directly to OOM PDP-X pods
without going through the service

Change-Id: I9acd171f483c28473442afad80361be374bd4901
Issue-ID: POLICY-1198
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Regenerate policy keystore

The keystore has been regenerated using the latest tooling
from AAF.

Previous versions seem to add a bogus " fqdn" which makes
the the particular policy.api.simpledemo.onap.org entry invalid.
The latest version does not append it.

This change is done for consistency with policy/engine for
which proper format of this entry is necessary to interact
with CLAMP.

Drools is never accessed with this identity, therefore the
changes are complimentary and do not require release of new
artifacts.

Change-Id: Ibf6c2e7130dda247ac320f334765e04d369d6b20
Issue-ID: POLICY-1193
Signed-off-by: Jorge Hernandez <jh1730@att.com>

Add coverage to test-transaction

Fixed bug in test-transaction, wherein it was sometimes using the controller
instead of the controller name as the key into the map.
Changed test to use latch instead of sleep.
Added a little more register/unregister testing.

Change-Id: I2647b44cf164038d211063bd499c25af70c2d9d7
Issue-ID: POLICY-1148
Signed-off-by: Jim Hahn <jrh3@att.com>

Add coverage to policy-core

Added easy junit test coverage cases.
Fixed sonar issue: removed sleep from DroolsContainerTest.
Fixed sonar bug in ClassExtractors.

Change-Id: I942badf17c42346c1735bc3951450fc31c02a769
Issue-ID: POLICY-1148
Signed-off-by: Jim Hahn <jrh3@att.com>

Add junit coverage to PolicyEngine class

Also fixed a typo in a test comment.

Change-Id: I4ad72cc7c103014e6f5920f912c862560af5a331
Issue-ID: POLICY-1148
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Add coverage for policy-management"

Merge "Add more coverage to drools-pdp health check"

Add coverage for policy-management

Added coverage for PolicyControllerFactory and AggregatedPolicyController.
Fixed some typos in comments.
Reformatted some code.

Change-Id: I33aea8e1e7dde29bd51218d0ecad7b34047b33e5
Issue-ID: POLICY-1148
Signed-off-by: Jim Hahn <jrh3@att.com>

Add more coverage to drools-pdp health check

Added getProperties() method to Factory, as the system properties could
not be found in the jenkins build, for some reason.
Removed Factory, as not needed for junit testing.

Change-Id: I5070b9502ae06be6995cccbb005653928591512c
Issue-ID: POLICY-1148
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Add junit coverage to drools-pdp"

Recreate latest keystore

Some changes have been made since the first set of
stores were generated from the cert manager tooling
in the windriver test instance used for this purpose.

Change-Id: Ic9a0b056fb2c4e8fb23a0c0e0462087b8b7125da
Issue-ID: POLICY-1156
Signed-off-by: Jorge Hernandez <jh1730@att.com>

Add junit coverage to drools-pdp

Added coverage to distributed locking.
Added coverage to test-transaction - still more to do.
Added coverage to healthcheck - still more to do.
Also fixed "code smell" in pooling.
Also fixed "code smell" in session persistence.
Fixed typo in comment.
Removed unneeded setUp() method from test.
Fixed new checkstyle errors.
Fixed another new checkstyle error.

Change-Id: I0d2ab13fcbd64486af88affb02e114e624c6a3d1
Issue-ID: POLICY-1148
Signed-off-by: Jim Hahn <jrh3@att.com>

Set drools-pdp checkstyle config

Fixed last remaining checkstyle issues along with
setting the pom.xml to the correct path setting.

Fixed forbidden summary fragments, not allowed
to prepend summary with "This method returns"

Issue-ID: POLICY-1153
Change-Id: I267842080494132a7b5507731e14f027a3f6fe44
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>