aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
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
2018-08-03Merge "Fix NPE when getting Consul configuration"Przemyslaw Wasala5-78/+92
2018-08-03Move MessageParametersParserfkrzywka9-18/+54
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
2018-08-03Fix NPE when getting Consul configurationPiotr Jaszczyk5-78/+92
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
2018-08-03Use proxy while publishing docker images on CIfkrzywka1-2/+6
Also fix gitlab-ci syntax Change-Id: I5e5e1db3888f976640bacc161d703cc6be3ab613 Signed-off-by: fkrzywka <filip.krzywka@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-03Define simple healthchecks for simulatorsfkrzywka6-1/+22
- 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
2018-08-03DCAE APP simulator reworkJakub Dudycz17-193/+320
- 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
2018-08-03Pass CommonEventHeader to XNF simulator apiJakub Dudycz13-114/+325
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
2018-08-03Enable UNDEFINED option in commonHeader GPB schemakjaniak4-35/+78
Closes ONAP-655 Change-Id: I6fc830d2b8c7bdd726f306a6fbf7e979f39e03f0 Signed-off-by: kjaniak <kornel.janiak@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-03Rework argument configurationJakub Dudycz18-129/+228
- 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
2018-08-03Configure xnf simulator api listen portJakub Dudycz9-71/+53
Closes ONAP-664 Closes ONAP-647 Change-Id: I8d1e57c0c66f1968925660fe47fa96a2a634c039 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-03Fix consul request timeout issueJakub Dudycz10-40/+107
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
2018-08-03Support scenarios for continuous streaming testJakub Dudycz8-99/+277
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
2018-08-03Add command line option to disable SSL/TLSkjaniak10-47/+126
Closes ONAP-508 Change-Id: If6c3935ede7b00dea9b36747c6cd1422c1c8d330 Signed-off-by: kjaniak <kornel.janiak@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-03Use Try/Option monad when decoding protobuffkrzywka3-19/+44
Closes ONAP-143 Change-Id: I33cb2d24cd5962318a6f405096db298bbdbab963 Signed-off-by: fkrzywka <filip.krzywka@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-03Merge "Include msg generator module in coverage report"Przemyslaw Wasala16-29/+21
2018-08-03Include msg generator module in coverage reportfkrzywka16-29/+21
* 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
2018-08-03Remove unused methodfkrzywka1-7/+0
Change-Id: I462e78f0bad9aba2e82d2766d748608b6231c839 Signed-off-by: fkrzywka <filip.krzywka@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-02Add API endpoint for DCAE App Sim reconfigurationfkrzywka4-10/+38
Closes ONAP-547 Change-Id: I26b7d5d61eb984d58600a612cd8d3e4dd7be05c5 Signed-off-by: fkrzywka <filip.krzywka@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-02Extract message generator modulekjaniak12-57/+154
Closes ONAP-501 Change-Id: Ia5dc5f67e18c302abbeac7e0373cef479f4befb8 Signed-off-by: kjaniak <kornel.janiak@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-02Use MessageValidator in VesMessage classJakub Dudycz5-8/+8
Closes ONAP-493 Change-Id: I15b5f0f1052865aa29ffa103bef2368bd94021e8 Signed-off-by: Jakub Dudycz <jdudycz@nokia.com> Issue-ID: DCAEGEN2-601
2018-08-02Use Flux.transform in VesHvCollectorJakub Dudycz2-16/+26
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
2018-08-02Add byte-buffer handling assertions when decodingfkrzywka1-2/+14
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