aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-11-06Upgrade Kotlin to 1.3Piotr Jaszczyk11-11/+24
Migrate to Kotlin 1.3 with non-experimental coroutines support. Also bump Arrow-Kt version so it follows 1.3 release. Change-Id: Ia2f26d0c710b7cfae709e35700b72607aeddaf00 Issue-ID: DCAEGEN2-949 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-11-05Change version numbers for Dublin releasePiotr Jaszczyk15-16/+16
Change-Id: I4e1a53ea95ec8d880ff00aa94d3ba5511c23eadf Issue-ID: DCAEGEN2-943 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-10-05Remove any inconsistencies with specification3.0.0-ONAP1.0.0Piotr Jaszczyk8-140/+38
* Fix payloadId field issue (was 1 byte, should use 2 byts) * Copy final version of protobuf and asn definitions (mostly comments) * Added links to yet-to-be updated RTD documentation Change-Id: I69bda676423ad601797d95577ff8af6707cacb0c Issue-ID: DCAEGEN2-857 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-10-05Introduce configurable payload size limitationPiotr Jaszczyk34-61/+137
Maximum payload size will be configurable (from command line parameter or environment variable). The default value is same as previous hardcoded value, ie. 1 MiB = 1024 * 1024 bytes. Change-Id: Iec83d8295252bac353d3794b13454fdbbc80ecc4 Issue-ID: DCAEGEN2-856 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-10-04Change magic byte from 0xFF to 0xAAPiotr Jaszczyk3-8/+8
In final protocol specification the value has changed to 0xAA. Change-Id: Id86c15117732b03bca38c674960d5eba8da88a6e Issue-ID: DCAEGEN2-854 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-10-03Change domain to lower casePiotr Jaszczyk10-45/+54
Change-Id: I097def2ba720fafe0a1e1a88e87f8c03663627c9 Issue-ID: DCAEGEN2-847 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-10-01HV-VES Domain updatePiotr Jaszczyk17-218/+215
Change domain name. In this project this yields following changes: * Comment change in VesEvent proto * Updated measurements proto (kept as documentation only, ie. not used in the collector) * Update name in test scenarios for unit and component scope Change-Id: I719077b5fcb642c067d25b7aa174a2e629404689 Issue-ID: DCAEGEN2-847 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-09-24Use JDK security providerPiotr Jaszczyk41-600/+948
Replace netty-tcnative bindings for OpenSSL with JDK provided implementation by default. Change-Id: I59a4797ce43d15a791eab00bfd25cb730a271207 Issue-ID: DCAEGEN2-816 Signed-off-by: Piotr Jaszczyk <piotr.jaszczyk@nokia.com>
2018-09-24Merge "Remove end-of-transmission message from protocol"Przemyslaw Wasala22-455/+201
2018-09-24Merge "Update README.md"Przemyslaw Wasala1-0/+55
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-20Update README.mdIzabela Zawadzka1-0/+55
Add important links, background and implementation details, describe compability aspects and extendability Change-Id: Ie38300a0d33b347c5d0d82b3e8547039e54829f3 Issue-ID: DCAEGEN2-798 Signed-off-by: Izabela Zawadzka <izabela.zawadzka@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>