Kubernetes tracing and metrics improvements #3298
Merged
Conversation
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
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
steven-sheehy
added
enhancement
Codecov Report
@@ Coverage Diff @@
## main #3298 +/- ##
============================================
+ Coverage 88.16% 88.25% +0.08%
- Complexity 2507 2517 +10
============================================
Files 502 502
Lines 14143 14164 +21
Branches 1306 1310 +4
============================================
+ Hits 12469 12500 +31
+ Misses 1365 1358 -7
+ Partials 309 306 -3
Continue to review full report at Codecov.
|
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
|
SonarCloud Quality Gate failed.
|
Nana-EC
approved these changes
Feb 14, 2022
matheus-dallrosa
pushed a commit
that referenced
this pull request
Feb 18, 2022
* Add missing REST configuration to docs * Change REST API to log at info level by default to reduce log volume * Change REST API to generate a consistent trace log for all responses * Change Traefik to only log non-2xx requests to reduce log volume * Fix client IP not showing up in any trace logs * Fix Lettuce histogram metrics enabled by default and generating ~10K series * Fix monitor not tracing requests in some scenarios and add test coverage * Improve GCP Pub/Sub performance by reusing printer across transactions * Remove some REST API metric buckets to reduce number of overall time series Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com> Signed-off-by: Matheus DallRosa <matheus.dallrosa@swirlds.com>
matheus-dallrosa
pushed a commit
that referenced
this pull request
Feb 21, 2022
* Add missing REST configuration to docs * Change REST API to log at info level by default to reduce log volume * Change REST API to generate a consistent trace log for all responses * Change Traefik to only log non-2xx requests to reduce log volume * Fix client IP not showing up in any trace logs * Fix Lettuce histogram metrics enabled by default and generating ~10K series * Fix monitor not tracing requests in some scenarios and add test coverage * Improve GCP Pub/Sub performance by reusing printer across transactions * Remove some REST API metric buckets to reduce number of overall time series Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com> Signed-off-by: Matheus DallRosa <matheus.dallrosa@swirlds.com>
matheus-dallrosa
pushed a commit
that referenced
this pull request
Feb 21, 2022
* Add missing REST configuration to docs * Change REST API to log at info level by default to reduce log volume * Change REST API to generate a consistent trace log for all responses * Change Traefik to only log non-2xx requests to reduce log volume * Fix client IP not showing up in any trace logs * Fix Lettuce histogram metrics enabled by default and generating ~10K series * Fix monitor not tracing requests in some scenarios and add test coverage * Improve GCP Pub/Sub performance by reusing printer across transactions * Remove some REST API metric buckets to reduce number of overall time series Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com> Signed-off-by: Matheus DallRosa <matheus.dallrosa@swirlds.com>
matheus-dallrosa
pushed a commit
that referenced
this pull request
Feb 22, 2022
* Add missing REST configuration to docs * Change REST API to log at info level by default to reduce log volume * Change REST API to generate a consistent trace log for all responses * Change Traefik to only log non-2xx requests to reduce log volume * Fix client IP not showing up in any trace logs * Fix Lettuce histogram metrics enabled by default and generating ~10K series * Fix monitor not tracing requests in some scenarios and add test coverage * Improve GCP Pub/Sub performance by reusing printer across transactions * Remove some REST API metric buckets to reduce number of overall time series Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com> Signed-off-by: Matheus DallRosa <matheus.dallrosa@swirlds.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description:
Related issue(s):
Notes for reviewer:
REST API output will now appear consistently in the below format:
INFO 9e4e7ca3 ::1 GET /api/v1/transactions?limit=2 in 37 ms: 200Checklist