TOMEE-4113 - Update to MicroProfile Metrics 5 & SmallRye Metrics 5 #972
Conversation
…nd increase memory limit for remote TomEE server
…f we expose via JMX.
|
@jgallimore What do we need to do here? |
|
I'll take a look. I did quite a lot of work in this area, so it would be nice to get this over the line. |
# Conflicts: # boms/tomee-microprofile/pom.xml # boms/tomee-plume/pom.xml # boms/tomee-plus/pom.xml # examples/mp-metrics-timed/pom.xml # pom.xml # tomee/tomee-microprofile/mp-common/pom.xml # tomee/tomee-microprofile/mp-common/src/main/java/org/apache/tomee/microprofile/TomEEMicroProfileListener.java # tomee/tomee-microprofile/mp-common/src/main/java/org/apache/tomee/microprofile/metrics/MPMetricsCDIExtension.java
# Conflicts: # pom.xml
|
I just merged in the latest changes from main (from the last years). The Metrics TCK has only 3 failures :) |
|
did it pass before the upgrade? Otherwise lets merge and try to fix all TCKs after M1? |
yes, it was ok ;-) (with 4.x) |
# Conflicts: # boms/tomee-microprofile/pom.xml # boms/tomee-plume/pom.xml # boms/tomee-plus/pom.xml # pom.xml
|
@jgallimore @tandraschko I expect the PR CI build to be green now, so we can move ahead with integrating this great PR into "main". TCK is passing (aside from 3 tests, which are optional) - therefore, I marked them as excluded (for now) |
Removes outdated extension Adjust example to metrics 5
|
Thanks @jgallimore - TCK for Metrics is good; some of the examples aren't compatible anymore which will be fixed with TOMEE-4329 - Fix Metrics examples to comply with Metrics 5 |
No description provided.