Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WFLY-10712] Observability: MicroProfile Metrics #11830

Merged

Conversation

jmesnil
Copy link
Member

@jmesnil jmesnil commented Nov 9, 2018

Add microprofile-metrics-smallrye extension to provide support for
MicroProfile Metrics 1.1

  • integrate smallrye-metrics:1.1.2 that implements MicroProfile Metrics
    1.1
  • provide the /metrics HTTP endpoint on the management interface
  • The extension is added to standalone profiles only (not available in
    domain mode)

Dev analysis: wildfly/wildfly-proposals#147
JIRA: https://issues.jboss.org/browse/WFLY-10712

@jmesnil
Copy link
Member Author

jmesnil commented Nov 9, 2018

@rsvoboda fiy I've squashed all the commits from the dev branch in a single commit

The community documentation is in progress, I'll add it a commit later.

@jmesnil jmesnil added Feature PR provides a new feature missing-reqs Features missing one or more of the pre-merge requirements fixme labels Nov 9, 2018
@jmesnil jmesnil force-pushed the WFLY-10712_microprofile_metrics-smallrye branch from f410f9a to a81f923 Compare November 9, 2018 17:30
@jmesnil jmesnil force-pushed the WFLY-10712_microprofile_metrics-smallrye branch 2 times, most recently from 9ae5f13 to a73a01c Compare November 12, 2018 13:20
@jmesnil jmesnil force-pushed the WFLY-10712_microprofile_metrics-smallrye branch from a73a01c to 5c6cbe5 Compare November 12, 2018 15:10
@rsvoboda
Copy link
Contributor

I played a bit with this at home yesterday before going to bed and I noticed issues in docs. I just created https://issues.jboss.org/browse/WFWIP-150.
There are some issues that need my attention for 7.2, not sure if I will be able to come back to MP Metric before FF of WF 15.

@jmesnil jmesnil force-pushed the WFLY-10712_microprofile_metrics-smallrye branch from 5c6cbe5 to f89b5f5 Compare November 13, 2018 09:06
@jmesnil
Copy link
Member Author

jmesnil commented Nov 13, 2018

@jamezp could you please review
f89b5f5 and approve it?

@jmesnil jmesnil removed the fixme label Nov 13, 2018
@MessageLogger(projectCode = "WFLYMETRICS", length = 4)
public interface MicroProfileMetricsLogger extends BasicLogger {
/**
* A logger with the category {@code org.wildfly.extension.batch}.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a big deal, but we might want to change the category name here.

@jmesnil jmesnil force-pushed the WFLY-10712_microprofile_metrics-smallrye branch from 6b834f2 to 1f9b163 Compare November 13, 2018 16:55
Add microprofile-metrics-smallrye extension to provide support for
MicroProfile Metrics 1.1
* integrate smallrye-metrics:1.1.2 that implements MicroProfile Metrics
  1.1
* provide the /metrics HTTP endpoint on the management interface
* The extension is added to standalone profiles only (not available in
  domain mode)

Dev analysis: wildfly/wildfly-proposals#147
JIRA: https://issues.jboss.org/browse/WFLY-10712
This ensures that the deployment will be considered complete once Batch
jobs services have been properly installed.
@jmesnil jmesnil force-pushed the WFLY-10712_microprofile_metrics-smallrye branch from 1f9b163 to 1e7b236 Compare November 13, 2018 17:52
@kabir kabir removed the missing-reqs Features missing one or more of the pre-merge requirements label Nov 13, 2018
@kabir kabir merged commit bada8cb into wildfly:master Nov 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature PR provides a new feature
Projects
None yet
5 participants