cloudstack icon indicating copy to clipboard operation
cloudstack copied to clipboard

syslog configuration update for log4j2

Open DaanHoogland opened this issue 8 months ago • 10 comments

Description

This PR updates the syslog configuration log4j 2. The configuration as was includes non-existent elements and seems incomplete.

Types of changes

  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] Enhancement (improves an existing feature and functionality)
  • [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
  • [ ] build/CI
  • [ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • [ ] Major
  • [ ] Minor

Bug Severity

  • [ ] BLOCKER
  • [ ] Critical
  • [ ] Major
  • [ ] Minor
  • [ ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

On a server the lines as in this PR were changed and rsyslog was configured with the imtcp module. Messages would then appear in /various/log/messages

How did you try to break this feature and the system with this change?

DaanHoogland avatar Jun 06 '25 08:06 DaanHoogland

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 16.15%. Comparing base (10fc1f3) to head (fd85caa). :warning: Report is 183 commits behind head on 4.20.

Additional details and impacted files
@@            Coverage Diff            @@
##               4.20   #10980   +/-   ##
=========================================
  Coverage     16.14%   16.15%           
- Complexity    13255    13273   +18     
=========================================
  Files          5656     5657    +1     
  Lines        497893   497898    +5     
  Branches      60374    60374           
=========================================
+ Hits          80391    80435   +44     
+ Misses       408540   408506   -34     
+ Partials       8962     8957    -5     
Flag Coverage Δ
uitests 4.00% <ø> (-0.01%) :arrow_down:
unittests 17.00% <ø> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Jun 06 '25 08:06 codecov[bot]

@DaanHoogland can you give more details in the description and the title ?

weizhouapache avatar Jun 06 '25 09:06 weizhouapache

@DaanHoogland can you give more details in the description and the title ?

will do, work in prgress though (untested as of yet)

DaanHoogland avatar Jun 06 '25 13:06 DaanHoogland

@blueorangutan package

paulgosden avatar Jun 10 '25 10:06 paulgosden

@paulgosden a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar Jun 10 '25 10:06 blueorangutan

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13690

blueorangutan avatar Jun 10 '25 11:06 blueorangutan

@blueorangutan package

DaanHoogland avatar Jun 13 '25 17:06 DaanHoogland

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar Jun 13 '25 18:06 blueorangutan

Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 13767

blueorangutan avatar Jun 13 '25 18:06 blueorangutan

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13776

blueorangutan avatar Jun 14 '25 10:06 blueorangutan

@harikrishna-patnala @weizhouapache , do we need/want this, or close?

DaanHoogland avatar Oct 17 '25 10:10 DaanHoogland