Skip to content

Adds test targets for QUICHE HTTP/2 libraries#20999

Merged
yanavlasov merged 9 commits intoenvoyproxy:mainfrom
birenroy:oghttp2-build-fixes-backup
Apr 26, 2022
Merged

Adds test targets for QUICHE HTTP/2 libraries#20999
yanavlasov merged 9 commits intoenvoyproxy:mainfrom
birenroy:oghttp2-build-fixes-backup

Conversation

@birenroy
Copy link
Copy Markdown
Contributor

This allows developers to run QUICHE HTTP/2 tests from an Envoy checkout.

$ bazel test --config=remote @com_github_google_quiche//...

Commit Message: Adds test targets for QUICHE HTTP/2 libraries
Additional Description:
Risk Level: low
Testing: Built and ran the test targets with bazel.
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

Signed-off-by: Biren Roy <birenroy@google.com>
Signed-off-by: Biren Roy <birenroy@google.com>
Signed-off-by: Biren Roy <birenroy@google.com>
Signed-off-by: Biren Roy <birenroy@google.com>
Signed-off-by: Biren Roy <birenroy@google.com>
…-backup

Signed-off-by: Biren Roy <birenroy@google.com>
Signed-off-by: Biren Roy <birenroy@google.com>
Signed-off-by: Biren Roy <birenroy@google.com>
Signed-off-by: Biren Roy <birenroy@google.com>
@birenroy
Copy link
Copy Markdown
Contributor Author

/assign @adisuissa

Copy link
Copy Markdown
Contributor

@adisuissa adisuissa 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.
/assign-from @envoyproxy/senior-maintainers

@repokitteh-read-only
Copy link
Copy Markdown

@envoyproxy/senior-maintainers assignee is @yanavlasov

🐱

Caused by: a #20999 (review) was submitted by @adisuissa.

see: more, trace.

@yanavlasov yanavlasov merged commit 96f052a into envoyproxy:main Apr 26, 2022
ravenblackx pushed a commit to ravenblackx/envoy that referenced this pull request Jun 8, 2022
Signed-off-by: Biren Roy <birenroy@google.com>
@birenroy birenroy deleted the oghttp2-build-fixes-backup branch November 16, 2022 19:20
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