68d721058c
Prepare async_end2end_test for graceful goaway ( #29054 )
3 years ago
b505da65e7
[BinderTransport] add authority to initial metadata if missing ( #29035 )
...
When C++ implementation acts as server and Java implementation acts as
client, authority will not be present in initial metadata. Although
authority is not used for binder transport, when it is missing, core API
layer will fail the RPC call silently.
This commit adds a fake authority to initial metadata when it is
missing to avoid the failure.
Test case is added to make sure the wire reader implementation appends
both path and authority to metadata correctly.
3 years ago
be4e1051a9
Add resource quota for secure endpoint ( #28970 )
...
* Add resource quota for secure_endpoint
* Fix double linked list error in tests
* Experiments to prevent memory leaks
* Fix memory leaks
* Fix asan error
* Add locking to fix data races
* Automated change: Fix sanity tests
* Address review comments
Co-authored-by: ananda1066 <ananda1066@users.noreply.github.com>
3 years ago
bd357f9a37
Remove shutdown_cq from grpc_end2end_test_fixture ( #29049 )
...
* Remove some shutdown_cq instances
* Remove shutdown_cq from grpc_end2end_test_fixture
3 years ago
b639e09c90
Enable ipv4 end2end tests inside Google ( #29013 )
3 years ago
00d587dc56
Include ADS stream error in XDS error updates ( #29014 )
...
* include ADS stream error in XDS error updates
* Re-package XDS client errors into unavailable rather than unknown, in XDS resolver
3 years ago
736441f288
Call push/pull abstraction ( #29033 )
...
* push/pull impl
* comment
* fix
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
762cde4d24
Delete end2end nosec tests ( #28946 )
...
* rm nosec tests (TODO: identify coverage differences)
* grpc::load_reporter
* delete unused nosec test files & config
* trigger build
* re-delete nosec test file (snuck in with a merge)
* revert TODO deletion
3 years ago
fe913387da
Remove idempotent/cacheable requests ( #28922 )
...
* Remove idempotent/cacheable requests
* more cleanup
* bump core version
* fix
* fix
* fix
* review feedback
* fixes
* fix
* remove more
* objc
* fix
* fix
* fix
* scrub
* Modify XdsRbacTests
Co-authored-by: Yash Tibrewal <yashkt@google.com>
3 years ago
047642c5c4
Call finalization for promises ( #29008 )
...
* Call finalization for promises
* comment
* split out and test
* dont use promise_detail:: directly
* fix
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
55ba777c7a
credentials_test: Add namespace ( #29006 )
3 years ago
d9883c2e66
Fix iOS build ( #28995 )
...
* Fix iOS compiler
* fix
3 years ago
bbe49dc16b
Delete dns resolver connectivity test ( #28994 )
3 years ago
20521fb33e
Fix memory leak in HTTP request security handshake cancellation ( #28971 )
...
* Fix memory leak in HTTP request security handshake cancellation
3 years ago
3eba748552
Remove retry throttle data from global init ( #28988 )
...
* Remove retry throttle data from global init
* fix
* cleanup
3 years ago
87acbadba1
Third attempt: Convert client_auth_filter to promises ( #28968 )
...
* wip
* Automated change: Fix sanity tests
* fixes
* progress
* progress
* grpc compiles
* Automated change: Fix sanity tests
* fixing tests
* x
* progress
* better code
* Automated change: Fix sanity tests
* progress
* progress
* windows fix
* Make Duration metadata trivial
* better message
* fix
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* progress
* fixes
* fix
* fix
* spam
* un-disable errantly disabled tests
* gain insight
* Automated change: Fix sanity tests
* fixes
* fixes
* fix
* debug
* tweak
* fix
* fix timeout
* fix comment
* fixes
* x
* better test
* tests
* Automated change: Fix sanity tests
* missed file
* fix
* x
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* fix
* merge
* Automated change: Fix sanity tests
* Revert "Revert "Revert "Revert "Convert client_auth_filter to promises (#28767 )" (#28951 )" (#28952 )" (#28967 )"
This reverts commit 0f73576b17
3 years ago
5fc3ff8203
grpc_millis -> Timestamp/Duration ( #28119 )
...
* wip
* Automated change: Fix sanity tests
* fixes
* progress
* progress
* grpc compiles
* Automated change: Fix sanity tests
* fixing tests
* x
* progress
* better code
* Automated change: Fix sanity tests
* progress
* progress
* windows fix
* Make Duration metadata trivial
* better message
* fix
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* progress
* fixes
* fix
* fix
* spam
* un-disable errantly disabled tests
* gain insight
* Automated change: Fix sanity tests
* fixes
* fixes
* fix
* debug
* tweak
* fix
* fix timeout
* fix comment
* fixes
* x
* better test
* tests
* Automated change: Fix sanity tests
* missed file
* fix
* x
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* fix
* merge
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
c27f8f1505
Fourth attempt: Implement transparent retries ( #28943 )
...
* Revert "Revert "Third attempt: Implement transparent retries (#28925 )" (#28942 )"
This reverts commit be70a8676c
3 years ago
faea518de0
Fix implicit std::string_view(nullptr) ( #28589 )
...
This fixes failure to compile on (pre-release) GCC 12 due to use of
explicitly deleted function:
3 years ago
c08eb4fc41
Attempt to upgrade upb to the latest ( #28685 )
...
* Update upb to 85e5c76cb7098d3ae35e575d652a40568013a149
* Update grpc_deps.bzl
* Run gen_upb_api.sh
* Run run_sed.sh
* Update gen_build_yaml.py
* Regen projects
* Reformat
* Update third_party/upb to 950cdab42fe4c83f06a2be46eeb46a068e486eed
* Update third_party/upb to c9c31241bc39922b092a81d192badd50cccaaa81
* Update src/upb/gen_build_yaml.py
* Generated projects
* Update third_party/upb to a02d92e0257a35f11d4a58b6a932506cbdbb2f29
* Patching in https://github.com/protocolbuffers/upb/pull/524
* Sed script went too far.
Co-authored-by: Nicolas 'Pixel' Noble <nicolas@nobis-crew.org>
3 years ago
0f73576b17
Revert "Revert "Revert "Convert client_auth_filter to promises ( #28767 )" ( #28951 )" ( #28952 )" ( #28967 )
...
This reverts commit 235098de22
3 years ago
3d4433beb2
retry: always free cached send op data ( #28956 )
...
* Revert "Revert "retry: fix memory leak due to not freeing cached send ops upon cancellation (#28945 )" (#28954 )"
This reverts commit 9498843a18
3 years ago
16a3ce51ff
Service config parser to core configuration ( #28883 )
...
* Service config parser to core configuration
* x
* Automated change: Fix sanity tests
* finish
* Automated change: Fix sanity tests
* oops
* fix race
* Automated change: Fix sanity tests
* back out mutex
* refactor
* optimize
* Automated change: Fix sanity tests
* fix
* fix
* split out interface
* review feedback
* x
* fixes
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
235098de22
Revert "Revert "Convert client_auth_filter to promises ( #28767 )" ( #28951 )" ( #28952 )
...
* Revert "Revert "Convert client_auth_filter to promises (#28767 )" (#28951 )"
This reverts commit 5bfe38f10f
3 years ago
f18e6ede04
Revert "Convert max_age filter to promises ( #28904 )" ( #28957 )
...
This reverts commit ae4b45d75a
3 years ago
ae4b45d75a
Convert max_age filter to promises ( #28904 )
...
* start
* stuff
* build
* Automated change: Fix sanity tests
* fix
* finish him
* fix
* better
* fix
* review feedback
* Update channel_idle_filter.cc
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
9498843a18
Revert "retry: fix memory leak due to not freeing cached send ops upon cancellation ( #28945 )" ( #28954 )
...
This reverts commit f48c4f1f4b
3 years ago
7c74d58cbe
Disable memory_usage test on cmake builds ( #28948 )
3 years ago
5bfe38f10f
Revert "Convert client_auth_filter to promises ( #28767 )" ( #28951 )
...
This reverts commit cadce47ebf
3 years ago
cadce47ebf
Convert client_auth_filter to promises ( #28767 )
...
* Convert client_auth_filter to promises
* outline
* x
* x
* fixes
* remove-spam
* Async sequences over iterators
* iterate
* better semantics
* fix
* clang-tidy
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fixes
* fixes
* Automated change: Fix sanity tests
* progress
* progress
* progress
* progress
* progress
* credentials_test passes
* Automated change: Fix sanity tests
* Make Activity more of an interface
Move mutex and wakeup logic into a new class between PromiseActivity<>
and Activity (so that the sharing can persist), but make Activity closer
to a pure interface, so that whilst we migrate code we can implement
better fakes without forcing allocation.
* fixes
* Automated change: Fix sanity tests
* fixes
* test the awful failure
* x
* fix typo
* fix race
* 4.9
* x
* review feedback
* review feedback
* Automated change: Fix sanity tests
* review feedback
* replace AuthMetadataContext interface with GetRequestMetadataArgs struct
* Automated change: Fix sanity tests
* fix merge
* Fixup
* fix
* fix
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
Co-authored-by: Mark D. Roth <roth@google.com>
3 years ago
f48c4f1f4b
retry: fix memory leak due to not freeing cached send ops upon cancellation ( #28945 )
...
* retry: fix memory leak due to not freeing cached send ops upon cancellation
* clang-format
* attempt to fix proxy tests on windows
3 years ago
6b8a17d1ea
add bazel 5.0 to bazel distribtest ( #28894 )
3 years ago
dd76a04b8c
Move resolver to core configuration ( #28881 )
...
* Move resolver to core configuration
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* resolver: clean up and modernize registry
* fix race
* update visibility
* fix internal error
* review feedback
* resolve backref issues
* windows
* x
* fix sockaddrs on windows?
* fix sockaddrs on windows?
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
Co-authored-by: Mark D. Roth <roth@google.com>
3 years ago
ff87ca02b2
Revive test/core/memory_usage ( #28931 )
...
* Revert "Deprecate gpr_get/set_allocation_functions"
This reverts commit 81df105ac8
3 years ago
05056e149a
eliminate bm_error ( #28944 )
3 years ago
be70a8676c
Revert "Third attempt: Implement transparent retries ( #28925 )" ( #28942 )
...
This reverts commit 9e7453409a
3 years ago
3f19e0cc02
Revert "Revert "CompositeChannelCredentials: Comparator implementation ( #28902 )" ( #28919 )" ( #28918 )
...
* CompositeChannelCredentials: Comparator implementation retry
* Fix test
3 years ago
9e7453409a
Third attempt: Implement transparent retries ( #28925 )
...
* Revert "Revert "Revert "Revert "Implement transparent retries (#28548 )" (#28888 )" (#28890 )" (#28909 )"
This reverts commit 23da14c1a7
3 years ago
85189b24bb
Reintroduce: Avoid fully qualifying namespaces (and add check) ( #28917 )
...
Based on a handful of https://abseil.io/tips , it's generally advised to
only fully-qualify namespaces when in a `using` statement, or when it's
otherwise required for compilation. In all other cases, the general
recommendation is to not fully-qualify.
This change fixes most `grpc.*` namespace uses. There are potential
challenges in trying to make blanket changes to non-gRPC namespace uses,
such as `::testing`, since there is also a `grpc::testing` namespace.
3 years ago
bbfcb7ca35
Revert CompositeChannelCreds merge fiasco ( #28930 )
3 years ago
778ad437b1
HttpRequestSSLCredentials: Comparator implementation ( #28900 )
...
* HttpRequestSSLCredentials: Comparator implementation
* Unused argument
* Singleton HttpRequestSSLCredentials
3 years ago
a72c8ebb7d
CredentialsTest: Fix TestGoogleDefaultCredsExternalAccountCredentials ( #28923 )
3 years ago
25aa80966a
retry: fix memory leak if call is unreffed without starting recv_trailing_metadata ( #28913 )
...
* retry: fix memory leak if call is unreffed without starting recv_trailing_metadata
* fix test flake
3 years ago
e6a54ef372
Revert "CompositeChannelCredentials: Comparator implementation ( #28902 )" ( #28919 )
...
This reverts commit 2fdb5f90c3
3 years ago
e72a5fe5dd
Revert "Avoid fully qualifying namespaces (and add check) ( #28901 )" ( #28916 )
...
This reverts commit fc7314c701
3 years ago
fc7314c701
Avoid fully qualifying namespaces (and add check) ( #28901 )
...
Based on a handful of https://abseil.io/tips , it's generally advised to
only fully-qualify namespaces when in a `using` statement, or when it's
otherwise required for compilation. In all other cases, the general
recommendation is to not fully-qualify.
This change fixes most `grpc.*` namespace uses. There are potential
challenges in trying to make blanket changes to non-gRPC namespace uses,
such as `::testing`, since there is also a `grpc::testing` namespace.
3 years ago
431d314eca
Calculate clock difference and accomodate it to deflake timeout tests ( #28868 )
...
* Calculating clock difference and accomendate it to deflake timeout
tests.
* Fixing according to code review comments.
3 years ago
2fdb5f90c3
CompositeChannelCredentials: Comparator implementation ( #28902 )
3 years ago
23da14c1a7
Revert "Revert "Revert "Implement transparent retries ( #28548 )" ( #28888 )" ( #28890 )" ( #28909 )
...
This reverts commit 9f90e6d906
3 years ago
e178a44931
XdsCredentials: Comparator implementation ( #28903 )
3 years ago
Yash Tibrewal
Ming-Chuan
Alisha Nanda
AJ Heller
apolcyn
Craig Tiller
Mark D. Roth
Ben Beasley
Esun Kim
Jan Tattermusch
donnadionne