Skip to content

Conversation

cbosdo
Copy link
Contributor

@cbosdo cbosdo commented Aug 25, 2025

Using rsyslog and logrotate to write the log file pulls too many useless dependencies like wicked. Implementing it directly in the program using lumberjack makes it more lightweight.

@cbosdo
Copy link
Contributor Author

cbosdo commented Aug 26, 2025

The PR has been tested with an image build:

  • wicked is gone
  • inter-server-sync writes log in /var/log/hub/iss2.log

@cbosdo
Copy link
Contributor Author

cbosdo commented Aug 26, 2025

Fixes #85 and uyuni-project/uyuni#10726

@cbosdo cbosdo requested a review from aaannz August 26, 2025 07:52
Using rsyslog and logrotate to write the log file pulls too many useless
dependencies like wicked. Implementing it directly in the program using
lumberjack makes it more lightweight.
Copy link
Contributor

@aaannz aaannz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks

@cbosdo cbosdo merged commit cc2ce73 into uyuni-project:main Aug 26, 2025
2 checks passed
@cbosdo cbosdo deleted the rsyslog branch August 26, 2025 09:44
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.

2 participants