Skip to content

build: exclude QUIC libraries when HTTP3 is disabled#42573

Merged
jwendell merged 1 commit intoenvoyproxy:mainfrom
jwendell:quiche4
Dec 11, 2025
Merged

build: exclude QUIC libraries when HTTP3 is disabled#42573
jwendell merged 1 commit intoenvoyproxy:mainfrom
jwendell:quiche4

Conversation

@jwendell
Copy link
Copy Markdown
Member

Follows up #41886

Also remove --build_tests_only from the CI test so that everything is built and errors like this are caught.

Follows up envoyproxy#41886

Also remove `--build_tests_only` from the CI test so that everything is
built and errors like this are caught.

Signed-off-by: Jonh Wendell <jwendell@redhat.com>
Comment thread ci/do_ci.sh
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @jwendell

@phlax
Copy link
Copy Markdown
Member

phlax commented Dec 11, 2025

/retest

Copy link
Copy Markdown
Member

@agrawroh agrawroh left a comment

Choose a reason for hiding this comment

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

Looks Good, Thanks!

@jwendell jwendell merged commit ccc4ccf into envoyproxy:main Dec 11, 2025
24 of 26 checks passed
MayorFaj pushed a commit to MayorFaj/envoy that referenced this pull request Dec 15, 2025
Follows up envoyproxy#41886

Also remove `--build_tests_only` from the CI test so that everything is
built and errors like this are caught.

Signed-off-by: Jonh Wendell <jwendell@redhat.com>
Signed-off-by: MayorFaj <mayorfaj@gmail.com>
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
Follows up envoyproxy#41886

Also remove `--build_tests_only` from the CI test so that everything is
built and errors like this are caught.

Signed-off-by: Jonh Wendell <jwendell@redhat.com>
Signed-off-by: Gustavo <grnmeira@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.

3 participants