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

Truncate status in reporter #524

Merged
merged 5 commits into from
Sep 28, 2022
Merged

Truncate status in reporter #524

merged 5 commits into from
Sep 28, 2022

Conversation

kdorosh
Copy link
Contributor

@kdorosh kdorosh commented Sep 28, 2022

Truncate size of statuses written to k8s / etcd to protect against large keys stored in k8s backend.

@solo-changelog-bot
Copy link

Issues linked to changelog:
#523

@elcasteel
Copy link
Contributor

We should document that this overrides a value set in PROXY_STATUS_MAX_SIZE_BYTES We should also deprecate that in future releases.

@soloio-bulldozer soloio-bulldozer bot merged commit 3f99b24 into master Sep 28, 2022
@soloio-bulldozer soloio-bulldozer bot deleted the truncate_status branch September 28, 2022 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants