Unit tests for various classes

Issue-ID: POLICY-1916
Change-Id: Ie7cafa16ce12ca542a4e76307caddb36b7753990
Signed-off-by: RossC <ross.carter@est.tech>

Minor fixes in examples-vcpe

Issue-ID: POLICY-1916
Change-Id: Ie3d1c33648e7996ac063bbaac6ea11f3df03564b
Signed-off-by: Ram Krishna Verma <ram_krishna.verma@bell.ca>

Minor fixes in examples-grpc

Issue-ID: POLICY-1916
Change-Id: I4bd93d72f234e1ce572b11bcb0621078480e061b
Signed-off-by: Ram Krishna Verma <ram_krishna.verma@bell.ca>

Point to latest SNAPSHOTs

Glassfish security fix in parent, also fixes in common/models

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

Merge "Fix sonar issues"

Bump policy-apex-pdp to 2.3.2-SNAPSHOT

Change-Id: Ic668ab20498bdc41edc78f52c7ce9d89a1385f19
Issue-ID: POLICY-2378
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Release 2.3.1 apex docker image

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

Fix sonar issues

Issue-ID: POLICY-1913
Signed-off-by: huaxing <huaxing.jin@est.tech>
Change-Id: Id1a9a6b6f7a9c8c74dd69bd0d30af05162a92cd1

Release apex 2.3.1

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

Merge "Point to released common/models"

Point to released common/models

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

SONAR issue fix

Issue-ID: POLICY-1913
Signed-off-by: aditya.puthuparambil <aditya.puthuparambil@est.tech>
Change-Id: Ie6dccc50ad63c5fdca1e79d7985aec2455041b56

Point common/models fix

Fix for naming policy and yaml string conversion.

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

Point to released parent, common models

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

Make sure Editor's css overrides common css

Issue-ID: POLICY-2446
Signed-off-by: huaxing <huaxing.jin@est.tech>
Change-Id: I1bc45367884ec78436a97256ac4286ab79c8a6b7

Fix intermittent unit test failures WebSocket

The Web Socket server is marked as started before the start process has
complted. This change fixes that issue.

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

Fix intermittent unit test failures reseterquestor

When consumers and producers are paired as in the case of the REST
Rquestor, both sides must come up and be wired together in the
initiation phase of apex-pdp before the consumers and producers start
handling envents.

In the ApexActivator class, the consumers were started immediately after
they were initialized meaning that a consumer could return events to a
producer that had not started yet.

This change fixes the ApexActivator so that it waits until all consumers
and producers are initialized before starting event handling.

It also fixes the timings on RestRequestor tests and tidies up the unit
tests.

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

Merge "Refer io.netty from policy/parent"

Refer io.netty from policy/parent

Issue-ID: POLICY-2467
Change-Id: Iea562574f466775f4dd443fc4ef9ecaa6ba610fd
Signed-off-by: Ram Krishna Verma <ram_krishna.verma@bell.ca>

Fix failing Kafka tests

All the Kafka components need something near 10 seconds to come up
completely. This review tweaks the timing to allow the test Kafka server
to come up and to allow the consumers to connect to it.

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

Merge "Fix hanging timeout on Executor test"

Merge "Fix UUID missing in policy editor"

Fix hanging timeout on Executor test

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

Fix UUID missing in policy editor

Issue-ID: POLICY-2445
Signed-off-by: huaxing <huaxing.jin@est.tech>
Change-Id: I7fda1961e95b1f18da6c0e22c7799195fc511443

My First Policy State Finalizer Example

Issue-ID: POLICY-1122
Signed-off-by: aditya.puthuparambil <aditya.puthuparambil@est.tech>
Change-Id: I552b58f39325982f1e26e59b352dfbe70cf35e7c

Merge "Fix State Finalizer Prepare and Cleanup"

Fix State Finalizer Prepare and Cleanup

The prepare() and cleanup() method were not called on state finalizers
for states. This review adds calls for them.

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

Bump version of Infinispan for security

Change-Id: Ie01b221e182dd43e85e3e7277bb55169f85a1460
Signed-off-by: liamfallon <liam.fallon@est.tech>
Issue-ID: POLICY-2449
Change-Id: Icbb7a769eb8f0d9ef278d222ab007f330576abb3
Signed-off-by: liamfallon <liam.fallon@est.tech>

