fix: enable rsyslog so rotv actually forwards its logs (RT #1460) - #580
Open
fatherlinux wants to merge 2 commits into
Open
fix: enable rsyslog so rotv actually forwards its logs (RT #1460)#580fatherlinux wants to merge 2 commits into
fatherlinux wants to merge 2 commits into
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
rotv shipped with the forwarding drop-in from ubi10-core present at /etc/rsyslog.d/00-crunchtools-forward.conf, but with rsyslog.service DISABLED -- so it sent nothing to the central collector while looking correctly configured. Cause: CI builds this image FROM quay.io/crunchtools/rotv-base rather than directly from ubi10-core, so the enable symlink ubi10-core creates does not reliably survive into the final image. Verified by inspecting both: ubi10-core: etc/systemd/system/multi-user.target.wants/rsyslog.service rotv: etc/systemd/system/multi-user.target.wants/postgresql.service (no rsyslog) Enabling it explicitly alongside the other services makes this independent of what the pre-baked base happens to carry. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014dGq9ivDC6sqeJJfkra6C2
fatherlinux
force-pushed
the
fix/enable-rsyslog-forwarding
branch
from
August 23, 2026 18:31
ea9e457 to
5f05921
Compare
Gatehouse flagged that nothing guarded the fix in the previous commit, which is fair -- the regression it fixes was invisible precisely because the config file was present and only the service was off. Asserted in tests.yml, which already boots the image under systemd. Checked here rather than assumed from ubi10-core's own smoke test, because CI builds this image FROM rotv-base and the enable does not reliably survive that hop. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014dGq9ivDC6sqeJJfkra6C2
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
rotv ships the forwarding drop-in from ubi10-core but with
rsyslog.servicedisabled, so it looks correctly configured and sends nothing.Why
CI builds
FROM quay.io/crunchtools/rotv-base:latest, not directly fromubi10-core, so the enable symlinkubi10-corecreates does not reliably reach the final image. Confirmed by comparing the two:Enabling it explicitly makes this independent of whatever the pre-baked base happens to carry.
Worth a separate look
The
rotv-baseindirection means rotv can silently lagubi10-corein other ways too. Anything added to the shared base is only guaranteed to reach rotv oncerotv-baseis rebuilt.Every other systemd container on lotor picked this up correctly in today's rollout — rotv is the only one that didn't.
🤖 Generated with Claude Code
https://claude.ai/code/session_014dGq9ivDC6sqeJJfkra6C2