AJ Heller
4a133abc37
Fix iomgr_timer target visibility ( #29702 )
3 years ago
AJ Heller
250b8d2dae
IomgrEventEngine Redux ( #29693 )
...
* Revert "Revert "IomgrEventEngine (#29616 )" (#29692 )"
This reverts commit 246d13e392
.
* temporarily disable EE usage to coordinate landing
* spelling
3 years ago
AJ Heller
246d13e392
Revert "IomgrEventEngine ( #29616 )" ( #29692 )
...
This reverts commit 7f09b98201
.
3 years ago
AJ Heller
7f09b98201
IomgrEventEngine ( #29616 )
3 years ago
Anirudh Ramachandra
063c36cb46
Revert "Revert "Move TCP Connect into its own handshaker. ( #29111 )" (#… ( #29626 )
...
Rollforward with TCP connect handshaker again(#29111 ) after fixing broken internal targets.
The changes needed were just visibility changes to the handshaker and the http_connect_handshaker libraries as they are used internally.
3 years ago
AJ Heller
adfd009d3a
Make stack_tracer more broadly usable ( #29660 )
...
These changes reduce the stack_tracer's dependencies, allowing it to be
added to other targets such as `gpr_base` without dependency cycles.
Why would I want to do this? printf debugging lock problems that only
occur on the CI's Mac VMs, which we cannot access.
3 years ago
Zhan Jiang
67cfc62690
Refactor tsi_ssl_credentials target. ( #29443 )
...
* Refactor tsi_ssl_credentials target.
3 years ago
Craig Tiller
7798e3b741
[slice] Introduce grpc_core::SliceBuffer ( #29635 )
...
* A starter SliceBuffer implementation
* Add comments and fix sanity checks
* Minor fixes
* more minor fixes
* Addressing review comments and adding a slice_buffer_test
* fix sanity checks
* regenerate projects
* fixing undefined function error
* reverting changes from memory_allocator.cc and adding them to src/core/lib/slice/slice_buffer.cc to allow memory allocator lib to build correctly
* fix sanity checks
* adding an open source slice definition
* regnerate projects
* fix asan error
* Automated change: Fix sanity tests
* addressing review comments
* fix sanity checks
* regenerate projects
* update
* fix sanity checks
* Converting slice buffer to retarin ownership of the underlying ctype
* [slice_buffer] Introduce grpc_core::SliceBuffer
* add test
* Automated change: Fix sanity tests
* missing files
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* sanity
Co-authored-by: Vignesh Babu <vigneshbabu@google.com>
Co-authored-by: Vignesh2208 <Vignesh2208@users.noreply.github.com>
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
da9f98a8bd
[iwyu] event_engine ( #29641 )
...
* slartibartfast
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Vignesh Babu
1494b1ef26
A starter slice buffer implementation to unblock event engine endpoints ( #29367 )
...
* A starter SliceBuffer implementation
* Add comments and fix sanity checks
* Minor fixes
* more minor fixes
* Addressing review comments and adding a slice_buffer_test
* fix sanity checks
* regenerate projects
* fixing undefined function error
* reverting changes from memory_allocator.cc and adding them to src/core/lib/slice/slice_buffer.cc to allow memory allocator lib to build correctly
* fix sanity checks
* adding an open source slice definition
* regnerate projects
* fix asan error
* Automated change: Fix sanity tests
* addressing review comments
* fix sanity checks
* regenerate projects
* update
* fix sanity checks
* Converting slice buffer to retarin ownership of the underlying ctype
* fix nits
Co-authored-by: Vignesh2208 <Vignesh2208@users.noreply.github.com>
3 years ago
AJ Heller
9f7311e399
Revert DNSResolver cancellation ( #29581 and #29631 ) ( #29634 )
...
This reverts commit 075e84314d
.
3 years ago
AJ Heller
0a21ecdff6
Change cancellation semantics in the iomgr DNSResolver ( #29581 )
...
DNS requests were previously cancellable, but it was assumed
that the resolution callback would be called in all cases. Now, requests
provide a `bool Cancel(TaskHandle handle)` method that lets callers know
whether cancellation is successful, and if so, the callback will not be
run.
This is in accordance with EventEngine's cancellation semantics, and it
is a step towards migration from iomgr to EventEngine.
3 years ago
Craig Tiller
f56282a6b6
[iwyu] surface ( #29588 )
...
* [iwyu] chttp2
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fwd file
* fix
* fix
* Automated change: Fix sanity tests
* fix
* [iwyu] surface
* [iwyu] surface
* add no-return hint
* fix
* Automated change: Fix sanity tests
* Add flag
* sweep1
* fixes
* speedup
* fixes
* fixes
* fixes
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* fix
* fic
* x
* ugh
* fix
* fix
* Automated change: Fix sanity tests
* fix?
* fixes
* fix
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
457adf3d58
[iwyu] client_channel ( #29579 )
...
* [iwyu] client_channel
* Update grpclb_client_stats.h
* Update load_balancer_api.h
* Update grpc_ares_ev_driver.h
* Update grpc_ares_wrapper.h
* Update grpc_ares_wrapper_event_engine.cc
* Update grpc_ares_wrapper_windows.cc
* clang-format
* Update grpc_ares_ev_driver_event_engine.cc
* Update grpc_ares_ev_driver_windows.cc
* [iwyu] chttp2
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fwd file
* fix
* fix
* Automated change: Fix sanity tests
* fix
* fix
* Automated change: Fix sanity tests
* Update grpc_ares_ev_driver.h
* Update grpc_ares_wrapper.h
* re-run
* fix
* fix
* fix
* fix
* fix
* fix
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
251e1458c7
Fix for visibility exposed by import ( #29614 )
3 years ago
Craig Tiller
5918f98ecb
[iwyu] Enable nosafe_headers ( #29591 )
...
* Add flag
* sweep1
* fixes
* speedup
* fixes
* fix
* fix
* fix
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* fix
* fix
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
c443e33cc6
[iwyu] chttp2 ( #29580 )
...
* [iwyu] chttp2
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fwd file
* fix
* fix
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
92609abed0
[promises] Expand filter fuzzer to contain a channel stack. ( #29513 )
...
* sketch
* Eliminate post-init in channel stack builder
We've had a post init function on channel stack builder for a very long
time, an it serves to run some code after initialization completes.
We need the functionality for a few things, but the function passed in
is intimately tied to the filter in use - we never vary it between
multiple functions for the same filter... which means it makes more
sense to locate this functionality as part of the filter interface.
* fix
* Automated change: Fix sanity tests
* introduce-channel-stack
* introduce-channel-stack
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fixes
* Fix
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* review feedback
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
1475ed4fb7
[iwyu] json ( #29536 )
...
* [iwyu] json
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
ed5e26e3ae
[iwyu] channel ( #29573 )
...
* [iwyu] channel
* Automated change: Fix sanity tests
* fix
* fix up script
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
f7f0dcf2cb
[build] Move unnecessary absl includes from metadata_batch.h ( #29523 )
...
* Move unnecessary absl includes from metadata_batch.h
metadata_batch.h is a widely used header, and so any includes it takes
that are for its internal use can negatively impact our build times.
* missing file
* build
* review feedback
3 years ago
Craig Tiller
be1a2ee500
[iwyu] resource quota ( #29538 )
...
* [iwyu] resource quota
* fix
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
AJ Heller
815029fe2b
Revert "Move TCP Connect into its own handshaker. ( #29111 )" ( #29558 )
...
This reverts commit 0675af8511
.
3 years ago
Anirudh Ramachandra
0675af8511
Move TCP Connect into its own handshaker. ( #29111 )
...
Currently the tcp connect is performed in chttp2_connector before the handshaking is triggered. For
use cases where the application wants to perform business logic before
the tcp connection, this is problematic. By moving the TCP connect into
its own handshaker and registering it by default at the beginning, this
allows applications to add a new handshaker at the beginning allowing
handshaker logic before a TCP connect.
This approach has the advantage of slightly simplifying the logic in
tcp_connect_handshaker and httpcli as tcp_connect/callback can be
removed.
As the TCP connect needs parameters like resolved_addr,
interested_parties, a new struct called connection args is created as a
member for Handshaker Args.
For server handshakers most of the arguments here are not directly
useful, other than the deadline.
3 years ago
Craig Tiller
4b5441382d
[iwyu] uri ( #29547 )
...
* [iwyu] uri
* fix
3 years ago
Craig Tiller
e7c79e2f7a
[build] Move PID controller to its own build target ( #29522 )
...
* Move PID controller to its own build target
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Craig Tiller
6b00bc7516
[build] Remove str_format.h from server_address.h ( #29524 )
...
This file gets included by resolver, which gets included by
core_configuration, which in turn sprays this inclusion everywhere (and
it's a costly file to parse)
3 years ago
Jan Tattermusch
e8404698d5
Remove C# implementation (individual packages will continue to be maintained through v2.46.x patches or moved to grpc-dotnet) ( #29225 )
...
* remove examples/csharp
* remove reference to examples/csharp
* remove C# references from doc/
* remove src/csharp
* update test_creds locations
* remove C# distribtests
* remove buildgen templates for files under src/csharp
* remove grpc_csharp_ext C build target
* regenerate projects
* update tools/run_tests/README.md
* change many C# test/build scripts to noop
* unbreak run_tests.py -l csharp
* unbreak C# build_package targets
* unbreak C# distribtests
* remove C# pre-build steps from prepare_build_windows.bat
* remove src/csharp from clang_format
* remove csharp and csharpcoreclr from run_interop_tests.py
* remove C# interop test docker images
* Revert "remove src/csharp"
This reverts commit 97f2b6d707
.
* remove src/csharp, but keep Grpc.Tools
* add back templates/src/csharp/build/dependencies.props.template
* remove everything but Grpc.Tools and Grpc.Tools.Tests from sln
* Revert "unbreak run_tests.py -l csharp"
This reverts commit 529b5e2ec1
.
* revive Grpc.Tools.Tests through run_tests.py
* Revert "unbreak C# build_package targets"
This reverts commit 79d8d83c8b
.
* experiment with build_nuget.sh fixes
* simplify create_fake_native_artifacts.sh
* building of Grpc.Tools in package_targets.py
* move shared files from Grpc.Core to Grpc.Tools
* yapf format: distribtest_targets.py
* Revert "remove C# pre-build steps from prepare_build_windows.bat"
This reverts commit bd38956a2b
.
* update C# README.md
* disable C# in gke OSS benchmarks
3 years ago
Craig Tiller
8bb45aa3a6
Convert grpc_channel to C++ ( #29266 )
...
* begin
* tests
* fix
* http
* Filter fuzzer
* progress
* basics
* progress
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Revert "Revert "HTTP Client Filter --> promises (#29031 )" (#29181 )"
This reverts commit 6ee276f672
.
* stuff
* debug
* minimal reproduction
* progress
* progress
* create call
* progress
* recv trailing metadata
* wakeups
* corpus
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* debug
* fix state machine for c#
* Revert "minimal reproduction"
This reverts commit 4d02d2e730
.
* Revert "debug"
This reverts commit 7960842f48
.
* Revert "debug"
This reverts commit a6f224e4a1
.
* no-logging
* initial-metadata
* Revert "Revert "debug""
This reverts commit 951844e857
.
* Better int conversion
* debug
* Fix for Cronet
* Revert "debug"
This reverts commit 4d641c4281
.
* Revert "Better int conversion"
This reverts commit 4001b957cb
.
* Revert "Revert "Revert "debug"""
This reverts commit d135c61043
.
* progress
* progress
* Automated change: Fix sanity tests
* fix, c++ize
* handle transport, use objects
* enable more stuffs
* remove placeholder
* contexts
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* x
* exceptional toast
* include idle filters, time
* fix
* namespace
* fixes
* final info
* progress
* cleanup
* progress
* progress
* progress
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* progress
* Set int
* Set int
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fixes
* fixes
* fixes
* fixes
* fix
* fix race
* fix race
* mac fix
* review feedback
* getgetget
* fix ios
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
Co-authored-by: Jan Tattermusch <jtattermusch@google.com>
3 years ago
Mark D. Roth
be53d2ce3f
LB policy API: add OOB backend metric API ( #29012 )
...
* WIP: add OOB backend metric API for LB policies
* fix some includes
* minor fixes
* picking this up again...
* more WIP
* health checking: cancel stream if response message fails to parse
* basic structure in place, but still have synchronization issues to address
* ORCA: implement ORCA RPC service for OOB backend metric reporting
* fix unused parameter error
* gen_upb_api
* add missing build deps
* increase test timing fudge factor
* add missing copyright header
* fix build and locking problems
* clang-format
* document API
* buildifier
* add test, but doesn't build yet
* new test working, but broke existing test, and need to fix server API
* don't register as a generic service
* update test for new orca service registration API
* fix build
* sanitize
* report interval defaults to min interval
* add channel trace event on UNIMPLEMENTED
* don't regenerate the response proto unless something changed
* add missing build dep
* fix comment
3 years ago
Craig Tiller
9f65dfed5b
Roll forward fault injection --> promises ( #29452 )
...
* Revert "Revert "Fault injection filter -> promises (#29288 )" (#29395 )"
This reverts commit 14cdbc68af
.
* fix refcounting
3 years ago
Yash Tibrewal
eb96f90eb7
HTTP: Use channel arg instead of global to allow PUT requests ( #29411 )
...
* HTTP: Use channel arg instead of global to allow PUT requests
* Reviewer comments
* Reviewer comments
* clang-format
* Repo manager - AJ
* Reviewer comments
* unused parameter
3 years ago
Anirudh Ramachandra
0922756050
Fix grpc_sockaddr_to_uri to handle embedded nulls for unix_abstract resolved addresses ( #29323 )
...
Currently when an embedded null is present, it is left as is. This causes an issue when grpc_sockaddr_to_uri is followed by any c style operations like copy, as the string is truncated at the non-encoded null character. For example this is triggered when channel args containing a string channel arg is copied.
To prevent this properly encode the URI with %00.
3 years ago
Yousuk Seung
371d231b53
Support unix socket in grpc_sockaddr_to_string ( #29169 )
...
* Support unix socket in grpc_sockaddr_to_string
* make it return statusor
* clang fix
* made grpc_sockaddr_to_string() to return statusor
* Let Chttp2ServerListener::Start crash
* test failure fixed
* api_fuzzer fixed
* comments addressed.
* more comments addressed
* comments addressed
* fix other broken builds
3 years ago
Lidi Zheng
98fc0260e3
Bump dev version to v1.47.0-dev ( #29414 )
...
* bump version to 1.47.0-dev
* regenerate projects
3 years ago
Mark D. Roth
1a983ed013
Duration: add method for generating JSON form of google.protobuf.Duration ( #29401 )
3 years ago
Lidi Zheng
14cdbc68af
Revert "Fault injection filter -> promises ( #29288 )" ( #29395 )
...
This reverts commit f3897a5f7a
.
3 years ago
Lidi Zheng
9f5735b604
Bump core version 202204081538 ( #29362 )
...
* bump C-core version
* regenerate projects
3 years ago
Mark D. Roth
60c56f7d01
Second attempt: implement ORCA RPC service for OOB backend metric reporting ( #29352 )
...
* Revert "Revert "ORCA: implement ORCA RPC service for OOB backend metric reporting (#29215 )" (#29351 )"
This reverts commit 71b355624f
.
* move ORCA service to its own BUILD rule
3 years ago
Mark D. Roth
71b355624f
Revert "ORCA: implement ORCA RPC service for OOB backend metric reporting ( #29215 )" ( #29351 )
...
This reverts commit 6d6380de58
.
3 years ago
Mark D. Roth
6d6380de58
ORCA: implement ORCA RPC service for OOB backend metric reporting ( #29215 )
...
* ORCA: implement ORCA RPC service for OOB backend metric reporting
* fix unused parameter error
* gen_upb_api
* add missing build deps
* increase test timing fudge factor
* add missing copyright header
* buildifier
* don't register as a generic service
* report interval defaults to min interval
* don't regenerate the response proto unless something changed
* use INTERNAL for proto parsing failure
* use absl::Duration in public API
3 years ago
Alisha Nanda
869ed910d5
Add resource quota reclamation to tcp endpoint ( #29292 )
...
* Add resource quota to tcp
* Automated change: Fix sanity tests
* Update style per comment
Co-authored-by: ananda1066 <ananda1066@users.noreply.github.com>
3 years ago
Craig Tiller
f3897a5f7a
Fault injection filter -> promises ( #29288 )
...
* Fault injection filter -> promises
3 years ago
Craig Tiller
33aaa55032
Filter fuzzer ( #29177 )
...
* begin
* tests
* fix
* http
* Filter fuzzer
* progress
* basics
* progress
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Revert "Revert "HTTP Client Filter --> promises (#29031 )" (#29181 )"
This reverts commit 6ee276f672
.
* stuff
* debug
* minimal reproduction
* progress
* progress
* create call
* progress
* recv trailing metadata
* wakeups
* corpus
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* debug
* fix state machine for c#
* Revert "minimal reproduction"
This reverts commit 4d02d2e730
.
* Revert "debug"
This reverts commit 7960842f48
.
* Revert "debug"
This reverts commit a6f224e4a1
.
* no-logging
* initial-metadata
* Revert "Revert "debug""
This reverts commit 951844e857
.
* Better int conversion
* debug
* Fix for Cronet
* Revert "debug"
This reverts commit 4d641c4281
.
* Revert "Better int conversion"
This reverts commit 4001b957cb
.
* Revert "Revert "Revert "debug"""
This reverts commit d135c61043
.
* progress
* progress
* Automated change: Fix sanity tests
* fix, c++ize
* handle transport, use objects
* enable more stuffs
* remove placeholder
* contexts
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* x
* exceptional toast
* include idle filters, time
* fix
* namespace
* fixes
* final info
* progress
* Automated change: Fix sanity tests
* fix
* fix
* combine cases
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
Co-authored-by: Jan Tattermusch <jtattermusch@google.com>
3 years ago
Craig Tiller
ee84fb9beb
Convert server config selector filter ( #28877 )
...
* server_config_selector_filter
* fix
* progress
* update
* x
* compiles!
* fixed!
* fix
* fix
3 years ago
Craig Tiller
14c078bcdc
Revert "Revert "Convert promise based filter channel args to new C++ type ( #29165 )" ( #29267 )" ( #29268 )
...
This reverts commit ee2b75e337
.
3 years ago
AJ Heller
ee2b75e337
Revert "Convert promise based filter channel args to new C++ type ( #29165 )" ( #29267 )
...
This reverts commit 6a13c26cef
.
3 years ago
Craig Tiller
6a13c26cef
Convert promise based filter channel args to new C++ type ( #29165 )
...
* begin
* tests
* fix
* http
* Revert "Revert "HTTP Client Filter --> promises (#29031 )" (#29181 )"
This reverts commit 6ee276f672
.
* debug
* minimal reproduction
* debug
* fix state machine for c#
* Revert "minimal reproduction"
This reverts commit 4d02d2e730
.
* Revert "debug"
This reverts commit 7960842f48
.
* Revert "debug"
This reverts commit a6f224e4a1
.
* no-logging
* Revert "Revert "debug""
This reverts commit 951844e857
.
* Better int conversion
* debug
* Fix for Cronet
* Revert "debug"
This reverts commit 4d641c4281
.
* Revert "Better int conversion"
This reverts commit 4001b957cb
.
* Revert "Revert "Revert "debug"""
This reverts commit d135c61043
.
* fix, c++ize
* x
Co-authored-by: Jan Tattermusch <jtattermusch@google.com>
3 years ago
Craig Tiller
e122c64000
Revert "Revert "HTTP Client Filter --> promises ( #29031 )" ( #29181 )" ( #29182 )
...
* Revert "Revert "HTTP Client Filter --> promises (#29031 )" (#29181 )"
This reverts commit 6ee276f672
.
* debug
* minimal reproduction
* debug
* fix state machine for c#
* Revert "minimal reproduction"
This reverts commit 4d02d2e730
.
* Revert "debug"
This reverts commit 7960842f48
.
* Revert "debug"
This reverts commit a6f224e4a1
.
* no-logging
* Revert "Revert "debug""
This reverts commit 951844e857
.
* Better int conversion
* debug
* Fix for Cronet
* Revert "debug"
This reverts commit 4d641c4281
.
* Revert "Better int conversion"
This reverts commit 4001b957cb
.
* Revert "Revert "Revert "debug"""
This reverts commit d135c61043
.
Co-authored-by: Jan Tattermusch <jtattermusch@google.com>
3 years ago
Craig Tiller
44167bdfd3
Convert server load reporting to promises ( #28927 )
...
* Call finalization for promises
* Convert filter to a promise
* Call finalization for promises
* comment
* split out and test
* dont use promise_detail:: directly
* fix
* Automated change: Fix sanity tests
* fix
* ?
* fix
* finish it
* modernize
* fix-test
* fix
* Review feedback
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago