Skip to content

Update envoy with upstream#5199

Merged
istio-testing merged 4 commits intoistio:masterfrom
zirain:update-envoy
Dec 11, 2023
Merged

Update envoy with upstream#5199
istio-testing merged 4 commits intoistio:masterfrom
zirain:update-envoy

Conversation

@zirain
Copy link
Copy Markdown
Member

@zirain zirain commented Dec 11, 2023

fixes: #5197

@zirain zirain requested a review from a team December 11, 2023 01:33
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 11, 2023
@zirain
Copy link
Copy Markdown
Member Author

zirain commented Dec 11, 2023

related to envoyproxy/envoy#30827

@zirain
Copy link
Copy Markdown
Member Author

zirain commented Dec 11, 2023

envoyproxy/envoy#31189

@istio-testing istio-testing added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 11, 2023
Comment thread extensions/common/util.cc
appendString(result, DNS_RESOLUTION_FAILURE);
}

if (response_flag & DropOverLoad) {
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.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

SG, thanks for updating.

@istio-testing istio-testing merged commit a165ba3 into istio:master Dec 11, 2023
@zirain zirain deleted the update-envoy branch December 11, 2023 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants