Merge pull request #10149 from hashicorp/dnephin/config-use-streaming-backend-defualt-true

config: default UseStreamingBackend to true
This commit is contained in:
Daniel Nephin 2021-04-30 16:29:11 -04:00 committed by GitHub
commit dcb5b924dc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 6 deletions

3
.changelog/10149.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:improvement
streaming: change `use_streaming_backend` to default to true so that streaming is used by default when it is supported.
```

View File

@ -1102,7 +1102,7 @@ func (b *builder) Build() (rt RuntimeConfig, err error) {
Watches: c.Watches,
}
rt.UseStreamingBackend = boolVal(c.UseStreamingBackend)
rt.UseStreamingBackend = boolValWithDefault(c.UseStreamingBackend, true)
if rt.Cache.EntryFetchMaxBurst <= 0 {
return RuntimeConfig{}, fmt.Errorf("cache.entry_fetch_max_burst must be strictly positive, was: %v", rt.Cache.EntryFetchMaxBurst)

View File

@ -2208,14 +2208,10 @@ bind_addr = "{{ GetPrivateInterfaces | include \"network\" \"10.0.0.0/8\" | attr
currently only supports numeric IDs.
- `mode` - The permission bits to set on the file.
- `use_streaming_backend` (experimental) when enabled Consul client agents will use
- `use_streaming_backend` defaults to true. When enabled Consul client agents will use
streaming rpc, instead of the traditional blocking queries, for endpoints which support
streaming. All servers must have [`rpc.enable_streaming`](#rpc_enable_streaming)
enabled before any client can enable `use_streaming_backend`.
At least one of [`dns_config.use_cache`](#dns_use_cache) or
[`http_config.use_cache`](#http_config_use_cache) must be enabled, otherwise
this setting has no effect.
`use_streaming_backend` will default to true in a future version of Consul.
- `verify_incoming` - If set to true, Consul
requires that all incoming connections make use of TLS and that the client