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>
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>
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>
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>
Przemyslaw Wasala [Tue, 18 Sep 2018 06:53:20 +0000 (06:53 +0000)]
Merge "Align with latest HV-VES proto definition"
Lusheng Ji [Mon, 17 Sep 2018 16:21:46 +0000 (16:21 +0000)]
Merge "update root license"
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>
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>
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
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>
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>
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>
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>
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>
Przemyslaw Wasala [Mon, 10 Sep 2018 09:19:46 +0000 (09:19 +0000)]
Merge "Enable env parameters read"
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>
Vijay Venkatesh Kumar [Fri, 7 Sep 2018 16:13:53 +0000 (16:13 +0000)]
Merge "Use managed guava version"
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>
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>
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>
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>
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>
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>
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>
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>
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>
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>
Przemyslaw Wasala [Tue, 21 Aug 2018 10:12:01 +0000 (10:12 +0000)]
Merge "Harmonize docker image names"
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>
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>
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>
Lusheng Ji [Fri, 17 Aug 2018 18:09:49 +0000 (18:09 +0000)]
Merge "Remove Ratpack dependency for HV-VES health checks"
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>
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>
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>
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>
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>
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>
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>
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>
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>
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>
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>
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>
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
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
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
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
Przemyslaw Wasala [Fri, 3 Aug 2018 08:59:40 +0000 (08:59 +0000)]
Merge "Move MessageParametersParser"
Przemyslaw Wasala [Fri, 3 Aug 2018 08:58:49 +0000 (08:58 +0000)]
Merge "Fix NPE when getting Consul configuration"
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
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
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
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
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
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
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
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
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
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
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
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
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
Przemyslaw Wasala [Fri, 3 Aug 2018 05:17:40 +0000 (05:17 +0000)]
Merge "Include msg generator module in coverage report"
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
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
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
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
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
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
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
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
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
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
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
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
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
Przemyslaw Wasala [Thu, 2 Aug 2018 10:00:42 +0000 (10:00 +0000)]
Merge "Enhancement of routing test"
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
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
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
Przemyslaw Wasala [Thu, 2 Aug 2018 09:29:12 +0000 (09:29 +0000)]
Merge "Reject messages with payload size > 1MiB"
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
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
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
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
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
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
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
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
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
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
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
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
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
Przemyslaw Wasala [Thu, 2 Aug 2018 06:26:50 +0000 (06:26 +0000)]
Merge "Dockerize DCAE APP simulator"
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
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
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
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
Code Review / dcaegen2 / collectors / hv-ves.git / log