luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

fix(peering): increase the gRPC limit to 8MB (#15503) 

* fix(peering): increase the gRPC limit to 50MB

* changelog

* update gRPC limit to 8MB
This commit is contained in:
cskh 2022-11-28 17:48:43 -05:00 committed by GitHub
parent c537bb51f9
commit 92e71318c1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/15503.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:bug
peering: fix the limit of replication gRPC message; set to 8MB
```

2
agent/consul/leader_peering.go
View File

@ -360,7 +360,7 @@ func (s *Server) establishStream(ctx context.Context,
// send keepalive pings even if there is no active streams // send keepalive pings even if there is no active streams
PermitWithoutStream: true, PermitWithoutStream: true,
}), }),
grpc.WithDefaultCallOptions(grpc.MaxCallSendMsgSize(50 * 1024 * 1024)), grpc.WithDefaultCallOptions(grpc.MaxCallSendMsgSize(8*1024*1024), grpc.MaxCallRecvMsgSize(8*1024*1024)),
} }
logger.Trace("dialing peer", "addr", addr) logger.Trace("dialing peer", "addr", addr)

4
agent/grpc-internal/client.go
View File

@ -148,7 +148,9 @@ func (c *ClientConnPool) dial(datacenter string, serverType string) (*grpc.Clien
grpc.WithKeepaliveParams(keepalive.ClientParameters{ grpc.WithKeepaliveParams(keepalive.ClientParameters{
Time: 30 * time.Second, Time: 30 * time.Second,
Timeout: 10 * time.Second, Timeout: 10 * time.Second,
})) }),
grpc.WithDefaultCallOptions(grpc.MaxCallSendMsgSize(8*1024*1024), grpc.MaxCallRecvMsgSize(8*1024*1024)),
)
if err != nil { if err != nil {
return nil, err return nil, err
} }

2
website/content/docs/connect/cluster-peering/index.mdx
View File

@ -46,7 +46,7 @@ Regardless of whether you connect your clusters through WAN federation or cluste
Consider the following technical constraints: Consider the following technical constraints:
- Services with node, instance, and check definitions totaling more than 50MB cannot be exported to a peer. - Services with node, instance, and check definitions totaling more than 8MB cannot be exported to a peer.
- Two admin partitions in the same datacenter cannot be peered. Use [`exported-services`](/docs/connect/config-entries/exported-services#exporting-services-to-peered-clusters) directly. - Two admin partitions in the same datacenter cannot be peered. Use [`exported-services`](/docs/connect/config-entries/exported-services#exporting-services-to-peered-clusters) directly.
- The `consul intention` CLI command is not supported. To manage intentions that specify services in peered clusters, use [configuration entries](/docs/connect/config-entries/service-intentions). - The `consul intention` CLI command is not supported. To manage intentions that specify services in peered clusters, use [configuration entries](/docs/connect/config-entries/service-intentions).
- Accessing key/value stores across peers is not supported. - Accessing key/value stores across peers is not supported.