-
Notifications
You must be signed in to change notification settings - Fork 38.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
Increase maxMsgSize for dockershim #77165
Increase maxMsgSize for dockershim #77165
Conversation
Increase the grpc max message size to be the same as the value defined in `pkg/kubelet/remote/utils.go`. Increase the limit because, `ListPodSandbox` (and possibly other) calls are hitting the limit. Long term, the best solution to this issue is to use pagination, but that is not currently available.
/approve @Random-Liu thanks :) |
/test pull-kubernetes-integration |
/assign @Random-Liu @derekwaynecarr /lgtm |
Thanks for the change. Agree it should have been kept in sync with remote CRI. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: derekwaynecarr, mattjmcnaughton The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…-#77165-upstream-release-1.14 Automated cherry pick of #77165: Increase maxMsgSize for dockershim
What type of PR is this?
/kind bug
What this PR does / why we need it:
Increase the grpc max message size to be the same as the value defined
in
pkg/kubelet/remote/utils.go
.Increase the limit because,
ListPodSandbox
(and possibly other) callsare hitting the limit. Long term, the best solution to this issue is to
use pagination, but that is not currently available.
Which issue(s) this PR fixes:
Fixes #77138
Special notes for your reviewer:
cc @dims
Does this PR introduce a user-facing change?: