dcaegen2/collectors/hv-ves.git
5 years agoEnhance releasing memory 49/68049/5
Filip Krzywka [Thu, 20 Sep 2018 10:17:46 +0000 (12:17 +0200)]
Enhance releasing memory

- Some buffers may be emitted as cancelled and thus they would not be
handled by doOnTerminate method
- Moved data stream creation for Netty inbound to time when
collector is fully functional

Change-Id: If2f2195fadeca957679f6be696802f48a616f48d
Issue-ID: DCAEGEN2-815
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoRevert reactor upgrade (fix for tcnative bug) 39/68039/1
Piotr Jaszczyk [Thu, 20 Sep 2018 10:05:14 +0000 (12:05 +0200)]
Revert reactor upgrade (fix for tcnative bug)

For now revert the update. Will provide another fix later.

Change-Id: Iaf5fdd64c6e06e63ddc93d759025ab989b0d63db
Issue-ID: DCAEGEN2-801
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoUpdate Kotlin and Reactor versions 87/67687/2
Piotr Jaszczyk [Wed, 19 Sep 2018 11:07:13 +0000 (13:07 +0200)]
Update Kotlin and Reactor versions

Change-Id: I7237cede772e79a1cd18835c85b43bcd81710a50
Issue-ID: DCAEGEN2-801
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoAlign to logging requirements 15/67615/3
Filip Krzywka [Wed, 19 Sep 2018 06:53:42 +0000 (08:53 +0200)]
Align to logging requirements

- Pattern does not contain MDCs and Markers as HV-VES will not
provide that kind of trace context in Casablanca release.
- Pattern does not contain replace(...){} parts from most recent version
of guideline as these parts significantly decrease logs readability
in current format

Change-Id: I03a9cef8b7f92208244a65c4846f280d560e037a
Issue-ID: DCAEGEN2-799
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoMerge "Align with latest HV-VES proto definition"
Przemyslaw Wasala [Tue, 18 Sep 2018 06:53:20 +0000 (06:53 +0000)]
Merge "Align with latest HV-VES proto definition"

5 years agoMerge "update root license"
Lusheng Ji [Mon, 17 Sep 2018 16:21:46 +0000 (16:21 +0000)]
Merge "update root license"

5 years agoAlign with latest HV-VES proto definition 75/66575/6
Filip Krzywka [Tue, 11 Sep 2018 12:45:53 +0000 (14:45 +0200)]
Align with latest HV-VES proto definition

- excluded measurements proto files from build to keep them
reference-only

Change-Id: I8c6de20eeeb1b9f8cd2ae4e865de368afe55cc91
Issue-ID: DCAEGEN2-775
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoRead flags from env variables 73/66973/1
Filip Krzywka [Mon, 17 Sep 2018 11:22:41 +0000 (13:22 +0200)]
Read flags from env variables

The application configuration should be read from command line
options as well as from environment variables. The priority: cmd-line over env over defaults.

Change-Id: Ib62d5d68003a006901d9be8a81fca354def8fea4
Issue-ID: DCAEGEN2-748
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoupdate root license 79/66679/1
vagrant [Fri, 14 Sep 2018 14:46:47 +0000 (14:46 +0000)]
update root license

Change-Id: If7db842404cc345057414d478b58874dd8b6853b
Signed-off-by: Vijay Venkatesh Kumar <vv770d@att.com>
Issue-ID: DCAEGEN2-595

5 years agoAlign with configuration present in consul 93/66293/1
Jakub Dudycz [Thu, 13 Sep 2018 07:55:02 +0000 (09:55 +0200)]
Align with configuration present in consul

Rename few configuration json keys inside collector code to match those on consul.

Change-Id: I400aebce98049118a30d7959dc6655a4da6016bf
Issue-ID: DCAEGEN2-774
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
5 years agoRemove security vulnerability 09/65709/1
Filip Krzywka [Tue, 11 Sep 2018 08:29:31 +0000 (10:29 +0200)]
Remove security vulnerability

Change-Id: If706e3eefb41f7c4a55212c70518f61069efeb40
Issue-ID: DCAEGEN2-766
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoEnable ssl-disable option for xNF simulator 39/65539/3
Filip Krzywka [Mon, 10 Sep 2018 09:35:13 +0000 (11:35 +0200)]
Enable ssl-disable option for xNF simulator

Change-Id: I8d4459c4b612e6066ae0da9521333fc3083952d3
Issue-ID: DCAEGEN2-760
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoFix common event header parsing 85/64985/8
Filip Krzywka [Thu, 6 Sep 2018 11:37:27 +0000 (13:37 +0200)]
Fix common event header parsing

During JSON parsing we were using 3 times 'version' field. Replaced
parsing with protobuf parser to avoid mistakes like this in future

Change-Id: I6224dc6533ab553e7e2315a95567a1fa48c1c5ad
Issue-ID: DCAEGEN2-710
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoIncrease of UT coverage in project 47/65547/2
kjaniak [Mon, 10 Sep 2018 13:06:34 +0000 (15:06 +0200)]
Increase of UT coverage in project

Change-Id: I8ac9ce21b5034f795da19e27e42ba95ce87db10d
Issue-ID: DCAEGEN2-741
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
5 years agoMerge "Enable env parameters read"
Przemyslaw Wasala [Mon, 10 Sep 2018 09:19:46 +0000 (09:19 +0000)]
Merge "Enable env parameters read"

5 years agoEnable env parameters read 45/64945/5
kjaniak [Thu, 6 Sep 2018 08:07:17 +0000 (10:07 +0200)]
Enable env parameters read

Disabling require option on mandatory parameters
to fetch them from env variables.

Change-Id: I007dea1a7f369a04479801aa508cf1034ac1341a
Issue-ID: DCAEGEN2-741
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
5 years agoMerge "Use managed guava version"
Vijay Venkatesh Kumar [Fri, 7 Sep 2018 16:13:53 +0000 (16:13 +0000)]
Merge "Use managed guava version"

5 years agoHandle non-existing Collector instance 59/64659/3
Filip Krzywka [Wed, 5 Sep 2018 08:37:51 +0000 (10:37 +0200)]
Handle non-existing Collector instance

Change-Id: I0b6cd5023b2bca0f0bee6958c107fc560fc95b52
Issue-ID: DCAEGEN2-751
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoBuild enhancements 17/64917/4
Piotr Jaszczyk [Thu, 6 Sep 2018 06:20:03 +0000 (08:20 +0200)]
Build enhancements

* Handling docker proxy configuration in a reasonable way
* Should skip coverage verification when skipTests was set
* Get rid of resources encoding warning
* Fix check-coverage output on Jenkins

Change-Id: If828fc41a88074931649e739c615a8b9d8db8739
Issue-ID: DCAEGEN2-755
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoImprove coverage of xNF simulator 53/63453/13
Piotr Jaszczyk [Wed, 29 Aug 2018 11:24:59 +0000 (13:24 +0200)]
Improve coverage of xNF simulator

Also refactor to make it possible.

Change-Id: I6da6d3f33e57c524a7e353ecebd3e045d8ceed2a
Issue-ID: DCAEGEN2-739
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoRead config from environment variables 51/64351/4
Piotr Jaszczyk [Tue, 4 Sep 2018 07:59:20 +0000 (09:59 +0200)]
Read config from environment variables

The application configuration should be read from command line options
as well as from environment variables. The priority: cmd-line over env
over defaults.

Change-Id: I785fd1fbaf66f3eab84a162f037153f1688ed791
Issue-ID: DCAEGEN2-748
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoImprove DCAE APP Simulator coverage 01/62301/13
Piotr Jaszczyk [Fri, 24 Aug 2018 10:51:14 +0000 (12:51 +0200)]
Improve DCAE APP Simulator coverage

Also there was a need to refactor the code, because application logic
was placed inside Ratpack handlers.

Change-Id: Iba3d4d039a98ba88e0dba580c1b7726b53440538
Issue-ID: DCAEGEN2-732
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoRemove additional SNAPSHOT from docker tag 87/63687/1
Piotr Jaszczyk [Thu, 30 Aug 2018 06:42:21 +0000 (08:42 +0200)]
Remove additional SNAPSHOT from docker tag

Change-Id: I3fa764feef63d1318adf7ba8128ea2b86055951a
Issue-ID: DCAEGEN2-742
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoUse managed guava version 73/62573/2
Gary Wu [Fri, 24 Aug 2018 22:20:36 +0000 (15:20 -0700)]
Use managed guava version

Use centrally managed guava version specified in
oparent.  Includes upgrade to oparent 1.2.0.

This change was submitted by script and may include
additional whitespace or formatting changes.

Change-Id: I0f54a8a9fb9ce4536c621c81f9c3be54d121a4e1
Issue-ID: INT-619
Signed-off-by: Gary Wu <gary.i.wu@huawei.com>
5 years agoFail build when coverage is too low 89/62089/4
Piotr Jaszczyk [Thu, 23 Aug 2018 12:45:39 +0000 (14:45 +0200)]
Fail build when coverage is too low

Minimum coverage set to 60%. It works by parsing aggregated Jacoco
report as oposed to jacoco:check goal which checks coverage at most
on submodule level.

Change-Id: Ie6f50ce9b2f15e62ad84480611897a98321a7af2
Issue-ID: DCAEGEN2-681
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoUpdate to Jacoco 0.8.2 & fix coverage report 93/61993/3
Piotr Jaszczyk [Thu, 23 Aug 2018 07:06:19 +0000 (09:06 +0200)]
Update to Jacoco 0.8.2 & fix coverage report

* Upgrade to Jacoco version with Kotlin support
* Reorganize hv-collector-coverage pom.xml so aggregated report works
  again

Change-Id: I636d901114701e2c6669d45add85aafcad8a6783
Issue-ID: DCAEGEN2-729
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoAlign with latest proposal of wire proto frame 89/61589/2
Piotr Jaszczyk [Tue, 21 Aug 2018 12:46:40 +0000 (14:46 +0200)]
Align with latest proposal of wire proto frame

Change-Id: I8f989a3913f1592b4d740a80ed30b01bc3aceff2
Issue-ID: DCAEGEN2-722
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoMerge "Harmonize docker image names"
Przemyslaw Wasala [Tue, 21 Aug 2018 10:12:01 +0000 (10:12 +0000)]
Merge "Harmonize docker image names"

5 years agoHarmonize docker image names 95/61495/1
Piotr Jaszczyk [Tue, 21 Aug 2018 07:44:46 +0000 (09:44 +0200)]
Harmonize docker image names

Use groupId.artifactId naming convention.

Change-Id: Iddcf59c895305677db10534696e4b026ca97e042
Issue-ID: DCAEGEN2-719
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoUpdate dependenies' versions 81/61481/3
Piotr Jaszczyk [Tue, 21 Aug 2018 06:49:39 +0000 (08:49 +0200)]
Update dependenies' versions

Kotlin and Arrow updated. Also used Jacoco in version 0.8.1 because one
defined in oparent fails with new Kotlin.

Change-Id: I42e70c08454205fc732cca092c3ec0637f6118d3
Issue-ID: DCAEGEN2-720
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoRename artifacts to match LF requirements 69/61469/1
Piotr Jaszczyk [Tue, 21 Aug 2018 06:28:54 +0000 (08:28 +0200)]
Rename artifacts to match LF requirements

Group ID has been changed from org.onap.dcaegen2.collectors.veshv to
org.onap.dcaegen2.collectors.hv-ves.

Change-Id: I077e244ed56df22dd45bfe8dd1eee238f811a552
Issue-ID: DCAEGEN2-719
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoMerge "Remove Ratpack dependency for HV-VES health checks"
Lusheng Ji [Fri, 17 Aug 2018 18:09:49 +0000 (18:09 +0000)]
Merge "Remove Ratpack dependency for HV-VES health checks"

5 years agoRemove Ratpack dependency for HV-VES health checks 27/60527/10
Piotr Jaszczyk [Tue, 14 Aug 2018 10:52:28 +0000 (12:52 +0200)]
Remove Ratpack dependency for HV-VES health checks

In order to minimize complexity and possibly improve performance (thread count)
reactor-netty should be used instead of Ratpack. Also reorganize code to
be more consistent and differentiated readiness and liveness endpoints
(for future use in K8s Pod definition).

As an example I've defined health check probe in docker-compose YAML.

Change-Id: I1b5ce3d685e7ae5b0515b2146ae4fa88b3b41186
Issue-ID: DCAEGEN2-705
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoAlign with release docker & JARs CI definitions 49/60849/4
Piotr Jaszczyk [Thu, 16 Aug 2018 06:09:27 +0000 (08:09 +0200)]
Align with release docker & JARs CI definitions

If not explocitly called with -DskipDockerPush=false Maven will not push
docker images to the registry. The change is introduced in order for
'{project-name}-{stream}-docker-java-shell-daily' to do the push but all
other Jenkins jobs should skip this step.

Also set hv-collector-analysis version to 1.0.0-SNAPSHOT as it's not a
child of root pom (it contains only custom definitions needed for detekt
plugin to work).

Change-Id: Icc4a1d6cc1a763fad417860b45ffdad30524f67b
Issue-ID: DCAEGEN2-661
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoReorganize docker build 23/58923/8
Piotr Jaszczyk [Fri, 3 Aug 2018 11:05:15 +0000 (13:05 +0200)]
Reorganize docker build

* Docker is built in "docker" profile
* It is turned on by default
* "mvn install" builds and tags docker image
* "mvn deploy" pushes the image to the registry
* "mvn -DskipDocker ..." skips docker build
* Temporarily use OpenJDK 8 instead of 10 - it require us to
  specify additional JVM options for use inside containers on production
  hence addition of run-java.sh

Change-Id: Ifa67df4ed7de9f3d300d6f6de999a890486dea24
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-653
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoLink project as ONAP oparent module child 05/60005/4
Filip Krzywka [Fri, 10 Aug 2018 06:26:42 +0000 (08:26 +0200)]
Link project as ONAP oparent module child

- Downgraded checkstyle-plugin version to not break build - oparent
uses deprecated config property that cannot be overriden... Also it
uses plugin version from 2014...
- Moved jacoco plugin definition for similiar reasons.
- Bumped maven-surefire-plugin.
- Also added missing license in file. Not sure why CI did not catch
that one earlier
- removed nexus properties that should be inherited from onap
`version` artifact (parent of oparent)
- Use upgraded maven-failsafe-plugin - works with java 8, not sure
about Java 9+

