-
Notifications
You must be signed in to change notification settings - Fork 566
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
MP Metrics 5.0 support for 4.x #7139
Merged
Merged
Changes from all commits
Commits
Show all changes
67 commits
Select commit
Hold shift + click to select a range
b3d1193
Initial commit of MP based on Micrometer
tjquinno 88e75c1
More tests; start feature work; start CDI work
tjquinno a0a8324
More tests; start feature work; start CDI work
tjquinno 9d1ce27
Added basics for timers, gauges
tjquinno b19be5a
Improve formatter API; fix and expand tests; plug feature in; start o…
tjquinno 97dc30d
Finish missing method impls in metric registry impl; improve formatte…
tjquinno e4a9a67
Fix some checkstyle, JavaDoc comments, etc.
tjquinno 4693d83
Further style, etc. fixes
tjquinno c7b178e
metrics/api stable; metrics/metrics WIP
tjquinno ea53e8d
More improvements to metrics/api and metrics/metrics
tjquinno c08c7ce
Fixes to metrics/api and metrics/metrics mostly
tjquinno 8cef669
Copyright, spotbugs fixes
tjquinno 8aa27f2
Fix KPI metrics
tjquinno 9002585
Rebasing
tjquinno f7b7b0e
Copyright and style fixes mostly
tjquinno 3bdb3f6
Try older Prom client version
tjquinno 093cd13
Further small fixes to accomodate MP metrics API changes
tjquinno 318aa71
Fix isolated Prometheus support to deal with later release
tjquinno dc43b78
Remove temp metrics feature under MP metrics from bom
tjquinno eff3ade
Clean up tag handling and Prom. formatting
tjquinno 07b0409
Centralize tag handling in existing SystemTagsManager
tjquinno 608a2ce
Some MP clean-up; more to come - push for safekeeping
tjquinno aa2cc62
Improvements in tag handling for scopes, reject options requests now …
tjquinno 5a24b46
More changes adapting to change in metrics API
tjquinno f929d00
Fix some tag handling and Prometheus output filtering
tjquinno c7c1a37
Add back JSON formatting; related fixes
tjquinno f2fb1eb
Fixes in JSON output rework; various fixes in examples and tests
tjquinno ea62a4b
A few more fixes in other components and style clean-up
tjquinno 32e8038
A few more edits to FT
tjquinno 18c562e
Fix dead store
tjquinno c459516
Return to using the deprecated RegistryType annotation (which is a qu…
tjquinno 536debe
Add cross-scope validation of metadata and tag name sets for same-nam…
tjquinno 7608958
Add metric name to error message for more clarity
tjquinno 5f45d77
Enforce same-named metric restrictions within scope only, not across …
tjquinno 9bc9d65
Fix some meter filtering for output; temporarily disable enforcement …
tjquinno 62d8790
Add functional counter support
tjquinno f47739c
Improve system tag handling
tjquinno 9e20514
Uncomment stringent consistency checking for tag name sets; try arq. …
tjquinno 57f5169
Remove meters from archetype (meters are no longer part of metrics)
tjquinno 73ad2b9
Restore JSON output to MetricsSupport for now
tjquinno 3bcf1ec
Disable exemplar sample check for exemplar while we migrate exemplar …
tjquinno a7e0722
Fix KPI examples; it needs to look for gauge values now
tjquinno 6ce3654
Add missing verison spec for Helidon artifact
tjquinno b326082
More fixes; change native test to check timers instead of simple time…
tjquinno b39de03
Need to use MP FT 4.0.2 to depend on correct MP metrics release
tjquinno b6f0eea
Adjust to removal of scope tag from JSON output
tjquinno f368262
Fix bug in deletion; add test (not the cause of the TCK problems, tho…
tjquinno 08f08a4
Fix up JavaDoc link to match changes I made to the referenced method …
tjquinno 4a0f8fc
Minor fix to JSON formatting and to bookstore functional test (to be …
tjquinno 21da420
Handle per-metric enable/disable in formatting
tjquinno 2f2a429
Make sure to return 404 if selective metrics retrieval finds no match…
tjquinno 22a26a6
Add back EOF filtering from Prom. output; fix Optional issue
tjquinno 0151c96
Remove obsolete metrics usage
tjquinno d67a736
Remove refc to now-removed min value for timer
tjquinno f5de4d3
Fix bad change for registry type qualification on injection
tjquinno fc0e002
Various clean-up changes; pruning unneeded files and directories
tjquinno d187627
Style
tjquinno 8bf9659
Handle injections modified with @RegistryScope as well as @RegistryType
tjquinno add80e3
Bug fixes
tjquinno 7916312
Further fine-tune regex to exclude TCK jars from Weld scanning
tjquinno d706ce9
Refine the Arq config
tjquinno 60785d4
Add logic to clear data structures between same-JVM TCK tests (essent…
tjquinno 26c4cc1
Restore metrics TCK runs
tjquinno 21ae6ab
Report 'base' as a scope, even if that registry has not been on-deman…
tjquinno 9438aee
Move clean-out of registries from afterStart to beforeStop; some othe…
tjquinno ec5fd30
Move per-Arquillian cleanup from before bean discovery to before CDI …
tjquinno e138b14
Fix copyright date
tjquinno File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you place the issue number here as part of the follow-up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will update in this PR if there are other, functional, changes as well.