Click house/v1.51.3 #31
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Clean-up which * Reformat
* [experiments] Dont check expiry dates in sanity * Automated change: Fix sanity tests Co-authored-by: ctiller <[email protected]>
* [chttp2] Fix performance regression for small stream sends * fix * comment * comment * fix
* Remove `include/grpcpp/impl/codegen/client_context.h` * Update * Update
* [sleep] Make this robust against poorly implemented event engines * Automated change: Fix sanity tests * Reland x2: Make GetDefaultEventEngine return a shared_ptr * x * [sleep] Add test for sleep robustness with bad event engines * Automated change: Fix sanity tests * fix * fix * Automated change: Fix sanity tests * Update sleep_test.cc * Update mock_event_engine.h * Automated change: Fix sanity tests Co-authored-by: ctiller <[email protected]> Co-authored-by: AJ Heller <[email protected]>
* [promises] Fix cronet tests * the complexes * fix
* XdsClient: avoid nesting result in XdsResourceTypeImpl * fix xds_endpoint_resource_type_test * fix sanity
* python tests install pinned virtualenv themselves * more alpine image simplifications * regenerate alpine image * introduce dockerfile template for xds interop tests harness deps * cleanup interop_http2 docker image, remove python_deps.include
…` (#31005)" (#31079)" (#31087) This reverts commit da08fe1.
Release secure channel credentials in Python AioChannel
* [bloat] Rewrite promise_based_filter vtables to maximize code sharing * moar
This PR moves the PSM benchmark away from the benchmark-prod2 cluster to psm-benchmarks-performance cluster. This is to avoid any unexpected change on the OSS benchmarks due to the tuning we may perform for PSM benchmarks. The psm-benchmarks-performance cluster has 2 system nodes, 1 driver-ci and 2 worker-ci nodes. The node type is the same as the benchmark-prod2 cluster.
* Simplified GRPC_ERROR_CREATE macros * Code migration
* fixes * Automated change: Fix sanity tests * fix * fix * fix * fix * fixes * fixes * fix * fix-win * fix iwyu * fix * fix Co-authored-by: ctiller <[email protected]>
…1025) * 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 <[email protected]>
* OpenCensus: Ability to globally disable stats and tracing * Fix docs
…ect counter (#31250) * update test to use transport counter instead of iomgr object counter
…sn't enable them (#31253)
* [event_engine] Move security executor usage to event engine * build-fix * fix-iwyu
* Remove `include/grpcpp/impl/codegen/config.h` * Update
…tsockopt operation (#31565) * C++ize TracedBuffer Interface and fix bug to eliminate lock contention during getsockopt operation * fix * fix sanity * fix sanity * review comments
* Actually record CPU usage * Do the client path too * Actually build the image from the PR * Revert "Actually build the image from the PR" This reverts commit 3afa5cf3c301371fed7128b3c9c7910a68605c78. * Support importing on Windows
* Fix msbuild failing when '@' is present in path There was ambiguity in references to Source metadata on some Items. These are now qualified to remove the ambiguity. * Revert "Fix msbuild failing when '@' is present in path" This reverts commit bb5d5ddc61d30456aca375571eae3f53a03de35f. * Unescape Indentity when creating Source * Fix spelling in comment
Recently we had an issue with Service Account IAM policy getting too large. RC is the cleanup script missing GCP SA account. This PR adds adds missing `gcp_service_account` to the cleanup script.
* Add typing for some internal python files.
* bump C-core version * regenerate projects
* A basic set of EventEngine listener tests * review comments * replace expect with asserts * review comments
* [build] Stop publishing iomgr/error.h from grpc_base * fixes * fix
* bump version to 1.51.0-pre1 * regenerate projects
…endpoints #31616" and "Added url validation for aws metadata endpoints in aws external account #31626" (#31646) * Adding token url validation cases for psc endpoints (#31616) * Adding validation case for psc endpoint * formatting fix * Added url validation for aws metadata endpoints in aws external account (#31626) * Added url validation for aws metadata endpoints in aws external account * addressing review comments * fix error message back * Fix broken test
…668) (#31672) * xds resolver: fix edge cases in interactions between LDS and RDS * improve SwitchFromInlineRouteConfigToRds test * clang-tidy * Automated change: Fix sanity tests Co-authored-by: markdroth <[email protected]> Co-authored-by: markdroth <[email protected]>
…#31684) (#31691) * xds resolver: fix crash on wrong listener type * fix same bug on server side * fix clang-tidy and add requested TODO
* bump version to 1.51.0 * regenerate projects
This reverts commit b3d9833.
* Bump to 1.51.1 * Regenerate projects
This is a test running on legacy (GCE) framework. It was disabled in grpc/grpc#27355, however it has never been ported to the new framework. Unless there were reasons I'm not aware of, disabling it is a mistake. Other languages still run it: - Java: https://github.com/grpc/grpc-java/blob/cc5378453fcdb8d4fdc18a1a369184210ce9ece0/buildscripts/kokoro/xds_v3.sh#L21 - Go: https://github.com/grpc/grpc-go/blob/f2fbb0e07ebf3dd46aa641ee89c9f17e8083eaf6/test/kokoro/xds.sh#L30 - Node: https://github.com/grpc/grpc-node/blob/81083bd22912bd3272d193dcc14077d0c4f50399/packages/grpc-js-xds/scripts/xds.sh#L57
…#32093) (#32117) Apparently there's a difference between bash 3 and bash 4. OSX comes with bash 3 out-of-box, so for whoever wrote this logic it "worked on my machine". The `((` construct returns a 0 exit code if the value is non-zero. Since the value starts at 0 and we do a post-increment, it will always fail the first time. Changing it to a pre-increment should fix it.
<!-- 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. --> <!-- 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. -->
…iversal2)) (#32424)
|
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.