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>

Add demo for Control Loop Coordination.

New PipEngine and Junit test added to guard.
Also bug in existing Junit test fixed.

All other code added to new directory template.demo.clc

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

Merge "remove volumeGroup from SO request"

push-policies https for casablanca controller

https support for provisioning policies in
experimental casablanca controller

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

remove volumeGroup from SO request

After a new round of discussion with the SO team, it was learned that
the SO scale-out request should NOT include the volumeGroup in
relatedInstanceList[0]; that item should be eliminated altogether.
Re-ordered some imports.
Updated license date.
Left volumeGroup in TestDemo, as it's a pseudo VID request.

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

Merge "Option to disable target locking. Needed by CLC."

Merge "Fixed sonar code smell"

Fixed sonar code smell

removed unused method parameter aaiResponseWrapper from
constructDeleteRequest

Change-Id: Ie434130b040732ed4e656c5592fff5b049695d29
Issue-ID: POLICY-1079
Signed-off-by: Piyush Garg <piyush.garg1@amdocs.com>

fix NPE from rule timers

This problem occurs in rules that have "timer(expr)" expressions before
the "when" clause.  If a working memory element, other than the timer
object, appears in the "when" clause and is updated BY ANY RULE, then
an NPE is thrown.  The solution is to segregate the objects so that
rules having "timer(expr)" expressions only refer to one object, the
XxxTimer object.  In particular, the rule simply sets an "expired" flag
in the XxxTimer object and then all other rules check that flag, rather
than having their own "timer(expr)" expressions.

Corrected comment associated with rule TIMER.FIRED.

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

Option to disable target locking. Needed by CLC.

When used, the ControlLoopEventManager will set useTargetLock to false
converting TargetLock lock/unlock operations to no-ops.

Allows CLC-specified logic to coordinate closed loops instead
of hard-coded mutual-exclusion enforced by target locking.

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

implement Serializable in additional classes

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

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

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

disable http for drools apps using dmaap

per security guidelines ..

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

Added min and max vnf count support

Added variables for min and max with associated Constructor,  getters and setters
We will need to merge this commit before merging the policy/engine
commit to avoid compilation errors in policy/engine build

Made changes to address Jim Hahn's comments

Change-Id: I45357077cfe046028adcc0033ecf29f8cc1a525b
Issue-ID: POLICY-1038
Signed-off-by: Mike Mokry <mm117s@att.com>

Fix new sonar issue with vfModuleItem

find the last non-base vf module item from AAI response and use the same
to construct create or delete vf module SO request

Change-Id: I3b9b97ac3008f6e4f7952662d9e4262f77dbeec4
Issue-ID: POLICY-1079
Signed-off-by: Piyush Garg <piyush.garg1@amdocs.com>

Merge "Added support for VF Module Delete recipe"

Merge "Added junit test for commitAbatement."

Add appc lcm request formatter

Formatter is used to dash case recipe in URL and camel case it in body.

Issue-ID: POLICY-1083

Change-Id: I908ae93665f3d069b3a032ae378b7775ebe1110a
Signed-off-by: Gabriel <adam.krysiak@nokia.com>

Added junit test for commitAbatement.

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

Added support for VF Module Delete recipe

Enhanced SOActorServiceProvider to support VF Module Delete Recipe.
Added SOOperationType enum to map the policy recipe with the SO API that
we want to call for a recipe. vf module id from non-base vf module is
used to construct the delete request url and same will be deleted.

Updated SOManager so that it sends request to SO based on the
SOOperationType enum value.

Enhanced RESTManager to add support for the http delete using new class
HttpDeleteWithBody (to allow pass the payload to the SO delete VF module API).

Change-Id: I15b678ed9ebc85dfa7cb62bbf23e41c0fe6e4c69
Issue-ID: POLICY-1079
Signed-off-by: Piyush Garg <piyush.garg1@amdocs.com>