-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
WIP: Mitigate overloading stackdriver api until quota increased #639
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Assign the PR to them by writing The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@chxchx: The following tests failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
…siness. (istio#639) * Update the symbol names for all ConfigErrors to adopt camelCase consistently. * Only log from the dedup repear at a higher logging level to avoid noisiness. Former-commit-id: eb66b0c3d8826cec935ce5a6eec709abdbcdd7c5
…siness. (#639) * Update the symbol names for all ConfigErrors to adopt camelCase consistently. * Only log from the dedup repear at a higher logging level to avoid noisiness. Former-commit-id: 8f5bdba213890dc650b62494c08cbc089f5a317e
* Fixes for istiod injection Right now there is no way to remove the sidecar injector deployment when using istiod. This means you will always have a bunch of config that is unused. Instead, when istiod is enabled, remove the injector stuff. Alternative approach would be to add an explicit flag for this, like istiod.disableinjector or something, but I am not sure that is needed at this time. Additionally, add back features from the old injection to avoid regressions. These are critical features, not just legacy junk. Finally, reuse the old name for the configmap so existing tooling like istioctl can continue to function correctly. * Fix tests
* Add unit test and fix for ICP.DeepCopy() DeepCopy() always failed with an error, which was only logged and wasn't handled properly. The error occured when unmarshalling CreatedTimestamp, which meant that all remaining fields were never unmarshalled. * Add copyright header * Update go.mod * Add additional test and fix import formatting
We are trying to increase the stackdriver api quota limit. This is plan b if that does pass.
Release note: