Skip to content
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.

Sign up for GitHub

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

Jump to bottom

deps: use h2 with race condition fix #998

Merged
merged 1 commit into from
May 1, 2024
Merged

deps: use h2 with race condition fix #998

merged 1 commit into from
May 1, 2024

Conversation

howardjohn
Copy link
Member

Pulls in hyperium/h2#772. We might want to wait
on this, not sure how fast it will go

@howardjohn
deps: use h2 with race condition fix
82711eb 
Pulls in hyperium/h2#772. We might want to wait
on this, not sure how fast it will go
@howardjohn howardjohn requested a review from a team as a code owner May 1, 2024 23:11
@howardjohn howardjohn added release-notes-none Indicates a PR that does not require release notes. cherrypick/release-1.22 Set this label on a PR to auto-merge it to the release-1.22 branch labels May 1, 2024
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 1, 2024
@howardjohn howardjohn mentioned this pull request May 1, 2024
Merged
bleggett
bleggett approved these changes May 1, 2024
View reviewed changes
Copy link
Contributor

@bleggett bleggett left a comment •
edited
Loading

Choose a reason for hiding this comment

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

We'll get flakes until the fix is merged, and the fix is a single commit and a relatively small diff against upstream.

I'm ok with it

@bleggett bleggett mentioned this pull request May 1, 2024
Closed
@bleggett
Copy link
Contributor

bleggett commented May 1, 2024

xref with #1000

@istio-testing istio-testing merged commit 34c3a59 into istio:master May 1, 2024
3 checks passed
@istio-testing
Copy link
Contributor

In response to a cherrypick label: new pull request created: #1001

@istio-testing istio-testing mentioned this pull request May 1, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bleggett bleggett bleggett approved these changes

Assignees
No one assigned
Labels
cherrypick/release-1.22 Set this label on a PR to auto-merge it to the release-1.22 branch release-notes-none Indicates a PR that does not require release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@howardjohn @bleggett @istio-testing