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

fix: update log rollover configuration #6086

Merged
merged 1 commit into from Apr 13, 2023
Merged

Conversation

nathanklick
Copy link
Member

Description

Updates the log4j2 appender configurations to enable time and size based rollovers with automatic cleanup. Additionally, this enables automatic gzip compression of archived logs.

Related Issues

Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
@github-actions
Copy link

Node: Unit Test Results

  1 306 files    1 306 suites   1h 24m 59s ⏱️
97 170 tests 97 163 ✔️ 7 💤 0
98 810 runs  98 803 ✔️ 7 💤 0

Results for commit 413640b.

@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Patch coverage has no change and project coverage change: +23.26 🎉

Comparison is base (aac4674) 68.08% compared to head (413640b) 91.35%.

Additional details and impacted files
@@              Coverage Diff               @@
##             develop    #6086       +/-   ##
==============================================
+ Coverage      68.08%   91.35%   +23.26%     
+ Complexity     22015    17028     -4987     
==============================================
  Files           1964     1277      -687     
  Lines         134024    48672    -85352     
  Branches        7547     4865     -2682     
==============================================
- Hits           91253    44462    -46791     
+ Misses         41298     3280    -38018     
+ Partials        1473      930      -543     

see 921 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sonarcloud
Copy link

sonarcloud bot commented Apr 13, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@github-actions
Copy link

Node: Integration Test Results

    3 files      3 suites   13m 44s ⏱️
150 tests 150 ✔️ 0 💤 0
151 runs  151 ✔️ 0 💤 0

Results for commit 413640b.

@github-actions
Copy link

Node: E2E Test Results

    1 files      1 suites   16m 46s ⏱️
307 tests 307 ✔️ 0 💤 0
325 runs  325 ✔️ 0 💤 0

Results for commit 413640b.

@nathanklick nathanklick merged commit 2d9637a into develop Apr 13, 2023
19 of 20 checks passed
@nathanklick nathanklick deleted the 04745-Log4j2-Rollover branch April 13, 2023 20:38
nathanklick added a commit that referenced this pull request Apr 13, 2023
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
povolev15 pushed a commit that referenced this pull request Apr 14, 2023
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
@nickpoorman nickpoorman modified the milestones: v0.37.0, v0.38.0 Apr 14, 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.

swirlds-hashstream.log is not rolling over
5 participants