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
Craig Tiller
b463cabeec
Update flaky tests ( #30873 )
...
* Update flaky tests list
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Mark D. Roth
2bb6097412
XdsClient: remove now-unnecessary XdsCertificateProviderPluginMapInterface ( #30854 )
...
* remove now-unnecessary XdsCertificateProviderPluginMapInterface
* code review comments
* Automated change: Fix sanity tests
Co-authored-by: markdroth <markdroth@users.noreply.github.com>
2 years ago
Craig Tiller
dcfffd7603
[experiments] Dont run on arm64 ( #30871 )
...
* [experiments] Dont run on arm64
* remove duplication to eliminate bug
2 years ago
Jan Tattermusch
070a15aa29
switch cfstream tests to buildonly ( #30849 )
2 years ago
Craig Tiller
47a43d68c9
[chttp2] Tune initial window size/max frame size setting ( #30550 )
...
* [chttp2] Tune initial window size/max frame size setting
- Only choose frame sizes and initial window sizes that are powers of two - as a more predictable function to avoid flapping.
- If window size drops below 1kb then snap to 0 to avoid frantically changing a value that needn't be
- Allow the initial window size to drop to zero (we have fuzzing in place that says this is safe)
* copy/paste fix
* fix build
* fix
* Automated change: Fix sanity tests
* fix announce bug
* cleanup
* put this change under an experiment
* more tests
* fix build
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Hannah Shi
346afa40c6
[ObjC]fix c++14 errors in bazel objC ( #30448 )
...
* fix c++14 errors in bazel objC
* fix CI error
2 years ago
Mark D. Roth
68453275ab
XdsBootstrap: move two more methods out of the interface ( #30820 )
...
* XdsBootstrap: move two more methods out of the interface
* Automated change: Fix sanity tests
Co-authored-by: markdroth <markdroth@users.noreply.github.com>
2 years ago
Craig Tiller
e705b2e3c7
[resource_quota] Move experiments to the experiments framework ( #30830 )
...
* fix
* fix tests
* fix
* Automated change: Fix sanity tests
* better expansion
* bleh
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
6ee7a647bf
[experiments] Add support for testing disabling already enabled experiments ( #30831 )
2 years ago
Jan Tattermusch
2d7ebdb5fd
avoid frequent PHP mac distribtest timeouts ( #30847 )
2 years ago
Craig Tiller
9d047f88f5
[exec-ctx] Remove ApplicationCallbackExecCtx::Global{Init,Destroy} ( #30840 )
2 years ago
Craig Tiller
fed6c180e9
[census] Add time scaling to test ( #30851 )
2 years ago
tony
e313ac8398
Grpc.Tools: Parse warnings from libprotobuf ( fix #27502 ) ( #30371 )
...
* issue 27502 - parse warning from libprotobuf
The warnings and errors logged by some of the pluggins such as the csharp
plugin are in a different format from other warnings etc logged by the
protobuf compiler (the plugins use GOOGLE_LOG to log message).
The parsing code in Grpc.Tools has been modified to correctly parse these
so that warnings are no longer handled as errors.
* Fix typos in comments
2 years ago
tony
f0948a7c38
Grpc.Tools add support for env variable GRPC_PROTOC_PLUGIN ( fix #27099 ) ( #30411 )
...
* issue 27099 Grpc.Tools add env GRPC_TOOL_PLUGIN
Added environment variable GRPC_TOOL_PLUGIN to override the
GRPC plugin path
* Change GRPC_TOOL_PLUGIN to GRPC_PROTOC_PLUGIN
2 years ago
Craig Tiller
0079382cfd
[cq] Track cq type to determine api usage ( #30833 )
...
* [cq] Track cq type to determine api usage
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
592bfb9c00
[cleanup] Remove unused file ( #30824 )
...
* [cleanup] Remove unused file
* fix
2 years ago
tony
dce1c83617
Grpc.Tools document AdditionalImportDirs ( #30405 )
...
* Grpc.Tools document AdditionalImportDirs
* Update doc for AdditionalImportDirs
* fix type
Co-authored-by: Jan Tattermusch <jtattermusch@google.com>
2 years ago
Michael Nosthoff
3353d61c5a
cmake: fix cross-compilation with gRPC_BUILD_GRPC_CPP_PLUGIN=OFF ( #30378 )
...
commit 99752b173c
reintroduced the cross-compilation issue which was fixed by da10b795e1
2 years ago
Craig Tiller
ca8b437613
[experiments] Single experiments flag ( #30822 )
...
* single experiments flag
* add missing files
* fix
* empty-string-fix
* fix
* Automated change: Fix sanity tests
* ensure cold path is out of line
* add log for enabled/disabled experiments
* remove bad sentence
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
c3c4159d24
[build] Make stats test drastically faster under tsan ( #30712 )
...
* [fixit] Make stats test drastically faster under tsan
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Craig Tiller
8b13c0d581
[iomgr] SO_RCVLOWAT impl for grpc ( #30736 )
...
* [stats] Cleanup stats system
* clear out optionality
* fix
* might as well...
* Automated change: Fix sanity tests
* only allocate 8k or 64k blocks for tcp reads
* Automated change: Fix sanity tests
* fix
* Update tcp_posix.cc
* Automated change: Fix sanity tests
* [chttp2] Inform endpoint of min progress size
* respect target, min progress sizes
* cleaner loops
* clean out more unused stuff
* clean out more unused stuff
* Automated change: Fix sanity tests
* update
* [iomgr] SO_RCVLOWAT impl for tcp
* tsan race
* tsan race
* fix
* Automated change: Fix sanity tests
* Introduce flag
* protect with flag
* [experiments] Make output more diffable/readable
* Automated change: Fix sanity tests
* buildifier sized indentations
* add experiment
* Update tcp_posix.cc
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Vignesh Babu
70dd34b438
Revert "Revert "Modifying iomgr tcp code to use event engine EndpointConfig instead of channel_args"" ( #30509 )
...
* Revert "Revert "Modifying iomgr tcp code to use event engine EndpointConfig instead of channel_args (#30028 )" (#30495 )"
This reverts commit 0b1a8c984b
.
* bug fix
* bug fix
* fix sanity
2 years ago
Craig Tiller
903e0490ea
[iomgr] Restrict TCP read allocations to 8kb or 64kb ( #30626 )
...
* [stats] Cleanup stats system
* clear out optionality
* fix
* might as well...
* Automated change: Fix sanity tests
* only allocate 8k or 64k blocks for tcp reads
* Automated change: Fix sanity tests
* fix
* Update tcp_posix.cc
* Automated change: Fix sanity tests
* respect target, min progress sizes
* cleaner loops
* clean out more unused stuff
* clean out more unused stuff
* Automated change: Fix sanity tests
* update
* tsan race
* Automated change: Fix sanity tests
* Introduce flag
* protect with flag
* [experiments] Make output more diffable/readable
* Automated change: Fix sanity tests
* buildifier sized indentations
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
Mark D. Roth
bf9304ef17
client_channel, client_auth: rewrite disallowed status codes from the control plane ( #30789 )
...
* client_channel: rewrite illegal status codes from control plane
* rewrite illegal status codes for call creds
* move fail_lb policy out of retry_lb_fail test so it can be reused
* test resolver and LB policy status rewrites
* add test for ConfigSelector status rewriting
* attempt to add client_auth filter unit test
* fix client_auth_filter test
* cleanup test
* fix build
* fix some memory leaks
* Automated change: Fix sanity tests
* Update client_auth_filter_test.cc
* fix build
* code review comments
* clang-tidy
Co-authored-by: markdroth <markdroth@users.noreply.github.com>
Co-authored-by: Craig Tiller <ctiller@google.com>
2 years ago