Merge "Fix hanging tests in JavascritExecutor"

Fix hanging tests in JavascritExecutor

Added tests to check threads start, execute, and stop correctly and
fixed JavescriptExecutor class to handle startup and shutdown correctly
without hanging.

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

Fix timeout issues in gRPC tests

When running the gRPC tests in a slow or overloaded environment, the
timeupts must be set longer.

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

Point to latest snapshots

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

Merge "Point to models 2.2.2"

Point to models 2.2.2

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

Bump policy-apex-pdp to 2.3.1-SNAPSHOT

Change-Id: I59b93fccb0f3a41f8db753d1e8e2978bbb51e9a0
Issue-ID: POLICY-2378
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Release policy-apex-pdp 2.3.0 docker image

Change-Id: I7ad38253cab457f92912e17ea9eaf8300347f62e
Issue-ID: POLICY-2378
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Release policy-apex-pdp 2.3.0 artifacts

Change-Id: Id23e5c848b6a338f09bbaecbf89e184e4e9f4e4f
Issue-ID: POLICY-2378
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Ignore JUnit tests that hang on await

This change ignores the tests that are hanging in Jenkins until the Apex
images are released. Once the images are released we'll fix these tests.

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

Add timeout on Javascript latch awaits

It is possible that the initiation and especially shutdown happens so
fast that the execution thread counts down the latch before the caller
calls await(), in which case the Javascript Rhino thread will hang. This
review adds a timeout on the await calls.

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

Reduce size of client artifacts

This change:
- Removes separate packaging of the three clients
- Reduces the size of the combined client by suppressing unused
  dependencies
- Removes the Servlet code from the Docker packaging
- Removes the client WARs from the docker package

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

Launch separate threads for Javascript task execution

When a policy is loaded, a separate thread is spawned for each
Javascript script executor. This allows us to precompile the Javascript
scripts and also to have a larger stack available for script execution.

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

Point to released artifacts

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

Adding example for APEX gRPC plugin for CDS interaction

Change-Id: I8361eb0cbe74c3548ca98f3005bc37f5a1f91747
Issue-ID: POLICY-2436
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Merge "Apex-pdp tutorial documentation changes"

Apex-pdp tutorial documentation changes

Issue-ID: POLICY-2424
Signed-off-by: aditya.puthuparambil <aditya.puthuparambil@est.tech>
Change-Id: I92491c904b65a0db93c9e6eaba676c653c276175

renew policy certs till 03/17/2021

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

Fix minor checkstyle/eclipse warnings

Some minor checkstyle and eclipse warnings have crept into the code in
the last few weeks. This review cleams them up.

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

Merge "Convert examples to Rhino Javascript format"

Merge "Fixing the gRPC consumer side"

Convert examples to Rhino Javascript format

All examples are converted to Rhino Javascript from Nashorn Javascript.

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

Merge "Sonar Security vulnerabilities fix"

Fixing the gRPC consumer side

Change-Id: I21d9253f41eee9b958e8fb723f6c19f266502cef
Issue-ID: POLICY-1656
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Adapt vCPE example for Rhino Javascript

Minor changes to Javascript source for execution under the Rhino script
engine.

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

Sonar Security vulnerabilities fix

Issue-ID: POLICY-1913
Signed-off-by: aditya.puthuparambil <aditya.puthuparambil@est.tech>
Change-Id: Ic86e04776c9300e37134210cd9db5b6d7e6a5a9e

Merge "Sonar Bug fix"

Use Rhino javascript executor in apex-pdp

Integation unit tests resultd in StackOverflowException errors in the
Graal Javascript interpreter. Following extensive testing and
trouobleshooting, it proved very difficult to fix these issues in Graal
as the stck overflow errors were occurring in native class instantiation
methods being invoked by Grall on the JVM.

The alternative Rhino Javascript engine is developed by mozilla, and was
incorporated into the Java 6 JVM and evolved into Nashorn in the Java 8
JVM. Oracle dropped Nashorn in Java 11. However, in parallel, Rhino
development has continued.

This review brings in the Rhino javascript engine into apex-pdp as the
replacement for Nashorn and instead of Graal. Graal seems to be pretty
unstable as yet so we may bring it in in future releases but for now
Rhino is a more stable and reliable alternative.

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

Sonar Bug fix

Change-Id: Icb465fdcf5a580f599a111ed3c5d3510f5346f64
Issue-ID: POLICY-1913
Signed-off-by: aditya.puthuparambil <aditya.puthuparambil@est.tech>

Merge "Adding gRPC plugin to dependencies in APEX"

Adding gRPC plugin to dependencies in APEX

Change-Id: Iee8fdca9f23b432188a61054fe177283fdddc1ba
Issue-ID: POLICY-1656
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Fixing SONAR bug + code smell

Change-Id: If76ddc860b67e0d8dcd4c48bacb5436ffc773faa
Issue-ID: POLICY-1913
Signed-off-by: aditya.puthuparambil <aditya.puthuparambil@est.tech>

Merge "replace test sleep() with awaitality package"

Merge "Preparing integration tests for Graal Javascript"

Merge "GRPC support for APEX-CDS interaction"

Preparing integration tests for Graal Javascript

Graal enforces Javascript more strictly than Nashorn does. This review
changes the tests in apex-pdp to comply with the stricter -Javascript
checking, re-enabling the integration test module.
- All log calls must be passed as strings, using toString()
- Byte/Float/Long Java types not supported in Javascript

Disable integration tests so that other changes can be brought in.

JMS integration test is disabled for now, it will be re-enabled in a
review shortly.

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

replace test sleep() with awaitality package

Signed-off-by: Henry.Sun <henry.a.sun@est.tech>
Change-Id: I305771ddef42bd3032ad52f4c5ecd55b01ed5a1a
Issue-ID: POLICY-1914
Signed-off-by: Henry.Sun <henry.a.sun@est.tech>

GRPC support for APEX-CDS interaction

Change-Id: I586153244dbd97a41e9b9d616ee9a84327b7c2da
Issue-ID: POLICY-1656
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

use policy/parent awaitility

use common version

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

Merge "FIxing Sonar issues in APEX"

Merge "Switch apex to Graal javascript engine"

FIxing Sonar issues in APEX

Change-Id: I8ac07eede6758c84a4a9ca505382f215d531cf47
Issue-ID: POLICY-1913
Signed-off-by: aditya.puthuparambil <aditya.puthuparambil@est.tech>

Point to docker 2.0.1

policy/docker is released.

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

Switch apex to Graal javascript engine

This is the main review to switch out the Nashorn Javascript interpreter
and switch in the Graal javascript interpreter for apex Javascript
tasks.

The Graal Javascript dialect differs slightly from Nashorn in the way it
accesses information passed over from the Java world so Javascript
tasks need to be converted.

Subsequent reviews will convert the JUnit tests marked @Ignore in
previous reviews to this one and will convert the Javascript tasks into
a format Graal can handle. Subsequent reviews will also convert the
examples over to Graal format.

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

Merge "Add integration tests for Parameter passing in APEX"

Add integration tests for Parameter passing in APEX

Change-Id: I5d01766ffb0104f632bebc9b553e7dca83b4829f
Issue-ID: POLICY-2364
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Merge "Fix Java 11/Checkstyle/Sonar warnings"

Fix Java 11/Checkstyle/Sonar warnings

A number of Java 11, checkstyle, and SONAR warnings have crept into the
Apex codebase over the last number of reviews. This change fixes those
issues.

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

remove duplication code in plugins-event producer

Issue-ID: POLICY-1884
Change-Id: I7774dfdc7d7edcc0409a817bfd21ddacec21d085
Signed-off-by: ning.xi <ning.xi@est.tech>

Merge "remove code duplication in plugins"

Merge "remove code duplication in plugin-event-carrier"

remove code duplication in plugins

Remove consumer code duplication in plugins-event-carrier

Issue-ID: POLICY-1884
Signed-off-by: shaoqiu <tim.huang@est.tech>
Change-Id: I2d222436a97224e54a03c2501f4dc14d3f5f6ac0

Passing taskParameters from ApexConfig to policy logic

TaskParameters can be used to pass parameters from ApexConfig to the policy logic. In the config, these are optional.
Usage as below:

