Skip to content

Add config flag to output certain streams to stdout - #11

Merged
thomas-samoht merged 1 commit into
mainfrom
feat/config-change-to-let-streams-to-stdout
Sep 7, 2026
Merged

Add config flag to output certain streams to stdout#11
thomas-samoht merged 1 commit into
mainfrom
feat/config-change-to-let-streams-to-stdout

Conversation

@thomas-samoht

@thomas-samoht thomas-samoht commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

stdout was JSON, and the only way to make it readable replaced the whole
console. Now stdout is always plain text, JSON is the syslog format alone, and
console_streams picks which streams reach the terminal. One tagged line per
stream, each under its own field allow-list. Both paths run at once.

SIEM and APP are on the console by default. Can be changed by setting console_streams config setting.

Ticket: https://github.com/minvws/gfmodules-coordination-private/issues/1047

@thomas-samoht thomas-samoht changed the title Add console_streams config flag to output app, siem, debug to stdout Add console_streams config flag to output certain streams to stdout Sep 3, 2026
@thomas-samoht thomas-samoht changed the title Add console_streams config flag to output certain streams to stdout Add config flag to output certain streams to stdout Sep 3, 2026
@thomas-samoht
thomas-samoht force-pushed the feat/config-change-to-let-streams-to-stdout branch from d457c8d to 64aa8d6 Compare September 4, 2026 19:53
stdout carried JSON, and debug_logs_in_console replaced the console rather than
configuring it. stdout is now plain text, JSON is the syslog format alone, and
console_streams selects which streams reach the terminal: one tagged line per
stream, each under its own field allow-list. Defaults to ["app", "siem"], empty
silences stdout, an unknown name fails at boot.
@thomas-samoht
thomas-samoht force-pushed the feat/config-change-to-let-streams-to-stdout branch from 64aa8d6 to ebdac20 Compare September 7, 2026 07:26
@thomas-samoht
thomas-samoht marked this pull request as ready for review September 7, 2026 07:28
@thomas-samoht
thomas-samoht requested a review from a team as a code owner September 7, 2026 07:28
@thomas-samoht
thomas-samoht merged commit d8ae81a into main Sep 7, 2026
14 checks passed
@thomas-samoht
thomas-samoht deleted the feat/config-change-to-let-streams-to-stdout branch September 7, 2026 07:45
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