-
Notifications
You must be signed in to change notification settings - Fork 469
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
control plane robustness #10625
Comments
see also: |
Possibly helpful information for implementer. KRT has |
In addition to the issues outlined in the description, I think there's several areas in the translator that could use better error handling and/or require proper error handling instead of swallowing potential errors at lower levels. |
draft notes:
either block envoy updates on errors, or drop the bad route; either way propagate and log the errors
route replacement / not setting snapshot in the cache if we have errors
The text was updated successfully, but these errors were encountered: