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.

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

fix: init gostats to not flood log #520

Merged
merged 1 commit into from
Mar 27, 2024
Merged

Conversation

guilhem
Copy link
Contributor

@guilhem guilhem commented Feb 25, 2024

Default gostats store flood messages on stderr except if statd or if GOSTATS_LOGGING_SINK_DISABLED is set to "true". This commit init store in a unambiguous way.

Default gostats store flood messages on stderr except if statd or
if GOSTATS_LOGGING_SINK_DISABLED is set to "true".
This commit init store in a unambiguous way.

Signed-off-by: Guilhem Lettron <[email protected]>
Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale label Mar 26, 2024
@guilhem
Copy link
Contributor Author

guilhem commented Mar 27, 2024

ping 🤷‍♂️

@mattklein123 mattklein123 merged commit 78ed1dc into envoyproxy:main Mar 27, 2024
1 check passed
guilhem added a commit to guilhem/ratelimit that referenced this pull request Mar 28, 2024
guilhem added a commit to guilhem/ratelimit that referenced this pull request Mar 28, 2024
guilhem added a commit to guilhem/ratelimit that referenced this pull request Mar 28, 2024
fix envoyproxy#520 trailing line

Signed-off-by: Guilhem Lettron <[email protected]>
ysawa0 pushed a commit that referenced this pull request Mar 29, 2024
fix #520 trailing line

Signed-off-by: Guilhem Lettron <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants