Vignesh Babu
d7cce32449
Forking tcp socket utils for posix event engine endpoints ( #30383 )
...
* convert tcp code to use EndpointConfig and successfully compile
* regenerate projects
* copying config options used in tcp code to create map backed endpoint configs
* fix minor build issues
* fix includes in tests
* fix some build and test issues
* modifying endpoint config definition from an abstract interface into a concrete implementation
* minor fix
* add unreachable code check
* fix some windows build issues
* fix more build issues
* adding a grpc_tcp_generic_options struct to hold data extracted from EndpointConfig
* code formatting
* simplify
* fix sanity checks
* fix windows issues
* some posix fixes
* disabling copy constructor and copy assignment operator for endpoint config
* fix sanity checks
* fix syntax error
* fix weird formatting
* fix build issue
* fix review comments
* reverting un-necessary change
* remove channel args completely from windows impl since it is unused
* remove unused headers and fix usage in endpoint_pair_windows.cc
* cleanup
* cleanup
* fix some initialization issues
* re-generate projects
* removing some files
* fix ifdef for windows
* fixing windows build issue due to improper header file include
* Automated change: Fix sanity tests
* start
* regenerate-projects
* delete some files
* Automated change: Fix sanity tests
* review comments
* add comment
* review comments
* rename GetPointer to GetVoidPointer
* cleanup
* Automated change: Fix sanity tests
* removing EndpointConfig::Get method
* Automated change: Fix sanity tests
* cleanup
* fix
* minor fix
* fix
* update
* fix BUILD
* update
* fix sanity
* minor additions to configure global tcp user timeout values
* fix build syntax
* fix build error
* Add more unit tests
* fix msan issue
* add comment
* adding rcv lowat helper to socket wrapper
* fix sanity
* sanity
* iwyu sanity
* fix macos issue
Co-authored-by: Vignesh2208 <Vignesh2208@users.noreply.github.com>
2 years ago
Mark D. Roth
bcd8c991e6
XdsClient: add unit test and fix watcher notification bugs ( #30823 )
...
This adds a unit test for XdsClient and fixes several watcher-notification bugs found in the process. Specifically:
- When an ADS stream fails or an xDS channel reports a connectivity failure, report an error only to the watchers for resources being subscribed to on that particular channel, not to watchers on other channels.
- Cache the error status for the channel, so that if a new watcher is started after the channel reports the error, we can immediately report that error to the new watcher.
- If a resource is NACKed and has not been previously cached, or does not exist, report that fact to any new watcher that may be started later.
- If a resource in an ADS response is unparseable but is wrapped in a `Resource` wrapper, we do know its name, so record the validation failure in the cache and report it to the watchers.
Co-authored-by: markdroth <markdroth@users.noreply.github.com>
2 years ago
Cheng-Yu Chung
66749fd497
Fix layering_check violations ( #30926 )
...
* Fix layering_check violations
* Automated change: Fix sanity tests
Co-authored-by: Craig Tiller <ctiller@google.com>
2 years ago
Mark D. Roth
76588969c0
Second attempt: XdsClient: convert xDS bootstrap code to use new JSON parsing API ( #30925 )
...
* Revert "Revert "XdsClient: convert xDS bootstrap code to use new JSON parsing API (#30431 )" (#30924 )"
This reverts commit 3332e7e48d
.
* fix build for C++20
* iwyu and clang-format
2 years ago
Yash Tibrewal
ebcda5f0c1
GcpObservabilityConfig: s/disabled/enabled ( #30905 )
2 years ago
Yash Tibrewal
f7f4c9b584
GcpObservability: Code for reading config from env var ( #30889 )
...
* GcpObservability: Code for reading config from env var
* Comments
* Fix deps
* clang-tidy
2 years ago
Craig Tiller
a27073f7a8
[config] Move proxy mapper into core configuration ( #30842 )
...
* [config] Move proxy mapper into core configuration
* fix
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
346c103a1c
[build] Remove dependency cycle for grpc_init/shutdown ( #30921 )
...
* [build] Remove dependency cycle for grpc_init/shutdown
* missing files
* Automated change: Fix sanity tests
* fix
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Mark D. Roth
96e09c0747
client_channel: fix crash when cancelling a watch after SHUTDOWN ( #30885 )
2 years ago
Craig Tiller
4e024db990
[stats] Tune down resource usage ( #30846 )
...
* recurse
* bsearch
* bounds
* optimize stats
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* Automated change: Fix sanity tests
* [stats] Tune down histogram sizes
* Automated change: Fix sanity tests
* fix
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
3332e7e48d
Revert "XdsClient: convert xDS bootstrap code to use new JSON parsing API ( #30431 )" ( #30924 )
...
This reverts commit 04ddf3d0b7
.
2 years ago
Craig Tiller
89de6312cd
[build] Remove nofixdeps for call_metric_recorder ( #30918 )
2 years ago
Craig Tiller
2b46cb40d3
[config] Move certificate provider registry into core config ( #30890 )
...
* [config] Move certificate provider registry into core config
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
134401a4d6
[build] Remove nofixdeps for EE targets ( #30920 )
...
* [build] Remove nofixdeps for EE targets
* fix
2 years ago
Craig Tiller
d47e58544f
[flaky] Increase timeout ( #30909 )
2 years ago
Craig Tiller
ca6f8aacfc
[build] Remove nofixdeps for resource_quota_trace ( #30911 )
2 years ago
Craig Tiller
cfbb07e965
[build] Remove nofixdeps for gpr_platform ( #30912 )
2 years ago
Craig Tiller
3502837c95
[init] Move client backup poller init to be directly called by init.cc ( #30893 )
...
* [init] Move backup poller init out of client channel path
* fix
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
AJ Heller
bf1856d2cc
A modest split of `:gpr` for mpscq support ( #30798 )
...
* A modest split of `:gpr` for mpscq support
A full split of the `gpr` target into 25 separate targets is building,
but there are some hurdles to get over with respect to ODR violations
and public API support for the gpr library.
This PR splits off a small chunk of that work.
* Automated change: Fix sanity tests
Co-authored-by: drfloob <drfloob@users.noreply.github.com>
2 years ago
Craig Tiller
fa47118216
[build] Remove nofixdeps for grpc++_xds_server ( #30916 )
2 years ago
Craig Tiller
42874a2831
[build] Remove nofixdeps for grpc++_xds_client ( #30917 )
2 years ago
Craig Tiller
d2fd81362e
[build] Remove nofixdeps for grpc++_alts ( #30914 )
2 years ago
Craig Tiller
160ba0faeb
[build] Remove nofixdeps for channel creds registry ( #30919 )
2 years ago
Esun Kim
e90972bef6
Bump gcc 6 to 7 ( #30903 )
...
* Bump gcc 6 to 7
* Update
* Update 2
2 years ago
Mark D. Roth
04ddf3d0b7
XdsClient: convert xDS bootstrap code to use new JSON parsing API ( #30431 )
...
* Declarative JSON parser
* Automated change: Fix sanity tests
* fix
* shrinking stuff a little
* static vtables
* separate fns
* simpler?
* make maps work
* windows fixes
* Automated change: Fix sanity tests
* simplify code
* Automated change: Fix sanity tests
* vtable-test
* dont always create vec/map impls for every type
* comments
* make error consistent
* move method private
* progress
* durations!
* Automated change: Fix sanity tests
* fix
* fix
* fix
* Automated change: Fix sanity tests
* post-load
* Automated change: Fix sanity tests
* document JsonPostLoad() and add static_assert
* don't copy field names, to avoid length limitations
* use absl::Status
* accept either string or number for numeric values
* add test for direct data member of another struct type
* remove unused method
* add support for retaining part of the JSON wirthout processing
* update test for changes in Json::Parse() API
* add absl::optional support
* Automated change: Fix sanity tests
* add new parsing code
* use absl::optional<> parsing
* switch to new parsing code
* fix tests
* clang-format
* fix tests, improve error messages, and add overload to parse to existing object
* remove overload of LoadFromJson()
* make XdsBootstrap movable
* work around googletest bug
* Automated change: Fix sanity tests
* change special case for Json to instead use Json::Object
* use Json::Object instead of just Json
* remove copy ctor/assignment, spell out move ctor/assignment
* fix regex portability issue
* Automated change: Fix sanity tests
* fix build
* improve error structure, add missing types, and improve tests
* clang-format
* Automated change: Fix sanity tests
* fix build
* fix build
* attempt to work around gcc6 bug
* add LoadJsonObjectField(), add LoadFromJson() overload that takes an ErrorList parameter, and add tests for parsing bare top-level types
* fix msan
* Automated change: Fix sanity tests
* fix error message
* Automated change: Fix sanity tests
* add mechanism to conditionally disable individual fields
* clean up channel creds parsing
* use conditional enabling mechanism for federation-specific fields
* fix build
* Automated change: Fix sanity tests
* fix build
* Automated change: Fix sanity tests
* avoid unnecessary copies
* Automated change: Fix sanity tests
* make XdsBootstrap a clean interface
* add missing build dep
* fix xds_lb_policy_registry_test
* iwyu
* add missing build deps
Co-authored-by: Craig Tiller <craig.tiller@gmail.com>
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
Co-authored-by: Craig Tiller <ctiller@google.com>
Co-authored-by: markdroth <markdroth@users.noreply.github.com>
2 years ago
Craig Tiller
1f1f923a72
[benchmarks] Remove stats integration ( #30900 )
...
* remove old stats cruft
* remove
* remove
* fix
* fix
2 years ago
Jan Tattermusch
b9dfcc092e
disable testKeepaliveWithV2API on InteropTestsRemote ( #30870 )
2 years ago
Jan Tattermusch
3d7fc93e1f
restore sane repeat count and timeout for objC tests ( #30897 )
2 years ago
Yash Tibrewal
a644d8c9b4
GcpObservabilityConfig: cloud_trace.sampling_rate ( #30904 )
2 years ago
Wanlin Du
ba6e5af377
Revert "Stop dotnet tests in the OSS benchmark. ( #30787 )" ( #30906 )
...
This reverts commit 97f29a1dca
.
2 years ago
AJ Heller
d547df2872
Upgrade google/benchmark to v1.7.0 ( #30902 )
...
* Upgrade google/benchmark to v1.7.0
Reason: Setup and Teardown methods are nice to have for multithreaded
tests.
* use version tag
* Revert "use version tag"
This reverts commit 6c7f5e0d35
.
2 years ago
Cheng-Yu Chung
3b728d78f1
Build target reorganization Part 3 ( #30797 )
...
* Clean up `grpc++_base` and `grpc++_base_unsecure`
* Clean up `grpc++`
* Clean up `grpc++_unsecure`
* Fix issues in cherry picking
2 years ago
Sergii Tkachenko
326a88a9a2
xDS interop: Handle the edge case when rand deployment_id is all nums ( #30901 )
...
Fun edge case: when `rand_string()` happen to generate numbers only,
yaml interprets `deployment_id` label value as an integer,
but k8s expects label values to be strings.
K8s responds with a barely readable 400 Bad Request error:
`ReadString: expects \" or n, but found 9, error found in #10 byte of ...|ent_id`.
Prepending deployment name forces deployment_id into a string,
as well as it's just a better description.
2 years ago
Craig Tiller
bc5db5395a
[experiments] Allow specifying experiments defaults to be different for debug/release builds ( #30895 )
...
* cpp-codegen
* bazel-gen
* Automated change: Fix sanity tests
* Update grpc_build_system.bzl
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Yash Tibrewal
260a02d1ad
Revert "Revert "GcpObservabilityConfig: Add some basic field parsing structure"" ( #30894 )
...
* Revert "Revert "GcpObservabilityConfig: Add some basic field parsing structure (#30866 )" (#30892 )"
This reverts commit 37b631d1fd
.
* Add json_args deps for layering check
2 years ago
Craig Tiller
610a248059
[stats] Optimize histogram system ( #30834 )
...
* recurse
* bsearch
* bounds
* optimize stats
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Jan Tattermusch
becab4418d
upgrade rules_apple and apple_support bazel deps ( #30896 )
2 years ago
Craig Tiller
f9b866ff8b
[chttp2] Use experiments framework for peer_state_based_framing ( #30853 )
...
* [chttp2] Use experiments framework for peer_state_based_framing
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Jan Tattermusch
631392f2e7
Upgrade bazel version to bazel 5.3 (repo-wide) ( #30844 )
...
* update bazel to 5.3.0
* regenerate dockerfiles
* update bazel distribtest
* update docker image templates
* regenerate dockerfiles 2
* regenerate bazel_support.md
* upgrade rules_apple and apple_support bazel deps
* Revert "upgrade rules_apple and apple_support bazel deps"
This reverts commit eb9a0d2155
.
* fix objc bazel 5.3 build on kokoro mojave image
2 years ago
Jan Tattermusch
b45c5244cb
increase arm64 C/C++ bazel test timeout ( #30872 )
2 years ago
Craig Tiller
6cd33e9fd2
[experiments] Reduce test count ( #30891 )
2 years ago
Craig Tiller
37b631d1fd
Revert "GcpObservabilityConfig: Add some basic field parsing structure ( #30866 )" ( #30892 )
...
This reverts commit 31c68b5f10
.
2 years ago
Craig Tiller
ec27e98adc
[fork] Move init to once-only, remove shutdown ( #30839 )
...
* [fork] Move init to once-only, remove shutdown
* fix build
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
fb4fd9ced0
[windows] Fix forward for CBF in artifact build ( #30886 )
...
* fix
* Update tchar.cc
2 years ago
Craig Tiller
7ba58bb925
[build] Move certificate provider registry into its own build target ( #30836 )
...
* [build] Move cert registry into its own build target
* Automated change: Fix sanity tests
* fix
* forgot to save file
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Yash Tibrewal
31c68b5f10
GcpObservabilityConfig: Add some basic field parsing structure ( #30866 )
...
* GcpObservabilityConfig: Add some basic field parsing structure
* IWYU
* clang format
2 years ago
Craig Tiller
8996233989
[build] Remove tracer shutdown ( #30837 )
2 years ago
Craig Tiller
381976dc99
[windows] Handling for tchar type in C++ ( #30845 )
...
* [windows] Handling for tchar type in C++
* fix
* fix
* Automated change: Fix sanity tests
* fix
* fix
* Update tchar.cc
* Update tchar.h
* Update tchar.h
* Update env_windows.cc
* Update string_util_windows.cc
* Update tmpfile_windows.cc
* Update env_windows.cc
* Update env_windows.cc
* Update subprocess_windows.cc
* Automated change: Fix sanity tests
* fix
* Update subprocess_windows.cc
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
98a92544b7
[experiments] Add API to enable/disable experiments ( #30867 )
...
* [experiments] Add API to enable/disable experiments
* ensure !loaded
2 years ago
Vignesh Babu
3c720b092e
Fix flakiness in EventEngineClientTest.ConnectToNonExistentListenerTest ( #30874 )
2 years ago