Drools-App changes to support CQ

Changes in event manager to support Aai custom query

Issue-ID: POLICY-1278
Change-Id: I693dd6b94f1fa6f24b08a389db099914636cf2ba
Signed-off-by: pramod.jamkhedkar <pramod@research.att.com>

Parse new model ids from operation policy

Takes VFModule info from policy yaml,instead AAI.

Change-Id: If745b3ba32562cbc37571cea29ec7797ee697e88
Issue-ID: POLICY-1545
Signed-off-by: vamshi.nemalikonda <vn00480215@techmahindra.com>

Code changes done for OOF SON use case

Added a new control loop for SON ANR changes
Introducing control loop response flow using DCAE_CL_RSP topic

Change-Id: I81d0e92ce2f5c489596ad70d7b523cab0d8436ce
Issue-ID: POLICY-1463
Signed-off-by: Saravanan A <saravanan.a75@wipro.com>
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

update dependency to policy/models

Update the dependency of model-impl from drools-applications
to policy/models.

Issue-ID: POLICY-1264
Change-Id: I07fcb29501499a97689d2d27c845bf8e411e7825
Signed-off-by: shaoqiu <tim.huang@est.tech>
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

update dependency on drools-applications/tmp

update dependencies from policy/models

Issue-ID: POLICY-1264
Change-Id: Idd7809b11ae9e63ca75d49d78a9e7b52a63e49b9
Signed-off-by: Hengye <yehui.wang@est.tech>
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Add CCVPN Bandwidth on demand policy"

Add CCVPN Bandwidth on demand policy

CCVPN Bandwidth on demand policy

Issue-ID: POLICY-1405
Change-Id: I67bceb35e5a849933b3e46772c9cbbbaab1e9a75
Signed-off-by: Vidyashree Rama <vidyashree.rama@huawei.com>

Add lifecycle feature to image.

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

Rename "casablanca" controller to "usecases"

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

do not enable controlloop-utils feature

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

Change LoggerFactoryWrapper to NetLoggerUtil

Removed LoggerFactoryWrapper references to use NetLoggerUtil.

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

Update INFO.yaml file

Update missing information blocks and
indentation to pass verify job.

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

Baking "runtime" software into image

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

Merge "Fix policy doesn't send payload to APPC"

Pull right version of pdpd image in docker-cl

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

Fix policy doesn't send payload to APPC

Fix policy doesn't send payload to APPC using AppcLcm

Change-Id: Ic3cb59203a412297f40f77d744cc86e4dddff7a2
Issue-ID: POLICY-1522
Signed-off-by: awudzins <adam.wudzinski@nokia.com>

Place most of the binaries in the pdpd-cl image

This is work in progress although most of the software will
be loaded with these changes.  pdpd-cl is not automatically
started just yet.  It will be done in another review.

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

Merge "Add controller logger configurations"

Add controller logger configurations

Logger configurations have been added for each feature that
installs a controller. The convention that the controller
logging feature looks for in a controller logback config
file is "logback-include-.*.xml".

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

Replace hard install vars with environment vars

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

YAML clc directive spec/compilation to XACML

Created java definition of CoordinationDirective class.
Initial library supports one coordination directive FirstBlocksSecond,
prototype for this directive added as firstBlocksSecond.xml.
Functions for compiling example yaml directive are in SupportUtil.

Change-Id: If7c22402729670403510f1fa2f7f43de6a4a522c
Issue-ID: POLICY-1173
Signed-off-by: Joshua Reich <jreich@research.att.com>

Merge "pull + tag base image for ci workings"

pull + tag base image for ci workings

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

Change Controller Filters

The controller properties files have been updated to use
json-path filters in support of the drools-pdp infrastructure
changes.

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

Removed null custom coders

Some code still referenced the customJacksonCoder() method, which
has been removed from drools-pdp.  However, all the code did was
set the jackson & gson coders to null, which is unnecessary, as
that is the default value for both.  Thus the code was removed
altogether so-as not to cause issues in the future.
Updated license dates.

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

Merge "Consolidate xacml dependencies"

Merge "basic image with CL app preloaded"

basic image with CL app preloaded

The image can replace the drools-pdp and only
overrides what is necessary to make sure the CL
application is in the image.  There should be
no functional difference.

Tooling to operate with docker images is provided
as well, for example:

controlloop/tools/docker-cl --help

BUILD INFO:
    image: policy-pdpd-cl
    image-path: controlloop/packages/docker-controlloop/target/policy-pdpd-cl
    image: policy-pdpd-cl
    registry: nexus3.onap.org:10003
    patch: 1.4.0-SNAPSHOT
    version: 1.4-SNAPSHOT
    timestamp: 20190207T220835

TAGS:
    BUILD:
        verify:
            onap/policy-pdpd-cl:latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4-SNAPSHOT-latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4.0-SNAPSHOT-20190207T220835Z
        merge:
            onap/policy-pdpd-cl:latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4-SNAPSHOT-latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4.0-SNAPSHOT-20190207T220835Z
        release:
            onap/policy-pdpd-cl:latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4-SNAPSHOT-latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4.0-SNAPSHOT-STAGING-20190207T220835Z
    PUSH:
        verify:
        merge:
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4-SNAPSHOT-latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4.0-SNAPSHOT-20190207T220835Z
        release:
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4-SNAPSHOT-latest
            nexus3.onap.org:10003/onap/policy-pdpd-cl:1.4.0-SNAPSHOT-STAGING-20190207T220835Z

SYNTAX:
    docker-cl
        [--help|-h] [--build|-b <repo:tag>] [--push|-p <repo:tag>]
        [--verify|-v] [--merge|-m] [--release|-r]
        [--run <volume-full-path>] [--cmd <volume-full-path> <cmd>]

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

Consolidate xacml dependencies

The xacml dependency appeared in multiple drools-applications
pom files, with different exclusion lists.  The dependencies
have been consolidated into a single pom, with a combined
exclusion list.
Added comment to the exclusion list.

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

Remove msb project from drools-applications

The msb project is no longer needed, so it has been removed.

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

Assign control loop domain to rules artifact

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

Use helper class to retrieve network logger

Instead of abusing constant from Topic interface use new dedicated
helper class to retrive network logger.

Issue-ID: POLICY-1486
Depends-On: Ic1f07466e1ab6abcc00098e617f0e82da19e852d
Change-Id: I453a14c3dda337f2a4d08914ead32b37adc17353
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Remove jackson from policy/drools-applications

Excluded jackson-databind from xacml.jar dependencies.

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

Remove guard checkstyle suppressions

Renamed a number of fields and methods, in the Guard classes,
to conform to camel-case format.
Also updated some toString() methods to return "xxxId=" instead
of "xxxID=", if the corresponding field names were also xxxId.
Updated license data in one changed file.

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

Remove eventmanager checkstyle suppressions

Renamed enum class name and various method names in ControlLoopEventManager.

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

Remove model-imp/rest checkstyle suppressions

Renamed "RESTManager" class to "RestManager".  This did not require
any changes to rules.
Updated license data in pom.

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

Remove model-impl/so checkstyle suppressions

Renamed classes from "SOxxx" to "SoXxx".  Also renamed a few method
names to make them camelcase, too.
Updated drl files with above changes.

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

Merge "Remove trafficgenerator checkstyle suppressions"

Merge "Remove model-impl/vfc checkstyle suppressions"

Remove trafficgenerator checkstyle suppressions

Changed class names from "PGxxx" to "PgXxx".

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

Remove feature-cl-trans checkstyle suppressions

Extracted CacheBasedControlLoopMetricsManager into its own class file.

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

Remove model-impl/vfc checkstyle suppressions

Entails renaming VFCxxx classes and updated drl files, as well.

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

Restructuring packaged features installation

Some cleaning on the installation.

feature-controlloop-management will be installed by default and will contain
new capabilities plus common functionality from amsterdam and casablanca.

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

Merge "Add sdnc simulator in controlloop-utils"

Add sdnc simulator in controlloop-utils

Add sdnc simulator in controlloop-utils

Issue-ID: POLICY-1356