{
  "engineParameters": {
    "taskParameters": [
      {
        "key": "ParameterKey1",
        "value": "ParameterValue1"
      },
      {
        "taskId": "TaskIdVal",
        "key": "ParameterKey2",
        "value": "ParameterValue2"
      }
    ]
  }
}

In the taskLogic, taskParameters can be accessed as below:
eg:  executor.parameters.get("ParameterKey1"))

If taskId is provided in ApexConfig for an entry, then that
parameter is updated only for that particular task. Otherwise, the task
parameter is added to all tasks.

Change-Id: I9e1b3d3697428309e7d86db40b63ffe822935b69
Issue-ID: POLICY-2364
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

remove code duplication in plugin-event-carrier

Issue-ID: POLICY-1884
Change-Id: I610d7ee34d640cc3a7d3381b24993ee9b6f76f2d
Signed-off-by: ning.xi <ning.xi@est.tech>

Updating module names to resolve conflicts

Issue-ID: POLICY-2355
Change-Id: I67f0ca2ced9cbe37a3d00d9f312101e756454107
Signed-off-by: Ram Krishna Verma <ram_krishna.verma@bell.ca>

Adding Tosca compliant operational policy type to default config file for apex-pdp

Change-Id: I6de1fd31c2e73814a546b05ee71419fded4a3357
Issue-ID: POLICY-2147
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Remove content section from ToscaPolicy properties in APEX

Currently APEX specific information is placed under properties|content in ToscaPolicy.
Avoid keeping under "content" and keep the information directly under properties.

Change-Id: Ic437271c9a2d71104013b5568af5525df4a4bb56
Issue-ID: POLICY-2332
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Remove unnecessary sonar config

This is no longer needed now that it is pointing to
latest policy/parent.

BTW - I decided there were too many changes required
for combining into one single .exec jacoco file. And it
was not improving any of the numbers. This repo is very
close to original numbers before the move to sonar cloud.

Will investigate another time.

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

Remove apex-pdp TextFileUtils class

This class was moved to policy-common some time ago. Removing it in
apex-pdp.

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

Run apex-pdp in Java 11: base changes

This change brings in the minimum changes to get apex-pdp running in
Java 11. Other reviews will bring in changes to remove Java 11 warnings
and to convert apex-pdp Javascript handling from the now deprecated
nashorn engine.

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

First sonar update

Just updating this way gets apex to around 79%, which
passes. Previously around 80%, when JDK11 is done I will
fix it up and hopefully we will get it back to over 80%.

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

Point to policy/models release version

Issue-ID: POLICY-1592
Change-Id: I4a795fc1496f5608884a7b8bdad0c212c4321392
Signed-off-by: HOCKLA <ah999m@att.com>

Merge "Populate the PdpStatistics data in heartbeat"

Merge "Enabling logback file to be loaded using oom configmap"

Merge "Update vCPE example to use Avro schemas"

Enabling logback file to be loaded using oom configmap

Change-Id: I746ee57003175ae9e3f903ee3e5948ed1875dceb
Issue-ID: POLICY-2308
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Update vCPE example to use Avro schemas

THis change updates the vCPE example to show how the example can work
towards the APPC using Avro schemas only. This means that Policies can
be adapted to changes in controller APIs using Avro alone.

This example works with the old and new format policy-models.

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

Populate the PdpStatistics data in heartbeat

Apex to populate the PdpStatistics data in every heartbeat sent to PAP

Issue-ID: POLICY-2302
Change-Id: I2d3a6c1e8605d53bf754a135980527bd50762abd
Signed-off-by: shaoqiu <tim.huang@est.tech>

Changing ownership of configuration files in APEX

Default user when logging into APEX container is apexuser. Updating the
ownership of configuration files such as logback.xml and key files so
that these can be updated at runtime.

Change-Id: Icf21e0b408d3b4ef829e1b1c5505f142a7d08adc
Issue-ID: POLICY-2308
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Merge "Adding native policy type to the config file in APEX"

Merge "Point to release versions of policy/parent and policy/common"

Point to release versions of policy/parent and policy/common

Issue-ID: POLICY-1583
Change-Id: I5dd6f77991bd5e8419176ea4cba751f58c05d501
Signed-off-by: HOCKLA <ah999m@att.com>