Chiebot-Mirror/grpc - grpc - Gitea: Git with a cup of tea

Commit Graph

Author	SHA1	Message	Date
Craig Tiller	175ccc3a90	Reland global config changes (#32661 ) <!-- If you know who should review your pull request, please assign it to that person, otherwise the pull request would get assigned randomly. If your pull request is for a specific language, please add the appropriate lang label. --> --------- Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Jan Tattermusch	0c1797cd9f	Revert "[config] Move global config alongside core configuration" (#32659 ) Reverts grpc/grpc#30788 (it breaks grpc_objc_bazel_test (see https://github.com/grpc/grpc/pull/30788#issuecomment-1476372187) and also seems to be breaking some other internal stuff).	2 years ago
Craig Tiller	b7a83305e6	[config] Move global config alongside core configuration (#30788 ) This is a big rewrite of global config. It does a few things, all somewhat intertwined: 1. centralize the list of configuration we have to a yaml file that can be parsed, and code generated from it 2. add an initialization and a reset stage so that config vars can be centrally accessed very quickly without the need for caching them 3. makes the syntax more C++ like (less macros!) 4. (optionally) adds absl flags to the OSS build This first round of changes is intended to keep the system where it is without major changes. We pick up absl flags to match internal code and remove one point of deviation - but importantly continue to read from the environment variables. In doing so we don't force absl flags on our customers - it's possible to configure grpc without the flags - but instead allow users that do use absl flags to configure grpc using that mechanism. Importantly this lets internal customers configure grpc the same everywhere. Future changes along this path will be two-fold: 1. Move documentation generation into the code generation step, so that within the source of truth yaml file we can find all documentation and data about a configuration knob - eliminating the chance of forgetting to document something in all the right places. 2. Provide fuzzing over configurations. Currently most config variables get stashed in static constants across the codebase. To fuzz over these we'd need a way to reset those cached values between fuzzing rounds, something that is terrifically difficult right now, but with these changes should simply be a reset on `ConfigVars`. <!-- If you know who should review your pull request, please assign it to that person, otherwise the pull request would get assigned randomly. If your pull request is for a specific language, please add the appropriate lang label. --> --------- Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Mark D. Roth	df80a31199	Second attempt: xDS RouteConfig: use ValidationErrors and add unit test (#31658 ) * Revert "Revert "xDS RouteConfig: use ValidationErrors and add unit test (#31418)" (#31653)" This reverts commit `335b74fea7`. * use auto instead of explicitly naming the protobuf map type	2 years ago
Mark D. Roth	335b74fea7	Revert "xDS RouteConfig: use ValidationErrors and add unit test (#31418 )" (#31653 ) This reverts commit `c72a67b6da`.	2 years ago
Mark D. Roth	c72a67b6da	xDS RouteConfig: use ValidationErrors and add unit test (#31418 ) Specific changes: - Use absl::variant<> for HashPolicy types. - Pull validation of resulting LB policy configs out of ClusterSpecifierPlugin registry and into RouteConfig validation. - Don't ignore missing Any fields in HTTP filter configs if is_optional is true, since is_optional should apply only to unsupported filter types, not to malformed resource protos. - Use ExtractXdsExtension() for HttpConnectionManager config itself. - Minor API improvements to StatusCodeSet to make it a bit more useful in tests. - Changed StringMatcher to return the underlying error message when a regex fails to compile. - Fix a bug whereby a ClusterSpecifierPlugin would not be returned if it was not used in a vhost, even if it was used in another vhost.	2 years ago
Mark D. Roth	43c8cdd2e9	xDS: fix bug when a cluster specifier plugin was not used in all vhosts (#31583 )	2 years ago
Mark D. Roth	bd4b5d70c8	xDS: remove v2 support (#31327 ) * xDS: remove v2 support * remove unnecessary templating in fake xDS server code * remove now-unnecessary workaround for clang bug * clang-format * fix unit tests * iwyu * remove now-unused v2 proto files * fix from merge * clang-format * fix typo	2 years ago
Mark D. Roth	f2a377d35e	xDS cluster resource type: use ValidationErrors and add unit test (#31025 ) * XdsBootstrap: move two more methods out of the interface * Automated change: Fix sanity tests * XdsClient: add unit test * Automated change: Fix sanity tests * fix memory leaks * add helper method * add unsubscription * add test for multiple subscriptions * clang-format * fix build * fix flakiness * add checking for other node fields * add v2 test * add response builder * add test for update from server * add test for update containing only changed resources * clang-format * fix build * add test for resource not existing upon subscription * add test for stream closed by server * add test for multiple watchers for the same resource * add test for connection failure * clang-format * add test for resources wrapped in Resource wrapper message * add test for resource validation failure * add test for multiple invalid resources, and fix a case in XdsClient * add test for validation failure for already-cached resource * add test for server not resending resources after stream disconnect * clang-format * fix XdsClient to report channel errors to newly started watchers * fix XdsClient to send cached errors/does-not-exists to newly started watchers * fix watcher to ensure events arrive in the expected order * fix tests * clang-format * add test for multiple resource types * fix xds_cluster_e2e_test * Automated change: Fix sanity tests * cleanup * add federation tests * clang-format * remove now-unnecessary XdsCertificateProviderPluginMapInterface * code review comments * simplify XdsResourceType::Decode() API * XdsClient: add unit tests for XdsClusterResourceType * add XdsClient with gRPC bootstrap config * add LB policy tests * started adding CertificateProvider tests * update for recent API changes * fix merge bugs * xDS resource validation: identify extensions by type_url instead of name * fix build * migrate to ValidationErrors * add xds_common_types_test * finish TLS tests and add LRS tests * move ScopedExperimentalEnvVar to its own library and remove redundant e2e tests * add circuit breaking and outlier detection tests * add validation to outlier detection LB policy parsing * clang-format * Automated change: Fix sanity tests * fix signedness * fix sanity * fix sanity * iwyu * update code for XdsResourceTypeImpl changes Co-authored-by: markdroth <markdroth@users.noreply.github.com>	2 years ago
Craig Tiller	9c98d67d7a	[c++14] absl::make_unique --> std::make_unique (#31159 ) * [c++14] absl::make_unique --> std::make_unique * fix * fix	2 years ago
Mark D. Roth	e475e165a7	LB policies: convert to new JSON API (#30468 ) * 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 * fix tests, improve error messages, and add overload to parse to existing object * remove overload of LoadFromJson() * change special case for Json to instead use Json::Object * fix build * improve error structure, add missing types, and improve tests * clang-format * Automated change: Fix sanity tests * update grpclb parsing * fix build * convert outlier_detection LB config * convert priority LB policy * convert ring_hash LB config * 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 * WIP on RLS parsing conversion * fix error message * Automated change: Fix sanity tests * fixed RLS parser tests * fix error prefix in OD, priority, and ring_hash policies * convert weighted_target policy * convert xds_cluster_manager * convert cds policy * convert xds_cluster_resolver * convert xds_cluster_impl * Automated change: Fix sanity tests * fix test * fix xds_cluster_impl drop config parsing * Automated change: Fix sanity tests * attempt to fix mac build * work around gcc6 problem * Automated change: Fix sanity tests * fix build * add mechanism to conditionally disable individual fields * fix build * Automated change: Fix sanity tests * fix move assignment operator * fix build * Automated change: Fix sanity tests * fix build and simplify RLS duplicate key check * Automated change: Fix sanity tests * Automated change: Fix sanity tests * fix build * iwyu * fix build * fix sanity * add LoadRefCountedFromJson() and use it to eliminate some moves * fix build * Automated change: Fix sanity tests 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	c2ab8c99bb	Revert "Revert "[c++] Move environment functions to C++ (#30937 )" (#30986 )" (#30988 ) This reverts commit `96264e07b8`.	2 years ago
Craig Tiller	96264e07b8	Revert "[c++] Move environment functions to C++ (#30937 )" (#30986 ) This reverts commit `74c0d6fe3f`.	2 years ago
Craig Tiller	74c0d6fe3f	[c++] Move environment functions to C++ (#30937 ) * [gprpp] Move env to C++ * move headers/impl * Automated change: Fix sanity tests * fix * fix * Automated change: Fix sanity tests * Update http_proxy.cc * fix * fix * rename * fix merge * fix Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Mark D. Roth	8459824e5c	xds end2end tests: change tests to check all RPC failure statuses (#29974 ) * WIP * fix from merge * fix ring_hash tests * ring hash: fix picker propagation bug in xds_cluster_manager policy * fix build * clang-format (ish) * fix build * more conversion * remove old CheckRpcSendFailure * fix SendRpcsAndCountFailuresWithMessage * fix WaitForBackend * clang-format * revert xds resolver change, that will be done in a separate PR * Automated change: Fix sanity tests Co-authored-by: markdroth <markdroth@users.noreply.github.com>	3 years ago
Mark D. Roth	6bda57194a	xds_end2end_test_lib: use DEBUG_LOCATION to log where failures originate (#29444 )	3 years ago
Mark D. Roth	214e3f3622	xds_end2end_test: Move RLS tests to their own file (#29290 ) * move some code around * remove num_backends parameter from XdsEnd2endTest * remove use_xds_enabled_server param from XdsEnd2endTest * remove xds_resource_does_not_exist_timeout_ms param from XdsEnd2endTest * remove client_load_reporting_interval_seconds param from XdsEnd2endTest * start moving CreateAndStartBackends() into individual tests * finish moving CreateAndStartBackends() into individual tests * remove unused variable * remove SetEdsResourceWithDelay * fix test flake * clang-tidy * clang-format * move test framework to its own library * fix build * clang-format * fix windows build * rename TestType to XdsTestType * move BackendServiceImpl inside of BackendServerThread * clang-format * move AdminServerThread to CSDS test suite * move RLS tests to their own file * remove unnecessary deps * generate_projects	3 years ago

17 Commits (c32cbb57dfca02a0324a90f3a4e0a2ce6d6abae6)