summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-09-24Remove end-of-transmission message from protocolFilip Krzywka22-455/+201
Also update protobuf files definitions to latest version. Change-Id: I0cd5d2d8deec5c787e2d3948d3d905fa672f9fea Issue-ID: DCAEGEN2-775 Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
2018-09-20Enhance releasing memoryFilip Krzywka3-18/+21
- 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>
2018-09-20Revert reactor upgrade (fix for tcnative bug)Piotr Jaszczyk1-1/+1
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>
2018-09-19Update Kotlin and Reactor versionsPiotr Jaszczyk1-2/+2
Change-Id: I7237cede772e79a1cd18835c85b43bcd81710a50 Issue-ID: DCAEGEN2-801 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-09-19Align to logging requirementsFilip Krzywka1-29/+39
- 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>
2018-09-18Merge "Align with latest HV-VES proto definition"Przemyslaw Wasala36-475/+535
2018-09-17Merge "update root license"Lusheng Ji1-202/+36
2018-09-17Align with latest HV-VES proto definitionFilip Krzywka36-475/+535
- 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>
2018-09-17Read flags from env variablesFilip Krzywka1-1/+2
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>
2018-09-14update root licensevagrant1-202/+36
Change-Id: If7db842404cc345057414d478b58874dd8b6853b Signed-off-by: Vijay Venkatesh Kumar <vv770d@att.com> Issue-ID: DCAEGEN2-595
2018-09-13Align with configuration present in consulJakub Dudycz2-4/+4
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>
2018-09-11Remove security vulnerabilityFilip Krzywka1-0/+7
Change-Id: If706e3eefb41f7c4a55212c70518f61069efeb40 Issue-ID: DCAEGEN2-766 Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
2018-09-11Enable ssl-disable option for xNF simulatorFilip Krzywka1-8/+15
Change-Id: I8d4459c4b612e6066ae0da9521333fc3083952d3 Issue-ID: DCAEGEN2-760 Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
2018-09-11Fix common event header parsingFilip Krzywka6-36/+144
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>
2018-09-11Increase of UT coverage in projectkjaniak3-12/+75
Change-Id: I8ac9ce21b5034f795da19e27e42ba95ce87db10d Issue-ID: DCAEGEN2-741 Signed-off-by: kjaniak <kornel.janiak@nokia.com>
2018-09-10Merge "Enable env parameters read"Przemyslaw Wasala5-25/+47
2018-09-10Enable env parameters readkjaniak5-25/+47
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>
2018-09-07Merge "Use managed guava version"Vijay Venkatesh Kumar1-12/+5
2018-09-06Handle non-existing Collector instanceFilip Krzywka4-11/+25
Change-Id: I0b6cd5023b2bca0f0bee6958c107fc560fc95b52 Issue-ID: DCAEGEN2-751 Signed-off-by: Filip Krzywka <filip.krzywka@nokia.com>
2018-09-06Build enhancementsPiotr Jaszczyk4-36/+60
* 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>
2018-09-06Improve coverage of xNF simulatorPiotr Jaszczyk38-321/+1037
Also refactor to make it possible. Change-Id: I6da6d3f33e57c524a7e353ecebd3e045d8ceed2a Issue-ID: DCAEGEN2-739 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-09-04Read config from environment variablesPiotr Jaszczyk5-5/+169
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>
2018-08-31Improve DCAE APP Simulator coveragePiotr Jaszczyk22-202/+968
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>
2018-08-30Remove additional SNAPSHOT from docker tagPiotr Jaszczyk1-1/+1
Change-Id: I3fa764feef63d1318adf7ba8128ea2b86055951a Issue-ID: DCAEGEN2-742 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-08-27Use managed guava versionGary Wu1-12/+5
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>
2018-08-27Fail build when coverage is too lowPiotr Jaszczyk3-0/+50
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>
2018-08-23Update to Jacoco 0.8.2 & fix coverage reportPiotr Jaszczyk2-20/+22
* 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>
2018-08-22Align with latest proposal of wire proto framePiotr Jaszczyk7-125/+144
Change-Id: I8f989a3913f1592b4d740a80ed30b01bc3aceff2 Issue-ID: DCAEGEN2-722 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-08-21Merge "Harmonize docker image names"Przemyslaw Wasala3-3/+0
2018-08-21Harmonize docker image namesPiotr Jaszczyk3-3/+0
Use groupId.artifactId naming convention. Change-Id: Iddcf59c895305677db10534696e4b026ca97e042 Issue-ID: DCAEGEN2-719 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-08-21Update dependenies' versionsPiotr Jaszczyk2-3/+4
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>
2018-08-21Rename artifacts to match LF requirementsPiotr Jaszczyk13-13/+13
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>
2018-08-17Merge "Remove Ratpack dependency for HV-VES health checks"Lusheng Ji25-178/+362
2018-08-16Remove Ratpack dependency for HV-VES health checksPiotr Jaszczyk25-178/+362
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>
2018-08-16Align with release docker & JARs CI definitionsPiotr Jaszczyk1-2/+3
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>
2018-08-13Reorganize docker buildPiotr Jaszczyk4-16/+35
* 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>
2018-08-13Link project as ONAP oparent module childFilip Krzywka3-191/+175
- 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>
2018-08-09Add netcat to docker imageFilip Krzywka1-0/+1
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>
2018-08-09Implement simple health check mechanismJakub Dudycz20-157/+488
Change-Id: Ic4b8b59ced9dc19c9ebf26131036a9e1a752164f Issue-ID: DCAEGEN2-659 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
2018-08-08Create health check moduleJakub Dudycz11-10/+162
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>
2018-08-07Clean up repositoryJakub Dudycz6-302/+0
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>
2018-08-07Correction of version.properties typokjaniak1-0/+0
Change-Id: Ib321f24ffc54b6dca321cc3a5929c7bdaebc790a Issue-ID: DCAEGEN2-683 Signed-off-by: kjaniak <kornel.janiak@nokia.com>
2018-08-06Add INFO.yaml fileJessica Wagantall1-0/+71
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>
2018-08-06Add of version.properties filekjaniak1-0/+6
Change-Id: Id6e282d8865a425bc1a093d6a495cd4fabe3d98f Issue-ID: DCAEGEN2-683 Signed-off-by: kjaniak <kornel.janiak@nokia.com>
2018-08-06Upgrade to Kotlin 1.2.60Jakub Dudycz1-1/+1
Change-Id: I089c63edd1859c6941f0791f1cf3a09526b6d028 Issue-ID: DCAEGEN2-660 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
2018-08-03Disable proxy when building Docker imagesPiotr Jaszczyk1-0/+2
Change-Id: I72d5c511a90fd2d357227aa94b968cc63bd4ca68 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com> Issue-ID: DCAEGEN2-653
2018-08-03Re-enable docker buildPiotr Jaszczyk4-41/+35
* 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
2018-08-03Remove default consul configurationJakub Dudycz5-43/+48
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
2018-08-03Extract test-utils modulefkrzywka23-261/+321
- 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
2018-08-03Merge "Move MessageParametersParser"Przemyslaw Wasala9-18/+54