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-14947] Create analysis doc for Micrometer Support #445

Merged
merged 1 commit into from
Feb 2, 2023

Conversation

jasondlee
Copy link
Contributor

* RBAC support for server/jvm metrics
* Remove base/vendor prefixes (in accordance with Prometheus standards)
* This subsystem will replace the existing WildFly Metrics subsystem. Documentation will be provided describing the change.
* If MicroProfile Metrics is enabled, this subsystem will decline to install itself, as is the current behavior of WildFly Metrics.
Copy link
Contributor

Choose a reason for hiding this comment

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

@jasondlee We need migration requirements for the existing WildFly Metrics subsystem.

Either:

  1. It continues to work but there is appropriate logic to deal with the case where both it and the Micrometer subsystem are in a non-admin-only server config.
  2. It becomes a model-only / admin-only subsystem with a migrate op added to migrate to the new subsystem.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've update the language here a bit, but I'm not entirely sure what more you're looking for if this is insufficient.

* https://issues.redhat.com/browse/WFLY-14947

=== Related Issues
* https://issues.redhat.com/browse7-1686
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a broken URL.

@bstansberry bstansberry merged commit 2f3c7e2 into wildfly:main Feb 2, 2023
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.

2 participants