Skip to content

http/2: do not keep closed streams for priority calculations#513

Merged
mattklein123 merged 2 commits intomasterfrom
http2_no_close_streams
Feb 27, 2017
Merged

http/2: do not keep closed streams for priority calculations#513
mattklein123 merged 2 commits intomasterfrom
http2_no_close_streams

Conversation

@mattklein123
Copy link
Copy Markdown
Member

Saves a ton of memory. Requires nghttp2 1.20.0.

fixes #471

Saves a ton of memory. Requires nghttp2 1.20.0.

fixes #471
@mattklein123
Copy link
Copy Markdown
Member Author

@lyft/network-team

cc @tatsuhiro-t

@wgallagher
Copy link
Copy Markdown

👍

@mattklein123 mattklein123 merged commit 62d75c6 into master Feb 27, 2017
@mattklein123 mattklein123 deleted the http2_no_close_streams branch February 27, 2017 17:39
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Signed-off-by: Alan Chiu <achiu@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Signed-off-by: Alan Chiu <achiu@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
mathetake pushed a commit that referenced this pull request Mar 3, 2026
**Commit Message**
Separate envoy ai gateway crds to its own chart so helm upgrade can
update the crd when there are changes.

**Related Issues/PRs (if applicable)**
Fixes #513

---------

Signed-off-by: Dan Sun <dsun20@bloomberg.net>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Commit Message**

This removes the redundant upload in release.yaml which resulted in
uploading the same artifacts twice.
The CI failed in
https://github.com/envoyproxy/ai-gateway/actions/runs/15404045295/job/43343173369
while we can see the artifacts already uploaded correctly at `gh release
create`.

**Related Issues/PRs (if applicable)**

Follow up on #513

---------

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
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.

perf: nghttp2 memory use

2 participants