Change-Id: I999a402370c2ebeeff75d647bd1d72c42dd598f7
Signed-off-by: Vidyashree Rama <vidyashree.rama@huawei.com>

Telemetry extension to manage Operational Policies

This is work in progress to directly work with Operational
Policies with the casablanca controller for now.

It is part of prototyping efforts to enhance the management
of control loops, but the functionality can be used in
production environments.

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

Merge "Update eclipselink version in drools-apps"

Update eclipselink version in drools-apps

Use eclipselink from policy parent.

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

Merge "Reduce resp values meth complexity"

Reduce resp values meth complexity

The response values determining method has more complexity
reduce it using enums. Fix build fail due to incorrect
response value. Modify LCM response code to enum. Fix
enum package. Modify status code enum. Add modification
copyright notice. Fix review comments. Add serialization key

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

Rename test classes in drools-applications

Make test classes name consitence by adding 'Support' or 'Dummy' at start of util or dummy type of test classes

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

Remove unused imports in controlpolicy

Remove unused imports that got introduced in recent commits
Fix issue with compile failure of test class due to dependency
in drools-pdp Datamodel classes. Fix test failures due to
compile issues

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

Merge "Rename test classes in drools-applications"

Rename test classes in drools-applications

Make test classes name consistence by putting 'Test' at end as per google java style guide

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

Merge "Use parameter obj for policy results"

Fix guard history database configuration

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

Use parameter obj for policy results

Policy result method uses more params than allowed
Use parameter object for setting params instead

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

Merge "Close timing loop-hole when YAML updated"

Use builder for trigger policy

Fix an issue reported by sonar for using more than
7 parameters in the method.

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

Close timing loop-hole when YAML updated

Noticed when YAML updates are pushed close together that some
Params objects are deleted that should not be.  Fixed that by
eliminating the "active" list concept and only deleting a Params
object that has an associated cleaner.  If it has no cleaner,
then it can't be deleted, regardless of timing with rule updates.

Added more tests for rule updates, including cases to check
for event objects being retracted, as well as Params objects
being retracted when rules are deleted.

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

Merge "Apply builder pattern for policy"

Apply builder pattern for policy

Fix an issue reported by sonar for using more than
7 parameters in the method. Fix imports order in
files. Fix line lengths to less than 120 chars

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

Merge "Cleanup all Param objects when any rules change"

Cleanup all Param objects when any rules change

Fix license date for new junit test.
Updated some comments.
Ensured different control loop names in different rule sets of new junit
test.
Used different mechanism for examining facts in new junit test.
Fixed typo in comment.
Changed the way new junit checks for existence of facts.
Fixed another typo in comment.
Updated clean-up rules in CLC template and added junit test to verify
that Params are cleaned up appropriately.
Removed code that retracts $manager objects so that the manager clean-up
rule will be used, instead, thus ensuring that all associated objects are
deleted when the manager is deleted.  This was done for amsterdam, CLC,
and casablanca templates.
Added an extra fireAllRules() after calls to fireUntilHalt() to enable
clean-up rules to fire in junit tests.
Address Josh's review comments.
Enhance junit test to compare actual object references to ensure that
Param objects are as expected.

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

Exclude commons-collections

This should have been carried over from template.demo.

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

Merge "Re-added requestParams to vDNS policy"

Re-added requestParams to vDNS policy

Undo inadvertent change to compiler test.

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

Merge "Upgrade xacml 1.0.2"

Upgrade xacml 1.0.2

This version of xacml only has license changes to the java
source files. This will clear ONAP license issues.

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

Changed actor from "MSO" to "SO"

Also added "VF Module Delete" to the list of recipes for "SO" accepted
by the compiler.
Removed "AOTS" and "SDNO" from the list of accepted actors, as code does
not exist for those actors.
Update remaining actors and recipes.  Left "AAI" in the list for now,
though it has no recipes listed (and no actor.aai code).
Also removed "AAI" from the list of accepted actors.

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

Remove single quotes from casablanca template

The shell scripts used to populate the operational policies for
the casablanca template do not work if the yaml includes single
quotes.  Replaced the single quotes with %27 instead.

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

Merge "Added config params to scale-out policy"

Merge "Add /serviceInstances/ to SO URL for scale-out"

Added config params to scale-out policy

Removed requestParameters and reverted name.

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

Add /serviceInstances/ to SO URL for scale-out

Also reformatted line that was subsequently too long.

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

Set new master version for Dublin

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

Remove openecomp repo from pom

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

Fix refering to snapshot in drools-application

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

Add CCVPN policy

CCVPN policy

Issue-ID: POLICY-1209

Change-Id: I8adea233f8672c9ff43b4a169b50336d9e43c91d
Signed-off-by: Vidyashree Rama <vidyashree.rama@huawei.com>

Add missing SDNC and fix JUnit

Identified in prior review, ensure all the new actors are
in the appropriate pom.xml and minor fix to JUnit for
assertEquals vs assertTrue.

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

Merge "Add implementation for OOF PCI use case"

Add implementation for OOF PCI use case

Receive DMaaP message from PCI-Handler MS with PCI Config
change recommendations through DCAE_CL_OUTPUT topic.

Trigger SDN-R (if allowed by policy) by sending
DMaaP request through SDNR-CL topic.

When the response is received from SDNR through
SDNR-CL-RSP topic, just parse and print.

Code review comments addressed
Added controller changes into amsterdam too.

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

Remove unused endpoint dependency

This should have been cleaned out when we moved the code into
policy/common. The release job fails since it looks for a
non-existent release artifact for this.

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

Add new actor SDNC to support CCVPN Closed Loop

Issue-ID: POLICY-1183
Change-Id: I1b77b5b9cefca104382d9d84dd00bbd63b20e0f2
Signed-off-by: Xin Miao <xin.miao@huawei.com>
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Fixing sonar issues

Some trivial functional interface and ordering.

Added some missing JUnit tests.

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

Merge "Fix remaining sonar"

Fix remaining sonar

Mostly moving variables to their right spot.

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

Merge "Add sdnr actor to archetype"

Merge "Yet more sonar issues in drools-applications"

Merge "Fix more sonar issues"

Add sdnr actor to archetype

The casablanca template had the declaration. I made
sure the declarations were in the same place.

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

Fix more sonar issues

Most of these are minor issues along with a few others.
They have been bugging me so I thought I would get them
done. Some also do not show up on sonarlint.

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

Yet more sonar issues in drools-applications

removed unused imports and variable definitions
moved variable declarations above methods
refactored deeply nested if/while
removed code smell due to unecessary use of static keyword
moved string literals to left side of comparator
fixed logging of exceptions
immediately return expression instead of assigning it to the temporary variable

Change-Id: Idfd2a2bdf3c2f3bbfcb54a06b713d57bc867b3ab
Issue-ID: POLICY-1129
Signed-off-by: Joshua Reich <jreich@research.att.com>

Merge "Use powermock artifacts from parent pom"

Fix for checkstyle declarations

I found the problem with policy-yaml. An exception was
occuring due to an extra line between an @return for
javadoc vs. just printing a warning.

See: https://github.com/checkstyle/checkstyle/issues/2398

I also messed up some of the other declarations in the
sub-modules.

Copyright

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

Use powermock artifacts from parent pom

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

Fix sonar critical

Sonar complains that the password is an actual password when
in fact we are doing exactly the fix they suggest.

I seem to not have access to resolve it as a false positive.
So it seems the better way to do this is rename this to
reflect that it is actually a property value.

Had to also address 120> due to new variable name size.

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

Fix checkstyle declarations

I fixed what I thought would be tolerable for this release.
Same issue as policy/common I had to defined the checkstyle
in each repo where the suppressions were needed.

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

more sonar issues in drools-applications

Modified SO code to not return null lists for various buildXxxParam
methods.
Fixed commit message.
Added additional junit coverage to SO code.
Added additional junit coverage to SO code - cover more error cases.
Extracted another constant.

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

new sonar issues in drools-applications

Don't return null lists from AAI.
Transient TargetLock.
Extract aai.url, et. al., constants.
Don't allocate extra unneeded variable.
Re-order fields in xacml attributes.
Extract "vserver" constant from simulator.
Replace thread sleep with join.

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