Skip to content

http/2 graceful drain support#103

Merged
mattklein123 merged 4 commits intomasterfrom
graceful_drain
Sep 28, 2016
Merged

http/2 graceful drain support#103
mattklein123 merged 4 commits intomasterfrom
graceful_drain

Conversation

@mattklein123
Copy link
Copy Markdown
Member

The connection manager goes into a drain sequence where we issue a shutdown
notice, wait 5s, then do a full shutdown.

The connection manager goes into a drain sequence where we issue a shutdown
notice, wait 5s, then do a full shutdown.
@mattklein123
Copy link
Copy Markdown
Member Author

@lyft/network-team

set, there is no idle timeout. When the idle timeout is reached the connection will be closed. If
the connection is an HTTP/2 connection a GOAWAY frame will be sent prior to closing
the connection.
the connection is an HTTP/2 connection a drain sequence will occur prior to closing the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: tag as config_http_conn_man_drain_timeout_ms

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@mattklein123 mattklein123 merged commit 898b187 into master Sep 28, 2016
@mattklein123 mattklein123 deleted the graceful_drain branch September 28, 2016 19:32
rshriram pushed a commit to rshriram/envoy that referenced this pull request Oct 30, 2018
yxue referenced this pull request in yxue/envoy Nov 20, 2019
Signed-off-by: John Plevyak <jplevyak@gmail.com>
shalk pushed a commit to shalk/envoy that referenced this pull request Nov 6, 2020
jwendell pushed a commit to jwendell/envoy that referenced this pull request Mar 30, 2022
* Add LuaJIT patch for s390x

* Move proxy luajit build flags patch to envoy
arminabf pushed a commit to arminabf/envoy that referenced this pull request Jun 5, 2024
mathetake added a commit that referenced this pull request Mar 3, 2026
This adds the RequestCost field to AIGatewayRoute,
which will allows users to do the rate limiting etc
based on the calculated "token usage". 

This is based on the new feature introduced in 
* envoyproxy/gateway#4957
* envoyproxy/gateway#5035

and because of the feature, the only thing we have to do
from AI Gateway side is to set a dynamic metadata as per 
the comment in the API.

---------

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
This commit implements the CEL expression API
left as a TODO in #103. Closes #97.

---------

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
Co-authored-by: Dan Sun <dsun20@bloomberg.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants