Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

optionally print alerts/messages/telemetry to stdout #23

Open
nyrahul opened this issue Feb 1, 2023 · 2 comments
Open

optionally print alerts/messages/telemetry to stdout #23

nyrahul opened this issue Feb 1, 2023 · 2 comments
Assignees
@lakshya8066
Labels
good first issue Good for newcomers

Comments

@nyrahul
Copy link
Contributor

nyrahul commented Feb 1, 2023

Currently, the alerts, logs are mandatorily printed to the stdout. This can induce an overhead in some deployments since the telemetry events could be substantially high. It would be better to enable the print to stdout only on the basis of an env var (PRINT_TO_STDOUT=1).
@nyrahul nyrahul added the good first issue Good for newcomers label Feb 1, 2023
@lakshya8066
Copy link

Hi @nyrahul I would like to work on this issue

@harisudarsan1 harisudarsan1 mentioned this issue Jan 28, 2024
Closed
@harisudarsan1
Copy link

@nyrahul I have linked a pr regarding this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lakshya8066 lakshya8066

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Introduce capability for conditional output to standard output (stdout)
3 participants
@nyrahul @lakshya8066 @harisudarsan1