Change-Id: I47e441315cc0e1dc49ed474450db0ee3d2701a10
Issue-ID: DCAEGEN2-700
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
5 years agoAdd netcat to docker image 47/59847/1
Filip Krzywka [Thu, 9 Aug 2018 11:39:21 +0000 (13:39 +0200)]
Add netcat to docker image

As it's not included in `slim` version of jre image and will be
required for healthcheck

Change-Id: Ic2e73e58c46d72fc845266a6953bd0dc4d8bc8b9
Issue-ID: DCAEGEN2-697
Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
5 years agoImplement simple health check mechanism 03/59603/6
Jakub Dudycz [Wed, 8 Aug 2018 07:17:14 +0000 (09:17 +0200)]
Implement simple health check mechanism

Change-Id: Ic4b8b59ced9dc19c9ebf26131036a9e1a752164f
Issue-ID: DCAEGEN2-659
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
5 years agoCreate health check module 01/59601/4
Jakub Dudycz [Tue, 7 Aug 2018 12:18:37 +0000 (14:18 +0200)]
Create health check module

Create ves-hv-collector-health-check module with dummy api server and connect it with ves-hv-collector-main
This is a preparation for health check mechanism implementation

Change-Id: I2f668ab7337b1ed7e2afea6c56f34880de3ef1b5
Issue-ID: DCAEGEN2-659
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
5 years agoClean up repository 37/59337/1
Jakub Dudycz [Tue, 7 Aug 2018 07:32:43 +0000 (09:32 +0200)]
Clean up repository

There are some leftover from gitlab that should be removed from/updated in official repo.

Change-Id: I9e47321433ed2f744135b7fb85f140f381fe84e1
Issue-ID: DCAEGEN2-690
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
5 years agoCorrection of version.properties typo 27/59327/1
kjaniak [Tue, 7 Aug 2018 05:46:11 +0000 (07:46 +0200)]
Correction of version.properties typo

Change-Id: Ib321f24ffc54b6dca321cc3a5929c7bdaebc790a
Issue-ID: DCAEGEN2-683
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
5 years agoAdd INFO.yaml file 67/57767/3
Jessica Wagantall [Thu, 26 Jul 2018 21:08:39 +0000 (14:08 -0700)]
Add INFO.yaml file

Add INFO.yaml to list:
- Project description
- Properties
- PTL information
- Meeting information
- Committer information

Change-Id: I1180a4d0840ef7b5f6f5f684ed2bf6d531e565bf
Issue-ID: CIMAN-134
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
5 years agoAdd of version.properties file 07/59207/1
kjaniak [Mon, 6 Aug 2018 12:55:16 +0000 (14:55 +0200)]
Add of version.properties file

Change-Id: Id6e282d8865a425bc1a093d6a495cd4fabe3d98f
Issue-ID: DCAEGEN2-683
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
5 years agoUpgrade to Kotlin 1.2.60 85/59185/1
Jakub Dudycz [Mon, 6 Aug 2018 11:56:40 +0000 (13:56 +0200)]
Upgrade to Kotlin 1.2.60

Change-Id: I089c63edd1859c6941f0791f1cf3a09526b6d028
Issue-ID: DCAEGEN2-660
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
5 years agoDisable proxy when building Docker images 11/58911/1
Piotr Jaszczyk [Fri, 3 Aug 2018 10:07:28 +0000 (12:07 +0200)]
Disable proxy when building Docker images

Change-Id: I72d5c511a90fd2d357227aa94b968cc63bd4ca68
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-653

5 years agoRe-enable docker build 07/58907/1
Piotr Jaszczyk [Fri, 3 Aug 2018 09:50:23 +0000 (11:50 +0200)]
Re-enable docker build

* re-enable after seed code temporary workaround
* move docker build from package to deploy phase

Change-Id: Ic338247590b019c90cf9241075a255bfaff28c5b
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-653

5 years agoRemove default consul configuration 97/58897/1
Jakub Dudycz [Tue, 31 Jul 2018 12:49:35 +0000 (14:49 +0200)]
Remove default consul configuration

Default consul configuration is not needed in real life use case
and it doesnt even bring anything useful in our test environment,
since it's immediately overriden.

Tasks:
- remove default consul configuration
- make "config-url" argument required

Closes ONAP-727

Change-Id: I4a01cebfc6b2918fa3aa6020504def60572b09da
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoExtract test-utils module 83/58883/1
fkrzywka [Tue, 31 Jul 2018 12:26:09 +0000 (14:26 +0200)]
Extract test-utils module

- removed duplicate code that was creating VesMessages
and similiar objects
- removed duplicate code in command line parsing tests
- made minor refactorings to avoid passing unnecessary params
and to be as verbose as possible in tests

Closes ONAP-699

Change-Id: I2607f1f775054ae1c5f275c231895f838b415371
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoMerge "Move MessageParametersParser"
Przemyslaw Wasala [Fri, 3 Aug 2018 08:59:40 +0000 (08:59 +0000)]
Merge "Move MessageParametersParser"

5 years agoMerge "Fix NPE when getting Consul configuration"
Przemyslaw Wasala [Fri, 3 Aug 2018 08:58:49 +0000 (08:58 +0000)]
Merge "Fix NPE when getting Consul configuration"

5 years agoMove MessageParametersParser 79/58879/1
fkrzywka [Tue, 31 Jul 2018 12:22:59 +0000 (14:22 +0200)]
Move MessageParametersParser

To avoid dependency of utilities module on modules other than domain

Change-Id: I90ef8640a86501315d84848118d3e748aafd095c
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoFix NPE when getting Consul configuration 77/58877/1
Piotr Jaszczyk [Tue, 31 Jul 2018 07:28:29 +0000 (09:28 +0200)]
Fix NPE when getting Consul configuration

No initial value for AtomicReference was provided hence we had a little
race condition.

Retry when consul returns error.

Change-Id: Ie38ca7fbf445123e98ee94703eba501bb5233fab
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoUse proxy while publishing docker images on CI 75/58875/2
fkrzywka [Tue, 31 Jul 2018 07:31:05 +0000 (09:31 +0200)]
Use proxy while publishing docker images on CI

Also fix gitlab-ci syntax

Change-Id: I5e5e1db3888f976640bacc161d703cc6be3ab613
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoDefine simple healthchecks for simulators 71/58871/1
fkrzywka [Mon, 30 Jul 2018 10:55:03 +0000 (12:55 +0200)]
Define simple healthchecks for simulators

- currently containers are healthy if they have successfully started
their Http servers thus simple endpoint at /healthcheck is sufficient
- to enable docker healthcheck we need to add `curl` to docker images
- http_proxy needs to be provided for docker to be able to build images
behind proxy

Closes ONAP-695

Change-Id: I02f0b8cc2732713b5307dbc34ba604d8185b74fc
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoDCAE APP simulator rework 69/58869/1
Jakub Dudycz [Thu, 26 Jul 2018 09:49:45 +0000 (11:49 +0200)]
DCAE APP simulator rework

- Extract message parameters parsing logic to standalone class in utils
- Make DCAE APP simulator store whole received messages history
- Add validation endpoint
- Add new messege type: FIXED_PAYLOAD

Closes ONAP-686

Change-Id: I865804716ad5e46a7503a8eee70cfe9ac75a6c1e
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoPass CommonEventHeader to XNF simulator api 63/58863/1
Jakub Dudycz [Wed, 25 Jul 2018 14:13:28 +0000 (16:13 +0200)]
Pass CommonEventHeader to XNF simulator api

This change makes XNF simulator more configurable and
allows to validate more message parameters in robot integration tests

Closes ONAP-689

Change-Id: Ic0a10f1e1cdd84ac415c00050b4cca1ac496c56b
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoEnable UNDEFINED option in commonHeader GPB schema 61/58861/1
kjaniak [Wed, 25 Jul 2018 12:02:27 +0000 (14:02 +0200)]
Enable UNDEFINED option in commonHeader GPB schema

Closes ONAP-655

Change-Id: I6fc830d2b8c7bdd726f306a6fbf7e979f39e03f0
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoRework argument configuration 59/58859/1
Jakub Dudycz [Tue, 24 Jul 2018 09:48:12 +0000 (11:48 +0200)]
Rework argument configuration

- Unify names of argument configuration classes in
  DCAE APP simulator, XNF simualtor and VES HV Collector
- Make some of the arguments required
- Adjust docker-compose and Dockerfiles
- Adjust test cases and error handling

Closes ONAP-683

Change-Id: I4a9d43791cced9dcb52eb83e2f7956462e8712d9
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoConfigure xnf simulator api listen port 49/58849/1
Jakub Dudycz [Mon, 23 Jul 2018 12:51:00 +0000 (14:51 +0200)]
Configure xnf simulator api listen port

Closes ONAP-664
Closes ONAP-647

Change-Id: I8d1e57c0c66f1968925660fe47fa96a2a634c039
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoFix consul request timeout issue 45/58845/2
Jakub Dudycz [Fri, 20 Jul 2018 14:37:02 +0000 (16:37 +0200)]
Fix consul request timeout issue

Fix timeout issue when using consul blocking query calls
by switching to standard requests peformed in given interval

Closes ONAP-628

Change-Id: Ifaf7ddfa27045015a7a90c178e0d6d38955c0c58
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoSupport scenarios for continuous streaming test 43/58843/1
Jakub Dudycz [Wed, 18 Jul 2018 12:33:10 +0000 (14:33 +0200)]
Support scenarios for continuous streaming test

Added support for below scenarios
-too big payloads
-invalid wire frames
-invalid GPB data
-unsupported domains

