T8011: VPP fix log duplication in systemd journal #4882
Merged
+3
−0
Conversation
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
Set the standard output/error to null to avoid log duplication. Starting the service as "nodaemon" seems cause of this behavior.
|
👍 |
|
CI integration 👍 passed! Details
|
c-po
approved these changes
Dec 2, 2025
Member
c-po
left a comment
c-po
left a comment
There was a problem hiding this comment.
Avoid duplicate logs to syslog/journal.
As vpp runs supervised it captures both internal syslog glibc calls and stdout. Omit the latter beeing sent to syslog/journal, too.
Contributor
|
Are we absolutely certain that no logs are printed only to stdout/stderr and not forwarded to syslog? |
Member
Author
@zdc I do not see any way to say exactly. I see the log duplication and provided minimal set of commands to reproduce. I prefer this format, as another format with |
zdc
approved these changes
Dec 9, 2025
Contributor
zdc
left a comment
zdc
left a comment
There was a problem hiding this comment.
While it is not an ideal solution from my perspective, I think we can accept it.
vyosbot
added
mirror-initiated
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Change summary
Set the standard output/error to
nullto avoid log duplication.Starting the service as "nodaemon" seems cause of this behavior.
Fixes VPP log duplication
Types of changes
Related Task(s)
Related PR(s)
How to test / Smoketest result
Before the fix, we see duplicate log entries:
After the fix we do not see log duplication:
Checklist: