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

support shenyu-admin expose prometheus metrics #4336

Merged
merged 6 commits into from
Feb 23, 2023

Conversation

tomsun28
Copy link
Contributor

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Hi, this pr support shenyu-admin expose prometheus metrics. please take a review. thanks!

2023-01-30 23 07 45

@tomsun28
Copy link
Contributor Author

Use the grafana SpringBoot APM Dashboard, Dashboard ID is 2900.
023-01-31 12 53 56

@codecov-commenter
Copy link

codecov-commenter commented Jan 31, 2023

Codecov Report

Merging #4336 (c5fa7dd) into master (11d542b) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4336      +/-   ##
============================================
- Coverage     68.21%   68.19%   -0.02%     
+ Complexity     7484     7483       -1     
============================================
  Files          1020     1020              
  Lines         28870    28870              
  Branches       2577     2577              
============================================
- Hits          19693    19689       -4     
- Misses         7635     7638       +3     
- Partials       1542     1543       +1     
Impacted Files Coverage Δ
...controller/ShenyuClientHttpRegistryController.java 63.63% <0.00%> (-18.19%) ⬇️
...ruptor/RegisterClientServerDisruptorPublisher.java 55.55% <0.00%> (-11.12%) ⬇️
...henyu/common/cache/MemorySafeWindowTinyLFUMap.java 73.33% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@loongs-zhang
Copy link
Member

@tomsun28 , hi, please add wechat Z18179469818

@tomsun28 tomsun28 deleted the shenyu-admin-prometheus branch February 23, 2023 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants