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

Commit Graph

Author	SHA1	Message	Date
apolcyn	27ee3913d1	[iomgr and EE logging] Log ERROR if socket() returns EMFILE on posix (#32204 ) There are potentially surprising deployment bugs that can cause `EMFILE` to be hit. For example, file descriptor limits can be easily reached if - the round robin LB policy is used - the load balancer hands out an assignment with a lot of backends - using debian's default 1024 file descriptor limit. To make such problems more apparent, we can pay special attention to this error and log ERROR when it happens. Related: b/265199104	2 years ago
Esun Kim	905df9dee6	Update Abseil to 20230125 (#32139 ) * Update abseil to 1.20230125.0 * Fix IWYU	2 years ago
AJ Heller	3980ed706a	[EventEngine] Refactoring the EventEngine Test Suite: Part 1 (#32127 ) * Add back EventEngine test suite tools (echo binary) This reverts commit `d5c12195a2`. * more refactoring * fix * Automated change: Fix sanity tests * fix * fix posix oracle build def * fix FuzzingEventEngine includes * sanitize * tidy and rm DNS * fix cfstream includes * header guards * format & fix deps * fix * fix * iwyu * fix * remove redundant code * reviewer feedback Co-authored-by: drfloob <drfloob@users.noreply.github.com>	2 years ago
Vignesh Babu	8cf04e9a54	[EventEngine] Modify iomgr to allow creation of event engine clients and client side endpoints (#31661 ) * [WIP] EventEngine iomgr endpoint shims * [WIP] EventEngine::Endpoint iomgr shims for the PosixEventEngine * Util functions to help with posix engine listener implementation * sanity * update comments in posix_engine_listener_utils.h * review comments * iwyu * revert prev commit * iwyu * update build * update * regenerate projects * regenerate projects * minor fixes * update BUILD * sanity * update build * regenerate projects * fix unused parameter * sanity * update * sanity * regenerate_projects * remove unused variable * start * update * regenerate_projects * sanity * update * fixes * update * regenerate_projects * update * fix sanity and msan failure * more fixes * build failure * update * fix * sanity * fixes * update * regenerate projects * fix sanity * review comments * An EventEngine subclass to be implemented by all posix based event engines * sanity * comments * update * review comments * re-word * fix * update * review comments * regenerate projects * syntax fix * add lock free event benchmark * releasable mutex lock * fix build isue * update * start * regenerate projects * update * fix * windows build * update * windows portability issue * update * update * update * update * format * update * update * update * update * update * fix sanity * regenerate projects * update * iwyu * Fix resolved address length related bugs in tcp_socket_utils and listener_utils * iwyu * cleanup src/core/lib/event_engine/tcp_socket_utils.cc * iwyu * fix * regenerate projects * fix sanity * re-write endpoint shim * more re-write * cleanup * update * review comments * build issue * more build issue fixes plus adding event_engine_trace * even more build issue fixes * iwyu * add static_cast * update * remove redundant code * update * deduplicate * iwyu * Fix review comments and regenerate_projects * sanity * review comments * fix include guards Co-authored-by: AJ Heller <hork@google.com>	2 years ago
Yash Tibrewal	aa11978541	Fix include guards for src/ and test/ directories (#32167 ) * Fix include guards for src/ and test/ directories * Sanity * Update new files	2 years ago
AJ Heller	41e7aff3be	[EventEngine] Update EventEngine client experiment (#32099 ) * [EventEngine] Update EventEngine client experiment This bumps the expiry timeline, and guards certain poller behavior behind the flag. The Epoll1 poller has been shown to not support fork events, which is a problem for Python. Fixes for that are incoming, but for now, the poller instantiation is guarded behind the flag as well. * force the EE endpoint experiment on for posix EventEngine tests * fix experiment code, it was not respecting forced values	2 years ago
Craig Tiller	67f364e23e	[cleanup] Eliminate usage of GRPC_ASSERT(false...); (#31757 ) * crash function * progress * fix * fix * Automated change: Fix sanity tests * fix * fix * fix * fixes * Automated change: Fix sanity tests * fix * Automated change: Fix sanity tests * fix * fix * use cpp attr * Automated change: Fix sanity tests * fix * fix * fix * fix * fix * fix * add exclusion * fix * typo * fix * fmt * Update tcp_socket_utils.cc * Automated change: Fix sanity tests * fix * revert php changes * Automated change: Fix sanity tests Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Vignesh Babu	3baeab973a	Remove posix_event_engine_polling_enabled experiment (#32037 ) * Remove posix_event_engine_polling_enabled experiment * regenerate projects	2 years ago
Vignesh Babu	8bd2b9af52	Fixing some bugs in posix event engine (#31785 ) * update * regenerate projects * fix sanity * review comments * fix * add lock free event benchmark * releasable mutex lock * fix build isue * update * fix sanity * regenerate projects * iwyu * Fix resolved address length related bugs in tcp_socket_utils and listener_utils * iwyu * cleanup src/core/lib/event_engine/tcp_socket_utils.cc * update endpoint to use SliceCast into MutableSlice * fix sanity * review comments * review comments * regenerate projects * fix * rm unused variable * fix	2 years ago
Craig Tiller	a414dbd304	Revert #30349 (#32008 ) * [iwyu] Add missing #include <type_traits> to fix build breakage with LLVM after `e0a66116fc` * Update IWYU mappings New libc++ doesn't provide <type_traits> when including <utility>. * run iwyu Co-authored-by: alexfh <alexfh@google.com>	2 years ago
Yijie Ma	f99b8b5bc4	Convert c-style comments to C++-style comments (#31923 ) * baseline * fix clang-tidy * manually revert these files * manually fixup at eof * revert 2 more files * change check_deprecated_grpc++.py * change end2end_defs.include template * fix check_include_guards.py * untrack tools/distrib/python/convert_cstyle_comments_to_cpp.py not yet ready to be submitted * fix yapf check_include_guards.py remove a space... * fix version.cc.template * fix version_info.h.template	2 years ago
Craig Tiller	49a3035d44	[iwyu] Fix CI job (#31921 ) * fix-includes * fix * fix * fix * fix * fix * Automated change: Fix sanity tests * add warning label Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
AJ Heller	18029913b9	Remove posix_engine namespace (#31787 ) * Remove posix_engine namespace * Automated change: Fix sanity tests * rm redundant using statements Co-authored-by: drfloob <drfloob@users.noreply.github.com>	2 years ago
Mark D. Roth	5e4d9f4bcf	xDS stateful session affinity: implement C-core filter (#31788 ) * stateful session affinity: implement filter * register filter config parser * fix unused parameter errors * remove some FIXMEs that are not longer needed * clang-tidy * iwyu * iwyu * revert iwyu changes to observability_logging_sink.cc * generate_projects * shorten filter name * don't use absl::optional for path * fix build * don't add cookie to trailing metadata unless it's Trailers-Only	2 years ago
Cheng-Yu Chung	f5d21653fc	Revert "[event_engine] Move combiner executor usage to event engine (#31713 )" (#31753 ) This reverts commit `b04aa1cf92`. Co-authored-by: AJ Heller <hork@google.com>	2 years ago
AJ Heller	557e558825	[EventEngine] WindowsEndpoint (#31735 ) * [EventEngine] WindowsEndpoint Initial sketch, all tests passing * Port fix from #28432 * GPR_WINDOWS guard * use MemoryAllocator::MakeReservation for allocated buffers * better logging (respect slice length) * Automated change: Fix sanity tests * improvements * Automated change: Fix sanity tests * InlinedVector<WSABUF, kMaxWSABUFCount> * initial attempt at socket util reunification * posix fixes + local run of sanitize.sh * posix socket includes * fix * Automated change: Fix sanity tests * remove unused include (breaks windows) * remove stale comment Co-authored-by: drfloob <drfloob@users.noreply.github.com>	2 years ago
Cheng-Yu Chung	77b3972f95	Clean up the TODO item in build target `grpc_base` part `channel_args` (#31642 ) * Clean up the TODO item in build target `grpc_base` part `channel_args` * Update * Update * Update * Automated change: Fix sanity tests * update * Automated change: Fix sanity tests	2 years ago
Craig Tiller	b04aa1cf92	[event_engine] Move combiner executor usage to event engine (#31713 ) * [event_engine] Move combiner executor usage to event engine * fix * review feedback * fix * x * fix * fix	2 years ago
Vignesh Babu	864f08ff0c	Fix broken clang-tidy (#31666 )	2 years ago
Vignesh Babu	1e2af690f2	Add a check to flush stale TracedBuffer entries to prevent them growing too long. (#31588 ) * Add a check to flush stale TracedBuffer entries to prevent them growing too long * update * fix tsan and sanity * sanity * review comments	2 years ago
Esun Kim	26bc68c4af	Added readability-duplicate-include (#31608 ) * Added readability-duplicate-include * Fix * format	2 years ago
Cheng-Yu Chung	90d8754b0e	Sanitizer auto fix (#31582 )	2 years ago
Vignesh Babu	249645608e	Reland posix event engine tcp client (#31563 ) * Revert "Revert "Posix event engine tcp client (#31074)" (#31452)" This reverts commit `d760c1fef7`. * regenerate_projects	2 years ago
Esun Kim	50fbe51238	Fix IWYU with clang-15 (#31556 ) * Fix iwyu * iwyu fix * more fixed * Build fix * more fix * More fixes 3	2 years ago
Vignesh Babu	7eb5fb5a25	Util functions to help with posix event engine listener implementation (#31435 ) * Util functions to help with posix engine listener implementation * sanity * update comments in posix_engine_listener_utils.h * review comments * iwyu * revert prev commit * iwyu * update build * regenerate projects * minor fixes * comments	2 years ago
AJ Heller	2820734f47	Reland: "[EventEngine] Refactor TimerManager to leverage a shared ThreadPool (#31392 )" (#31482 ) * Revert "Revert "[EventEngine] Refactor TimerManager to leverage a shared ThreadPool (#31392)" (#31425)" This reverts commit `1cbf6d64ae`. * fix deadlock on windows engine shutdown	2 years ago
Vignesh Babu	d760c1fef7	Revert "Posix event engine tcp client (#31074 )" (#31452 ) This reverts commit `390ae5d412`.	2 years ago
Vignesh Babu	390ae5d412	Posix event engine tcp client (#31074 ) * removing EndpointConfig::Get method * Automated change: Fix sanity tests * cleanup * fix * fix review comments * review comments * cleanup * update comments * fix * cleanup * update comments * Automated change: Fix sanity tests * fix misleading comments * bug fixes * fix * fix * start * revert some changes for bug fixes and allow spurious wakeups for poll based poller * sanity * fix * minor fix * review comments * fix * comment * remove re-defined function * fix review comments * fix windows iocp build issue due to removed function * change Milliseconds return type * fix * remove header * regenerate projects * fix sanity * code * fix sanity * Automated change: Fix sanity tests * Forking iomgr internal_errqueue defines for posix event engine * fix BUILD file * start * update * sanity * regenerate_projects * Automated change: Fix sanity tests * rename * review comments * update * review comments * Automated change: Fix sanity tests * fix BUILD * add no_windows tag * update * update * remove debug statements * delete unused file * update * fix BUILD * regenerate projects * update * fix BUILD * update * fix sanity * minor additions to configure global tcp user timeout values * cleanup * fix build syntax * start * update * fix * change poller interface for posix event engine * update event_poller_posix_test * fix build error * Add more unit tests * windows ee changes * update comment * remove unused deps * fix msan issue * add comment * adding rcv lowat helper to socket wrapper * fix sanity * review comments * fix build issue * fix sanity * sanity * iwyu sanity * fix build * merge conflict * Automated change: Fix sanity tests * update * update * fix macos issue * fix msan and ubsan issues * fix macos build issue * comment * review comments * remove unused dep * sanity * merge conflict * mac build issue * regenerate projects * build issue * fix test issues * add test * fix typo and address review comments * review comments * Automated change: Fix sanity tests * update * merge issue * add a test to check connect timeouts and cancellation * regenerate projects * sanity * remove unnecessary code * Add utils to create and prepare socket for tcp client * some fixes * fixes * review comments * sanity * fix build issue * review comments * add thread annotations * MacOS fix unused variable * add some more #endif comments * fixes * update * more fixes * regenerate projects * sanity * some fixes * more fixes * rm hdr file * macos fixes * typo * more build issues * macos crashes * windows build * adding Poller::Work to event engine constructor * update client_test * iwyu * iwyu * regenerate projects * cleanup * cleanup * regenerate projects * regenerate projects * fix * fix * merge conflict * comments * add experiments flag * fix * newline * newline * iwyu * fix * fix * remove some ifdefs * fix tsan issue in bm_event_engine_run * sanity * sanity * Add a quiesce to threadpool * fix * Automated change: Fix sanity tests * fixes * typo * comments * fix build * fixes * macos fixes * more macos fixes * handle shutdown better * updated poller kicked api contract and fix bug * iwyu * fix sanity Co-authored-by: Vignesh2208 <Vignesh2208@users.noreply.github.com> Co-authored-by: Craig Tiller <ctiller@google.com> Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
AJ Heller	1cbf6d64ae	Revert "[EventEngine] Refactor ThreadManager to leverage a shared ThreadPool (#31392 )" (#31425 ) * Revert "[EventEngine] Refactor ThreadManager to leverage a shared ThreadPool (#31392)" This reverts commit `66df2c646a`. * sanitize	2 years ago
Yash Tibrewal	5f6c357e74	Move some targets from //: to //src/core: (#31402 ) * temp * temp * temp * top BUILD builds * everything builds * buildify * temp fix_build_deps change * fix gpr_atm * additional changes * fix build files * nofixdeps * Generate projects * json_args * fix context * Generate projects * Sprinkle some avoid_dep tags * Run fix_build_deps * Remove additional avoid_dep * Revert some targets * grpc_mock_cel	2 years ago
AJ Heller	66df2c646a	[EventEngine] Refactor ThreadManager to leverage a shared ThreadPool (#31392 ) * Refactor ThreadManager to leverage a shared ThreadPool This is a step towards a work-stealing thread pool implementation: unifying thread pools so that work stealing affects timer callbacks as well. A subsequent step will expose the timeout logic so that the timer wakeup and check can be triggered externally (by pollers, in the common case). * fix atomic uint64_t type missing on some platforms * sanitize + platform fixes * ->quiesce * shut down the timer manager to release the main thread * roll back atomics * use a dedicated thread for timer_manager to prevent local execution (work stealing) * drain the pools after timer manager tests; sanitize * iwyu * reintroduce fork handling * sanitize * fix	2 years ago
Vignesh Babu	e8c30b487f	Invoking Poller::Work in posix event engine constructor (#31128 ) * adding Poller::Work to event engine constructor * iwyu * iwyu * cleanup * regenerate projects * fix * comments * add experiments flag * fix * newline * newline * iwyu * remove some ifdefs * fix tsan issue in bm_event_engine_run * Add a quiesce to threadpool * fix * Automated change: Fix sanity tests * disable posix_event_engine_test on windows Co-authored-by: Craig Tiller <ctiller@google.com> Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Yash Tibrewal	e8b416e654	Add a BUILD file in src/core (#31294 ) * Initial * Modifying fix_build_deps.py * Move event_engine_common as a better signal * fixes * fixes * Regenerate projects * Export headers for event_engine_common * Regenerate projects * Fix build files * Some cleanup * Reviewer comments * Remove unnecessary blank line * Remove unnecessary visibility exception for event_engine_common	2 years ago
Mark D. Roth	f2c4795f4f	core tests: absl::make_unique -> std::make_unique (#31361 ) * core tests: absl::make_unique -> std::make_unique * iwyu	2 years ago
Craig Tiller	afa3a6c890	[event_engine] Fix for the EE/iomgr shutdown ordering problem (#31265 ) * Remove ResetDefaultEventEngine Now that it is a weak_ptr, there's no need to explicitly reset it. When the tracked shared_ptr is deleted, the weak_ptr will fail to lock, and a new default EventEngine will be created. * forget existing engine with FactoryReset * init/shutdown in event engine for now * fix * fix * fix windows deadlock * Automated change: Fix sanity tests * fix * better windows fix Co-authored-by: AJ Heller <hork@google.com> Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Vignesh Babu	e7ad460c7b	Event Engine related minor cleanup (#31188 ) * Minor event engine related cleanup * regenerate_projects * fix test * iwyu	2 years ago
Craig Tiller	db5fe72da9	[tsan] Threadsafe strerror implementation (#31167 ) * fix * spread * fix * Automated change: Fix sanity tests * fix * remove unnecessary .c_strs * lashes * fix * Automated change: Fix sanity tests Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Vignesh Babu	7655858e93	Reland tcp endpoints for posix event engine (#31150 ) * Revert "Revert "Tcp endpoint implementation for posix event engine. (#30559)" (#31134)" This reverts commit `5fa37322cb`. * make tx zero-copy more robust * make tx zero-copy more robust * typo * fix * re-enable commented code * sanity * sanity again * Add parsing of /etc/security/limits.d directory * review comments * review comments * typo in copyright * missing newline * sanity * sanity * iwyu	2 years ago
Craig Tiller	2ec220a6b1	[sanity] Fix it (#31163 ) * fix sanity * fix sanity	2 years ago
Craig Tiller	8174a75079	[iwyu] Add more test directories (#30952 ) * [iwyu] Add more test directories * fix * fix * fix * fix * fix * Automated change: Fix sanity tests * fix * iwyu * Automated change: Fix sanity tests * fix * fix * fix * fix * Automated change: Fix sanity tests * fix Co-authored-by: ctiller <ctiller@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
AJ Heller	2ee2c91c92	Reland x2: Make GetDefaultEventEngine return a shared_ptr (#30619 ) * Reland x2: Make GetDefaultEventEngine return a shared_ptr * remove thread leak from NativeDNSResolver This is not going to work for resolvers that support cancellation. * give resolvers bounded lifetimes Some resolver own EventEngines. EventEngines cannot run off the end of the process since they have unjoined threads (problematic in a small set of environments). This gives resolvers bounded lifetimes, and allows replacement of resolvers without ASAN issues of deleting resolvers in active use (occurs in tests). * fix * fix windows * fix surface init test * fix * sanitize * use after move * the test must wait for the callback to be destroyed * windows fix: delete the resolver on iomgr shutdown, not before * Make TimerManager threads non-joinable On gRPC shutdown, any unjoined TimerManager threads will cause TSAN to detect thread leaks. This fix resolves issues I saw in end2end test shutdown in another PR, where a single timer manager thread was always alive after the test ended. The long-term solution is to integrate the new ThreadPool here, but this unblocks me for now. * backport fix * fix * shared_ptr<EventEngine> in EventEngine benchmarks	2 years ago
Craig Tiller	5fa37322cb	Revert "Tcp endpoint implementation for posix event engine. (#30559 )" (#31134 ) This reverts commit `a81391d931`.	2 years ago
Vignesh Babu	a81391d931	Tcp endpoint implementation for posix event engine. (#30559 ) * 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 * Migrating posix event pollers to use new event poller interface * add inline attributes * Automated change: Fix sanity tests * remove ref/unref from iomgr engine closure ad add custom closure types internal to the pollers * updating time util usage * use unique_ptrs * update comments * Automated change: Fix sanity tests * review comments * rename GetPointer to GetVoidPointer * cleanup * Automated change: Fix sanity tests * removing EndpointConfig::Get method * Automated change: Fix sanity tests * cleanup * fix * fix review comments * review comments * cleanup * update comments * fix * cleanup * update comments * Automated change: Fix sanity tests * fix misleading comments * bug fixes * fix * fix * start * revert some changes for bug fixes and allow spurious wakeups for poll based poller * sanity * fix * minor fix * review comments * fix * comment * remove re-defined function * fix review comments * fix windows iocp build issue due to removed function * change Milliseconds return type * fix * remove header * regenerate projects * fix sanity * code * fix sanity * Automated change: Fix sanity tests * Forking iomgr internal_errqueue defines for posix event engine * fix BUILD file * start * update * sanity * regenerate_projects * Automated change: Fix sanity tests * rename * review comments * update * review comments * Automated change: Fix sanity tests * fix BUILD * add no_windows tag * update * update * remove debug statements * delete unused file * update * fix BUILD * regenerate projects * update * fix BUILD * update * fix sanity * minor additions to configure global tcp user timeout values * cleanup * fix build syntax * change poller interface for posix event engine * update event_poller_posix_test * fix build error * Add more unit tests * windows ee changes * update comment * remove unused deps * fix msan issue * add comment * adding rcv lowat helper to socket wrapper * fix sanity * review comments * fix build issue * fix sanity * sanity * iwyu sanity * fix build * merge conflict * Automated change: Fix sanity tests * update * update * fix macos issue * fix msan and ubsan issues * fix macos build issue * comment * review comments * remove unused dep * sanity * merge conflict * mac build issue * regenerate projects * build issue * fix typo and address review comments * review comments * Automated change: Fix sanity tests * update * merge issue * review comments * sanity * fix build issue * review comments * add thread annotations * MacOS fix unused variable * add some more #endif comments * missing semi-colon * Automated change: Fix sanity tests * sanity * sanity again * sanity try again * sanity again * try again with no_include pragma Co-authored-by: Vignesh2208 <Vignesh2208@users.noreply.github.com>	2 years ago
AJ Heller	c0e5e35c7a	Replace EventEngine::Promise with grpc_core::Notification (#31027 ) * Replace EventEngine::Promise with grpc_core::Notification * remove promise.h * Automated change: Fix sanity tests * fix windows * fix iocp * fix client_test Co-authored-by: drfloob <drfloob@users.noreply.github.com>	2 years ago
Craig Tiller	1ad4bbe78e	Revert "fix (#31018 )" (#31021 ) This reverts commit `2e6f268dfc`.	2 years ago
Craig Tiller	2e6f268dfc	fix (#31018 )	2 years ago
Craig Tiller	96f5cddb5f	[time] Introduce time sources (#30815 ) * [time] Introduce time sources * make import trivial * Automated change: Fix sanity tests * fix * Automated change: Fix sanity tests * fix * Automated change: Fix sanity tests * Automated change: Fix sanity tests * review feedback * fix * Automated change: Fix sanity tests * ios fix * fix Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago
Vignesh Babu	4e7f0e1eac	Update to the event engine poller interface (#30828 ) * change poller interface for posix event engine * update event_poller_posix_test * windows ee changes * update comment * remove unused deps * review comments * fix build issue * fix sanity * fix build * Automated change: Fix sanity tests * review comments * remove unused dep * sanity * fix typo and address review comments * review comments * Automated change: Fix sanity tests * fix typo in comment Co-authored-by: Vignesh2208 <Vignesh2208@users.noreply.github.com>	2 years ago
Craig Tiller	14c3839749	[event_engine] More robust ee::Promise<> implementation (#30977 ) * fix promise * iocp * iwyu * Automated change: Fix sanity tests Co-authored-by: ctiller <ctiller@users.noreply.github.com>	2 years ago

1 2

66 Commits (0003c320f40735fbb713125cea1be58e3eabef20)