Skip to content

Added the KumuluzEE implementation for the schedule microservice#169

Open
TFaga wants to merge 6 commits intomicroprofile:mainfrom
TFaga:master
Open

Added the KumuluzEE implementation for the schedule microservice#169
TFaga wants to merge 6 commits intomicroprofile:mainfrom
TFaga:master

Conversation

@TFaga
Copy link
Copy Markdown

@TFaga TFaga commented Sep 19, 2017

No description provided.

Signed-off-by: Tilen Faganel <tilen.faganel@me.com>
@OndroMih
Copy link
Copy Markdown
Contributor

I tried to build with mvn install -DskipTests and I got an exception when the app was started:

[ERROR] java.lang.NoClassDefFoundError: org/objectweb/asm/ClassVisitor
[ERROR]     a  t  org.eclipse.je tty.annotations.AnnotationConfiguration.createAnnotationParser(AnnotationConfiguration.java:506)
[ERROR]     a  t  org.eclipse.je tty.annotations.AnnotationConfiguration.scanForAnnotations(AnnotationConfiguration.java:421)
[ERROR]     a  t  org.eclipse.je tty.annotations.AnnotationConfiguration.configure(AnnotationConfiguration.java:363)

When I then executed the JAR from command line it was OK. So I think it's enough if the build doesn't start the app, only builds the JAR.

@TFaga
Copy link
Copy Markdown
Author

TFaga commented Sep 20, 2017

I see. I've put the tests to run as part of the integration-test phase, I'll try and move them to the test phase or make sure to disable them if the option is passed, so they can be skipped when installing.

@OndroMih
Copy link
Copy Markdown
Contributor

@TFaga, have you signed the Eclipse Contributor Agreement? If yes, did you use tilen.faganel@me.com with Eclipse? Otherwise, please see https://www.eclipse.org/legal/ECA.php to sign the agreement, so that we may accept your PRs.

…odule when supplying the -DskipTests property on maven build

Signed-off-by: Tilen Faganel <tilen.faganel@me.com>
@TFaga
Copy link
Copy Markdown
Author

TFaga commented Sep 21, 2017

I've added the tests to a profile that is disabled when -DskipTests is used. Just signed the agreement, should be OK now.

@TFaga
Copy link
Copy Markdown
Author

TFaga commented Sep 30, 2017

We have added KumuluzEE implementations of the Schedule and the Speaker microservices. Both services have Health (/health endpoint), Metrics (/metrics) and Config included and implemented. For Metrics, there is also a Prometheus exporter accessible on /prometheus (which is a nice addition for the demo).

@pilhuhn
Copy link
Copy Markdown

pilhuhn commented Sep 30, 2017 via email

@TFaga
Copy link
Copy Markdown
Author

TFaga commented Oct 1, 2017

We have mapped the Prometheus to /prometheus endpoint and JSON output to /metrics endpoint, in order to better demonstrate both of the formats.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants