-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
docs: misc doc debt #8678
Merged
Merged
docs: misc doc debt #8678
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,7 +3,7 @@ Cluster | |
|
||
.. toctree:: | ||
:glob: | ||
:maxdepth: 1 | ||
:maxdepth: 2 | ||
|
||
dynamic_forward_proxy/v2alpha/* | ||
redis/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,6 @@ Grpc Credentials | |
|
||
.. toctree:: | ||
:glob: | ||
:maxdepth: 1 | ||
:maxdepth: 2 | ||
|
||
v2alpha/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,6 @@ Health checkers | |
|
||
.. toctree:: | ||
:glob: | ||
:maxdepth: 1 | ||
:maxdepth: 2 | ||
|
||
*/v2/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,6 @@ Listener | |
|
||
.. toctree:: | ||
:glob: | ||
:maxdepth: 1 | ||
:maxdepth: 2 | ||
|
||
v2/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,6 @@ RBAC | |
|
||
.. toctree:: | ||
:glob: | ||
:maxdepth: 1 | ||
:maxdepth: 2 | ||
|
||
v2/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,6 +5,6 @@ Resource monitors | |
|
||
.. toctree:: | ||
:glob: | ||
:maxdepth: 1 | ||
:maxdepth: 2 | ||
|
||
*/v2alpha/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,6 @@ Transport sockets | |
|
||
.. toctree:: | ||
:glob: | ||
:maxdepth: 1 | ||
:maxdepth: 2 | ||
|
||
*/v2alpha/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
Why does Envoy use BoringSSL? | ||
============================= | ||
|
||
`BoringSSL <https://boringssl.googlesource.com/boringssl/>`_ is a slimmed down TLS implementation | ||
maintained by Google. Getting TLS right is very, very hard. Envoy has chosen to align with | ||
BoringSSL so as to obtain access to the world class experts that Google employs to work on this | ||
code base. In short: if BoringSSL is good enough for Google's production systems it is good enough | ||
for Envoy and the project will not offer first class support for any other TLS implementation. |
2 changes: 1 addition & 1 deletion
2
docs/root/faq/configure_flow_control.rst → docs/root/faq/configuration/flow_control.rst
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,96 @@ | ||
.. _faq_configuration_timeouts: | ||
|
||
How do I configure timeouts? | ||
============================ | ||
|
||
Envoy supports a wide range of timeouts that may need to be configured depending on the deployment. | ||
This page summarizes the most important timeouts used in various scenarios. | ||
|
||
.. attention:: | ||
|
||
This is not an exhaustive list of all of the configurable timeouts that Envoy supports. Depending | ||
on the deployment additional configuration may be required. | ||
|
||
HTTP/gRPC | ||
--------- | ||
|
||
Connection timeouts | ||
^^^^^^^^^^^^^^^^^^^ | ||
|
||
Connection timeouts apply to the entire HTTP connection and all streams the connection carries. | ||
|
||
* The HTTP protocol :ref:`idle timeout <envoy_api_field_core.HttpProtocolOptions.idle_timeout>` | ||
is defined in a generic message used by both the HTTP connection manager as well as upstream | ||
cluster HTTP connections. The idle timeout is the time at which a downstream or upstream | ||
connection will be terminated if there are no active streams. The default idle timeout if not | ||
otherwise specified is *1 hour*. To modify the idle timeout for downstream connections use the | ||
:ref:`common_http_protocol_options | ||
<envoy_api_field_config.filter.network.http_connection_manager.v2.HttpConnectionManager.common_http_protocol_options>` | ||
field in the HTTP connection manager configuration. To modify the idle timeout for upstream | ||
connections use the | ||
:ref:`common_http_protocol_options <envoy_api_field_Cluster.common_http_protocol_options>` field | ||
in the cluster configuration. | ||
|
||
Stream timeouts | ||
^^^^^^^^^^^^^^^ | ||
|
||
Stream timeouts apply to individual streams carried by an HTTP connection. Note that a stream is | ||
an HTTP/2 and HTTP/3 concept, however internally Envoy maps HTTP/1 requests to streams so in this | ||
context request/stream is interchangeable. | ||
|
||
* The HTTP connection manager :ref:`request_timeout | ||
<envoy_api_field_config.filter.network.http_connection_manager.v2.HttpConnectionManager.request_timeout>` | ||
mattklein123 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
is the amount of time the connection manager will allow for the *entire request stream* to be | ||
received by the client. | ||
|
||
.. attention:: | ||
|
||
This timeout is not enforced by default as it is not compatible with streaming requests | ||
(requests that never end). See the stream idle timeout that follows. However, if using the | ||
:ref:`buffer filter <config_http_filters_buffer>`, it is recommended to configure this timeout. | ||
* The HTTP connection manager :ref:`stream_idle_timeout | ||
<envoy_api_field_config.filter.network.http_connection_manager.v2.HttpConnectionManager.stream_idle_timeout>` | ||
is the amount of time that the connection manager will allow a stream to exist with no upstream | ||
or downstream activity. The default stream idle timeout is *5 minutes*. This timeout is strongly | ||
recommended for streaming APIs (requests or responses that never end). | ||
|
||
Route timeouts | ||
^^^^^^^^^^^^^^ | ||
|
||
Envoy supports additional stream timeouts at the route level, as well as overriding some of the | ||
stream timeouts already introduced above. | ||
|
||
* A route :ref:`timeout <envoy_api_field_route.RouteAction.timeout>` is the amount of time that | ||
mattklein123 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
Envoy will wait for the upstream to respond with a complete response. *This timeout does not | ||
start until the entire downstream request stream has been received*. | ||
|
||
.. attention:: | ||
|
||
This timeout defaults to *15 seconds*, however, it is not compatible with streaming responses | ||
(responses that never end), and will need to be disabled. Stream idle timeouts should be used | ||
in the case of streaming APIs as described elsewhere on this page. | ||
* The route :ref:`idle_timeout <envoy_api_field_route.RouteAction.idle_timeout>` allows overriding | ||
of the HTTP connection manager :ref:`stream_idle_timeout | ||
<envoy_api_field_config.filter.network.http_connection_manager.v2.HttpConnectionManager.stream_idle_timeout>` | ||
and does the same thing. | ||
* The route :ref:`per_try_timeout <envoy_api_field_route.RetryPolicy.per_try_timeout>` can be | ||
configured when using retries so that individual tries using a shorter timeout than the overall | ||
request timeout described above. This type of timeout will not work with streaming APIs (in which | ||
retries are typically not possible) but is useful for decreasing the tail latency of non-streaming | ||
APIs. | ||
|
||
TCP | ||
--- | ||
|
||
* The cluster :ref:`connect_timeout <envoy_api_field_Cluster.connect_timeout>` specifies the amount | ||
of time Envoy will wait for an upstream TCP connection to be established. This timeout has no | ||
default, but is required in the configuration. | ||
|
||
.. attention:: | ||
|
||
For TLS connections, the connect timeout includes the TLS handshake. | ||
* The TCP proxy :ref:`idle_timeout | ||
<envoy_api_field_config.filter.network.tcp_proxy.v2.TcpProxy.idle_timeout>` | ||
is the amount of time that the TCP proxy will allow a connection to exist with no upstream | ||
or downstream activity. This timeout currently has no default value but configuring it is | ||
strongly recommended. |
4 changes: 2 additions & 2 deletions
4
docs/root/faq/zipkin_tracing.rst → ...root/faq/configuration/zipkin_tracing.rst
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
.. _common_configuration_zipkin_tracing: | ||
|
||
How do I setup Zipkin tracing? | ||
============================== | ||
How do I configure Zipkin tracing? | ||
================================== | ||
|
||
Refer to the :ref:`zipkin sandbox setup <install_sandboxes_zipkin_tracing>` | ||
for an example of zipkin tracing configuration. |
4 changes: 2 additions & 2 deletions
4
docs/root/faq/zone_aware_routing.rst → .../faq/configuration/zone_aware_routing.rst
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to switch to using protodoc for service documentation? These seem more like pseudo-REST than gRPC..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is tracked in #3091. I agree this should all be auto generated, but I was just trying to get us to parity for now.