Changed input json for xnf simulator endpoint

Closes ONAP-500

Change-Id: I19e84a76cef501e274ea8152f3c33c95dddcaac9
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoAdd command line option to disable SSL/TLS 41/58841/2
kjaniak [Tue, 17 Jul 2018 09:50:10 +0000 (11:50 +0200)]
Add command line option to disable SSL/TLS

Closes ONAP-508

Change-Id: If6c3935ede7b00dea9b36747c6cd1422c1c8d330
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoUse Try/Option monad when decoding protobuf 39/58839/1
fkrzywka [Tue, 17 Jul 2018 05:54:58 +0000 (07:54 +0200)]
Use Try/Option monad when decoding protobuf

Closes ONAP-143

Change-Id: I33cb2d24cd5962318a6f405096db298bbdbab963
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoMerge "Include msg generator module in coverage report"
Przemyslaw Wasala [Fri, 3 Aug 2018 05:17:40 +0000 (05:17 +0000)]
Merge "Include msg generator module in coverage report"

5 years agoInclude msg generator module in coverage report 37/58837/1
fkrzywka [Mon, 16 Jul 2018 06:17:29 +0000 (08:17 +0200)]
Include msg generator module in coverage report

* Reordered alphabetically modules in main pom file
* Removed unused imports
* Corrected visibility for fields/methods
* Fix typos in tests

Change-Id: I35c515b3844bc8517cc6ffb0c6557596505536c9
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoRemove unused method 35/58835/1
fkrzywka [Fri, 13 Jul 2018 11:10:21 +0000 (13:10 +0200)]
Remove unused method

Change-Id: I462e78f0bad9aba2e82d2766d748608b6231c839
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoAdd API endpoint for DCAE App Sim reconfiguration 37/58737/1
fkrzywka [Thu, 12 Jul 2018 12:46:58 +0000 (14:46 +0200)]
Add API endpoint for DCAE App Sim reconfiguration

Closes ONAP-547

Change-Id: I26b7d5d61eb984d58600a612cd8d3e4dd7be05c5
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoExtract message generator module 25/58725/1
kjaniak [Thu, 12 Jul 2018 08:57:58 +0000 (10:57 +0200)]
Extract message generator module

Closes ONAP-501

Change-Id: Ia5dc5f67e18c302abbeac7e0373cef479f4befb8
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoUse MessageValidator in VesMessage class 23/58723/1
Jakub Dudycz [Wed, 11 Jul 2018 10:24:15 +0000 (12:24 +0200)]
Use MessageValidator in VesMessage class

Closes ONAP-493
Change-Id: I15b5f0f1052865aa29ffa103bef2368bd94021e8
Signed-off-by: Jakub Dudycz <jdudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoUse Flux.transform in VesHvCollector 09/58709/1
Jakub Dudycz [Tue, 10 Jul 2018 10:29:32 +0000 (12:29 +0200)]
Use Flux.transform in VesHvCollector

Goal: split the stream into logical parts

Closes ONAP-493
Change-Id: I87aa817a18674fad265df81b6a0b4a8f0c46b866
Signed-off-by: Jakub Dudycz <jdudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoAdd byte-buffer handling assertions when decoding 93/58693/1
fkrzywka [Wed, 4 Jul 2018 12:46:57 +0000 (14:46 +0200)]
Add byte-buffer handling assertions when decoding

ByteBuffer reader index should be reset in case of failed decoding
of wire frame, as parsing might be retried when more bytes arrive

Change-Id: I6dcb5c94c8cffba969f572fca8bbb728ea9500bd
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoEnhance wire protocol 89/58689/1
fkrzywka [Tue, 3 Jul 2018 08:14:38 +0000 (10:14 +0200)]
Enhance wire protocol

Handle new wire frame message type which should allow clients to
indicate that all data has been sent to collector

Change xNF Simulator to send end-of-transmission message
after sending all messages

Close ves-hv-collector stream after encountering EOT message

Remove duplicated file in project

Closes ONAP-391

Change-Id: Idb6afc41d4bb0220a29df10c2aecfd76acd3ad16
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoRename hv-collector-client-simulator 77/58677/1
Jakub Dudycz [Mon, 9 Jul 2018 06:06:01 +0000 (08:06 +0200)]
Rename hv-collector-client-simulator

New name: hv-collector-xnf-simulator

Closes ONAP-492

Change-Id: I161dda0d01e90bfe35e5e2d575ec9e81cc8bf3c3
Signed-off-by: Jakub Dudycz <jdudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoFix consul configuration delay 67/58667/1
Jakub Dudycz [Mon, 9 Jul 2018 11:09:56 +0000 (13:09 +0200)]
Fix consul configuration delay

Delay configuration received from consul instead of default one

Change-Id: I09ec1a1e0d4a8334f83c054e0891b55ff8d049db
Issue-Id: ONAP-519
Signed-off-by: jdudycz@nokia.com
Issue-ID: DCAEGEN2-601

5 years agoImplement blocking consul calls 63/58663/1
Jakub Dudycz [Thu, 5 Jul 2018 12:35:43 +0000 (14:35 +0200)]
Implement blocking consul calls

Replaced interval based requesting for consul configuration with blocking query calls

Closes ONAP-80

Change-Id: If70365bae9fde513d99b047209d085122a5df0dd
Signed-off-by: Jakub Dudycz <jdudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoComponent tests for consul configuration updates 61/58661/1
Jakub Dudycz [Mon, 2 Jul 2018 13:52:08 +0000 (15:52 +0200)]
Component tests for consul configuration updates

Added few component test cases for updating configuration

Closes ONAP-464

Change-Id: Id8dba1d1cf4bf641a65e27d2a257fb5c26ee2bbc
Signed-off-by: Jakub Dudycz <jdudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoUse IO monad when starting servers 59/58659/1
Piotr Jaszczyk [Wed, 4 Jul 2018 11:16:21 +0000 (13:16 +0200)]
Use IO monad when starting servers

Change-Id: I3e97161535fc721dda6109c4cb5f23a1db0afde3
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoMerge "Enhancement of routing test"
Przemyslaw Wasala [Thu, 2 Aug 2018 10:00:42 +0000 (10:00 +0000)]
Merge "Enhancement of routing test"

5 years agoEnhancement of routing test 53/58653/1
kjaniak [Thu, 5 Jul 2018 06:13:59 +0000 (08:13 +0200)]
Enhancement of routing test

Change-Id: I432c09a219928d1a89a0c619db0e8cc3104af0ec
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoBump Kotlin and Reactor versions 51/58651/1
Piotr Jaszczyk [Thu, 5 Jul 2018 08:06:46 +0000 (10:06 +0200)]
Bump Kotlin and Reactor versions

Change-Id: I949e1f5ac07be03a1537f07a0b5281171e43c09b
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoInclude "new" modules in coverage report 49/58649/1
Piotr Jaszczyk [Fri, 29 Jun 2018 12:57:02 +0000 (14:57 +0200)]
Include "new" modules in coverage report

Change-Id: I8735b9a0c7b9e7d623e071098ee00d91d8a69fbf
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoMerge "Reject messages with payload size > 1MiB"
Przemyslaw Wasala [Thu, 2 Aug 2018 09:29:12 +0000 (09:29 +0000)]
Merge "Reject messages with payload size > 1MiB"

5 years agoReject messages with payload size > 1MiB 41/58641/1
Jakub Dudycz [Thu, 28 Jun 2018 10:41:29 +0000 (12:41 +0200)]
Reject messages with payload size > 1MiB

- Update validation in WireFrameDecoder class
- Write unit and component tests for that case

Closes ONAP-340

Change-Id: I68cb608fd76118719b12a83de1ef930160f8a162
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoAdd links to IT results to glpages 39/58639/1
Piotr Jaszczyk [Fri, 29 Jun 2018 10:45:19 +0000 (12:45 +0200)]
Add links to IT results to glpages

Change-Id: I2a2ccc5f826a2d91c75fd8ea81f92df92dcb04d3
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoUse Either instead of exceptions in frame decoder 33/58633/1
Piotr Jaszczyk [Thu, 28 Jun 2018 12:42:05 +0000 (14:42 +0200)]
Use Either instead of exceptions in frame decoder

Goals:
* Make code cleaner (in a FP way)
* Avoid costly exception throw each time we wait for the rest of the
frame (collecting stack traces is costly and we do not need them
anyway)

Closes ONAP-437
Change-Id: I40341d3c2cb85f3ff581d89167245cb009dbb070
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoUse generator to simplify the WireFrame decoding 27/58627/1
Piotr Jaszczyk [Thu, 28 Jun 2018 08:09:24 +0000 (10:09 +0200)]
Use generator to simplify the WireFrame decoding

Performance tests have proven that manual creation of the Flux doesn't
give us any performance benefits. On the other side it is complicated
and error prone.

Closes ONAP-438

Change-Id: I45912f91a52cbc84322775f7bae6d73afda079b8
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoWrite performance tests 23/58623/1
Piotr Jaszczyk [Wed, 27 Jun 2018 10:30:56 +0000 (12:30 +0200)]
Write performance tests

Closes ONAP-434
Change-Id: I1139848f32ac19a4d0a0fd595f4b07c10cd83db0
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoChange Consul configuration update policy 13/58613/1
Jakub Dudycz [Fri, 15 Jun 2018 14:09:41 +0000 (16:09 +0200)]
Change Consul configuration update policy

- At startup default config is applied
- Configuration is updated in intervals given at VES-HV
service startup to allow dynamic changes
- Included consul service startup in docker-compose file
- VES-HV now exits when fails to acquire confguration from consul

Closes ONAP-229

Change-Id: I896cfd177fa45381f9822278c2dffc113dd3df72
Signed-off-by: jakub.dudycz@nokia.com
Issue-ID: DCAEGEN2-601

5 years agoFixing docker push form fabric8io mvn plugin 11/58611/1
Piotr Jaszczyk [Mon, 25 Jun 2018 07:59:01 +0000 (09:59 +0200)]
Fixing docker push form fabric8io mvn plugin

Change-Id: I40ea95c162067f9d099e578fb236c5877562041b
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoInclude payload content type in wire frame 09/58609/1
Piotr Jaszczyk [Tue, 19 Jun 2018 08:27:53 +0000 (10:27 +0200)]
Include payload content type in wire frame

Closes ONAP-404
Change-Id: I6adfb542ffdafad678e7bc6e062d3d59c250b39e
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoVarious improvements 99/58599/2
Piotr Jaszczyk [Thu, 14 Jun 2018 07:48:46 +0000 (09:48 +0200)]
Various improvements

* Kotlin upgrade
* Monad usage on APIs
* Idle timeout
* Simulator enhancements

Closes ONAP-390

Change-Id: I3c00fcfe38c722caf661ddaad428cf089eeefcaa
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoVESEvent payload generation introduction 97/58597/1
kjaniak [Thu, 14 Jun 2018 11:10:53 +0000 (13:10 +0200)]
VESEvent payload generation introduction

Change-Id: Ida15b9739d26b5db2f69a5296876f20ea83de761
Signed-off-by: kjaniak <kornel.janiak@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoAdd monitoring support by means of micrometer.io 95/58595/2
Piotr Jaszczyk [Wed, 13 Jun 2018 13:45:00 +0000 (15:45 +0200)]
Add monitoring support by means of micrometer.io

Closes ONAP-345

Change-Id: I58c145b1d37a6b32fbe5b157723c152eb571a2dd
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoImplemented rest server in client simulator 87/58587/2
Jakub Dudycz [Mon, 11 Jun 2018 14:54:47 +0000 (16:54 +0200)]
Implemented rest server in client simulator

Change-Id: I212b79fe2a0272f340c5ca889beff60b469f7f24
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoStart DCAE App Simulator from command line 85/58585/1
fkrzywka [Wed, 13 Jun 2018 13:02:40 +0000 (15:02 +0200)]
Start DCAE App Simulator from command line

Closes ONAP-377

Change-Id: Iab959835dfafcfcfaf1322ead4ea83eff1e9284c
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoMerge "Dockerize DCAE APP simulator"
Przemyslaw Wasala [Thu, 2 Aug 2018 06:26:50 +0000 (06:26 +0000)]
Merge "Dockerize DCAE APP simulator"

5 years agoDockerize DCAE APP simulator 81/58581/1
Piotr Jaszczyk [Tue, 12 Jun 2018 12:19:10 +0000 (14:19 +0200)]
Dockerize DCAE APP simulator

Closes ONAP-265
Closes ONAP-267

Change-Id: I394476cf7ba3851d663a2995dc7fe591dae5be41
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoRefactor ArgBasedConfiguration 79/58579/1
fkrzywka [Wed, 13 Jun 2018 08:02:03 +0000 (10:02 +0200)]
Refactor ArgBasedConfiguration

Extracted duplicate code from Client/Server ArgBasedConfiguration, as it
probably would be used third time in future work

Change-Id: I581abbcd5f1dd4a1a049e1d28e68a7e4d82a84f8
Signed-off-by: fkrzywka <filip.krzywka@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoCheck license notes during build 73/58573/1
Piotr Jaszczyk [Wed, 13 Jun 2018 07:56:17 +0000 (09:56 +0200)]
Check license notes during build

I wanted to use oparent as parent pom but unfortunately jacoco plugin is
configured differently there. Currently oparent provides support for
jacoco (static code analysis), checkstyle (licenses + code style) and
swagger api documentation.

In ves-hv:
* jacoco is preconfigured for kotlin (but currently not very usable as
it's not ready for kotlin-generated code ie. displayed coverage is
well under the actual coverage; there is an ongoing effort in jacoco
and kotlin communities to fix that).
* checkstyle - this task adds support for checking licenses using
oparent:checkstyle artifact. Static code analysis is already done
using Detekt.
* swagger - ves-hv is not exposing REST API so there is no need for
this.

Closes ONAP-364
Change-Id: If2727a97d826dcab473b2e1e80171d8124ae52d4
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601

5 years agoBasic Ratpack API in DCAE APP Simulator 71/58571/1
Piotr Jaszczyk [Tue, 12 Jun 2018 10:16:19 +0000 (12:16 +0200)]
Basic Ratpack API in DCAE APP Simulator

Closes ONAP-266

Change-Id: Iaa000e976fcdc4274aa88ce7d0a6cd5866987680
Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
Issue-ID: DCAEGEN2-601