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

Commit Graph

Author	SHA1	Message	Date
Sourabh Singh	14d15834b9	[grpc][Python][doc] Added CONTRIBUTING.md to guide new contributors through the process of contributing to gRPC Python (#36749 ) This PR introduces a new CONTRIBUTING.md file to the gRPC Python repository. The goal of this document is to provide clear and comprehensive guidelines for new contributors who are interested in contributing to the project. - Code of Conduct: Establishes expectations for respectful and inclusive interactions within the community. - Guidelines: Offers advice and best practices for contributing effectively, including how to find issues, get started with the codebase, and follow the project's standards. - Ways to Contribute: Highlights the diverse ways in which people can contribute beyond just code, including documentation, community support, and testing. - Steps to Contributing: Provides a step-by-step guide on how to set up a development environment, make changes, run tests, and submit a pull request. - Code Style: Outlines the coding conventions and tools used in the project to ensure consistency. Closes #36749 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/36749 from sourabhsinghs:documentation/python-contributing-guide `bcd1123742` PiperOrigin-RevId: 681889662	2 months ago
Yousuk Seung	a422c8ffe3	[experiments] retry work_serializer_dispatch with posix (#37839 ) All known flakes/bugs have been fixed. <!-- 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. --> Closes #37839 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37839 from yousukseung:work_serializer_dispatch `a82a068ebf` PiperOrigin-RevId: 681609545	2 months ago
Kukki	7c7a979ab2	Added static Type method to some credential types (#37795 ) * grpc_google_default_channel_credentials * TlsServerCredentials * grpc_alts(_server)_credentials * UrlExternalAccountCredentials * FileExternalAccountCredentials * AwsExternalAccountCredentials <!-- 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. --> Closes #37795 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37795 from AlienCookie:patch-1 `34ffa5eca5` PiperOrigin-RevId: 681554756	2 months ago
Yash Tibrewal	d87165acf4	[chttp2_server] Fix race between connection starting and it being orphaned (#37683 ) Sample race - https://btx.cloud.google.com/invocations/0c4e65f2-3a38-4b4f-b67e-c53a4a4650ea/targets/%2F%2Ftest%2Fcore%2Fend2end:connectivity_test@poller%3Dpoll;config=2aed862ff4fd4384687d63aa95df415c7cb955355c2ab6dc6c6d7a9d123a76ec/log ``` WARNING: ThreadSanitizer: data race (pid=18) Write of size 8 at 0x72300000c318 by thread T29: #0 grpc_core::Chttp2ServerListener* std::__exchange(grpc_core::Chttp2ServerListener&, grpc_core::Chttp2ServerListener&) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/move.h:152:13 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x68c85) #1 grpc_core::Chttp2ServerListener* std::exchange(grpc_core::Chttp2ServerListener&, grpc_core::Chttp2ServerListener&) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/utility:287:14 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x68c05) #2 grpc_core::RefCountedPtr::reset(grpc_core::Chttp2ServerListener) /proc/self/cwd/./src/core/lib/gprpp/ref_counted_ptr.h:126:20 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x68b32) #3 grpc_core::RefCountedPtr::operator=(grpc_core::RefCountedPtr&&) /proc/self/cwd/./src/core/lib/gprpp/ref_counted_ptr.h:66:5 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x54380) #4 grpc_core::Chttp2ServerListener::ActiveConnection::Start(grpc_core::RefCountedPtr, std::unique_ptr, grpc_core::ChannelArgs const&) /proc/self/cwd/src/core/ext/transport/chttp2/server/chttp2_server.cc:615:13 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x48914) #5 grpc_core::Chttp2ServerListener::OnAccept(void, grpc_endpoint, grpc_pollset, grpc_tcp_server_acceptor) /proc/self/cwd/src/core/ext/transport/chttp2/server/chttp2_server.cc:881:21 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x49ce2) #6 CreateEventEngineListener(grpc_tcp_server, grpc_closure, grpc_event_engine::experimental::EndpointConfig const&, grpc_tcp_server)::$_2::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const /proc/self/cwd/src/core/lib/iomgr/tcp_server_posix.cc:228:11 (liblibiomgr.so+0xef627) #7 decltype(std::declval()(std::declval>>(), std::declval())) absl::lts_20240116::base_internal::Callable::Invoke>, grpc_event_engine::experimental::MemoryAllocator>(CreateEventEngineListener(grpc_tcp_server, grpc_closure, grpc_event_engine::experimental::EndpointConfig const&, grpc_tcp_server)::$_2&, std::unique_ptr>&&, grpc_event_engine::experimental::MemoryAllocator&&) /proc/self/cwd/external/com_google_absl/absl/base/internal/invoke.h:185:12 (liblibiomgr.so+0xef3c2) #8 decltype(Invoker>, grpc_event_engine::experimental::MemoryAllocator>::type::Invoke(std::declval(), std::declval>>(), std::declval())) absl::lts_20240116::base_internal::invoke>, grpc_event_engine::experimental::MemoryAllocator>(CreateEventEngineListener(grpc_tcp_server, grpc_closure, grpc_event_engine::experimental::EndpointConfig const&, grpc_tcp_server)::$_2&, std::unique_ptr>&&, grpc_event_engine::experimental::MemoryAllocator&&) /proc/self/cwd/external/com_google_absl/absl/base/internal/invoke.h:212:10 (liblibiomgr.so+0xef325) #9 void absl::lts_20240116::internal_any_invocable::InvokeR>, grpc_event_engine::experimental::MemoryAllocator, void>(CreateEventEngineListener(grpc_tcp_server, grpc_closure, grpc_event_engine::experimental::EndpointConfig const&, grpc_tcp_server)::$_2&, std::unique_ptr>&&, grpc_event_engine::experimental::MemoryAllocator&&) /proc/self/cwd/external/com_google_absl/absl/functional/internal/any_invocable.h:132:3 (liblibiomgr.so+0xef2b5) #10 void absl::lts_20240116::internal_any_invocable::LocalInvoker>, grpc_event_engine::experimental::MemoryAllocator>(absl::lts_20240116::internal_any_invocable::TypeErasedState, absl::lts_20240116::internal_any_invocable::ForwardedParameter>>::type, absl::lts_20240116::internal_any_invocable::ForwardedParameter::type) /proc/self/cwd/external/com_google_absl/absl/functional/internal/any_invocable.h:310:10 (liblibiomgr.so+0xef1e2) #11 absl::lts_20240116::internal_any_invocable::Impl>, grpc_event_engine::experimental::MemoryAllocator)>::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) /proc/self/cwd/external/com_google_absl/absl/functional/internal/any_invocable.h:868:1 (libsrc_Score_Slibposix_Uevent_Uengine.so+0xa754f) #12 grpc_event_engine::experimental::ThreadyEventEngine::CreateListener(absl::lts_20240116::AnyInvocable>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()::operator()() /proc/self/cwd/src/core/lib/event_engine/thready_event_engine/thready_event_engine.cc:61:15 (libsrc_Score_Slibthready_Uevent_Uengine.so+0x27cdb) #13 decltype(std::declval>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&>()()) absl::lts_20240116::base_internal::Callable::Invoke>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&>(grpc_event_engine::experimental::ThreadyEventEngine::CreateListener(absl::lts_20240116::AnyInvocable>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&) /proc/self/cwd/external/com_google_absl/absl/base/internal/invoke.h:185:12 (libsrc_Score_Slibthready_Uevent_Uengine.so+0x27c45) #14 decltype(Invoker>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&>::type::Invoke(std::declval>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&>())) absl::lts_20240116::base_internal::invoke>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&>(grpc_event_engine::experimental::ThreadyEventEngine::CreateListener(absl::lts_20240116::AnyInvocable>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&) /proc/self/cwd/external/com_google_absl/absl/base/internal/invoke.h:212:10 (libsrc_Score_Slibthready_Uevent_Uengine.so+0x27bf5) #15 void absl::lts_20240116::internal_any_invocable::InvokeR>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&, void>(grpc_event_engine::experimental::ThreadyEventEngine::CreateListener(absl::lts_20240116::AnyInvocable>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&) /proc/self/cwd/external/com_google_absl/absl/functional/internal/any_invocable.h:132:3 (libsrc_Score_Slibthready_Uevent_Uengine.so+0x27ba5) #16 void absl::lts_20240116::internal_any_invocable::RemoteInvoker>, grpc_event_engine::experimental::MemoryAllocator)>, absl::lts_20240116::AnyInvocable, grpc_event_engine::experimental::EndpointConfig const&, std::unique_ptr>)::$_0::operator()(std::unique_ptr>, grpc_event_engine::experimental::MemoryAllocator) const::'lambda'()&>(absl::lts_20240116::internal_any_invocable::TypeErasedState) /proc/self/cwd/external/com_google_absl/absl/functional/internal/any_invocable.h:368:10 (libsrc_Score_Slibthready_Uevent_Uengine.so+0x279cd) #17 absl::lts_20240116::internal_any_invocable::Impl::operator()() /proc/self/cwd/external/com_google_absl/absl/functional/internal/any_invocable.h:868:1 (libtest_Score_Send2end_Slibconnectivity_Ulibrary.so+0x337ff) #18 grpc_core::Thread::Thread(char const, absl::lts_20240116::AnyInvocable, bool, grpc_core::Thread::Options const&)::'lambda'(void)::operator()(void) const /proc/self/cwd/./src/core/lib/gprpp/thd.h:108:15 (libsrc_Score_Slibthready_Uevent_Uengine.so+0x2e264) #19 grpc_core::Thread::Thread(char const, absl::lts_20240116::AnyInvocable, bool, grpc_core::Thread::Options const&)::'lambda'(void)::__invoke(void) /proc/self/cwd/./src/core/lib/gprpp/thd.h:105:13 (libsrc_Score_Slibthready_Uevent_Uengine.so+0x2e1e9) #20 grpc_core::(anonymous namespace)::ThreadInternalsPosix::ThreadInternalsPosix(char const, void ()(void), void, bool, grpc_core::Thread::Options const&)::'lambda'(void)::operator()(void) const /proc/self/cwd/src/core/lib/gprpp/posix/thd.cc:148:11 (liblibgpr.so+0x1d830) #21 grpc_core::(anonymous namespace)::ThreadInternalsPosix::ThreadInternalsPosix(char const, void ()(void), void, bool, grpc_core::Thread::Options const&)::'lambda'(void)::__invoke(void) /proc/self/cwd/src/core/lib/gprpp/posix/thd.cc:118:9 (liblibgpr.so+0x1d659) Previous read of size 8 at 0x72300000c318 by main thread: #0 grpc_core::RefCountedPtr::operator!=(std::nullptr_t) const /proc/self/cwd/./src/core/lib/gprpp/ref_counted_ptr.h:192:50 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x52345) #1 grpc_core::Chttp2ServerListener::ActiveConnection::HandshakingState::~HandshakingState() /proc/self/cwd/src/core/ext/transport/chttp2/server/chttp2_server.cc:394:30 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x463ed) #2 std::enable_if::value, grpc_core::Chttp2ServerListener::ActiveConnection::HandshakingState>::type grpc_event_engine::experimental::MemoryAllocator::New, grpc_pollset&, std::unique_ptr, grpc_core::ChannelArgs const&>(grpc_core::RefCountedPtr&&, grpc_pollset&, std::unique_ptr&&, grpc_core::ChannelArgs const&)::Wrapper::~Wrapper() /proc/self/cwd/include/grpc/event_engine/memory_allocator.h:117:65 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x67a1c) #3 std::enable_if::value, grpc_core::Chttp2ServerListener::ActiveConnection::HandshakingState>::type grpc_event_engine::experimental::MemoryAllocator::New, grpc_pollset&, std::unique_ptr, grpc_core::ChannelArgs const&>(grpc_core::RefCountedPtr&&, grpc_pollset&, std::unique_ptr&&, grpc_core::ChannelArgs const&)::Wrapper::~Wrapper() /proc/self/cwd/include/grpc/event_engine/memory_allocator.h:117:27 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x67a59) #4 void grpc_core::UnrefDelete::operator()(grpc_core::Chttp2ServerListener::ActiveConnection::HandshakingState) const /proc/self/cwd/./src/core/lib/gprpp/ref_counted.h:224:5 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x6449d) #5 grpc_core::InternallyRefCounted::Unref() /proc/self/cwd/./src/core/lib/gprpp/orphanable.h:132:7 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x52581) #6 grpc_core::Chttp2ServerListener::ActiveConnection::HandshakingState::Orphan() /proc/self/cwd/src/core/ext/transport/chttp2/server/chttp2_server.cc:407:3 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x466ac) #7 void grpc_core::OrphanableDelete::operator()(grpc_core::Chttp2ServerListener::ActiveConnection::HandshakingState) /proc/self/cwd/./src/core/lib/gprpp/orphanable.h:60:8 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x653b1) #8 std::unique_ptr::~unique_ptr() /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/unique_ptr.h:292:4 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x53ddf) #9 grpc_core::Chttp2ServerListener::ActiveConnection::Orphan() /proc/self/cwd/src/core/ext/transport/chttp2/server/chttp2_server.cc:581:1 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x487db) #10 void grpc_core::OrphanableDelete::operator()(grpc_core::Chttp2ServerListener::ActiveConnection) /proc/self/cwd/./src/core/lib/gprpp/orphanable.h:60:8 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x66b01) #11 std::unique_ptr::~unique_ptr() /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/unique_ptr.h:292:4 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x53d1f) #12 std::pair>::~pair() /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_pair.h:208:12 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x597f9) #13 void __gnu_cxx::new_allocator>>>::destroy>>(std::pair>) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/ext/new_allocator.h:152:10 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x597c1) #14 void std::allocator_traits>>>>::destroy>>(std::allocator>>>&, std::pair>) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/alloc_traits.h:496:8 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x59725) #15 std::_Rb_tree>, std::_Select1st>>, std::less, std::allocator>>>::_M_destroy_node(std::_Rb_tree_node>>) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_tree.h:642:2 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x59674) #16 std::_Rb_tree>, std::_Select1st>>, std::less, std::allocator>>>::_M_drop_node(std::_Rb_tree_node>>) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_tree.h:650:2 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x595f9) #17 std::_Rb_tree>, std::_Select1st>>, std::less, std::allocator>>>::_M_erase(std::_Rb_tree_node>>) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_tree.h:1920:4 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x5945f) #18 std::_Rb_tree>, std::_Select1st>>, std::less, std::allocator>>>::~_Rb_tree() /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_tree.h:1000:9 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x593c5) #19 std::map, std::less, std::allocator>>>::~map() /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_map.h:300:22 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x51525) #20 grpc_core::Chttp2ServerListener::Orphan() /proc/self/cwd/src/core/ext/transport/chttp2/server/chttp2_server.cc:923:1 (libsrc_Score_Slibgrpc_Utransport_Uchttp2_Userver.so+0x4b5dd) #21 void grpc_core::OrphanableDelete::operator()(grpc_core::Server::ListenerInterface) /proc/self/cwd/./src/core/lib/gprpp/orphanable.h:60:8 (libsrc_Score_Slibchaotic_Ugood_Userver.so+0x1d0981) #22 std::unique_ptr::reset(grpc_core::Server::ListenerInterface) /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/unique_ptr.h:402:4 (liblibserver.so+0x1d5c21) #23 grpc_core::Server::StopListening() /proc/self/cwd/src/core/server/server.cc:1211:23 (liblibserver.so+0x1b9c62) #24 grpc_core::Server::ShutdownAndNotify(grpc_completion_queue, void) /proc/self/cwd/src/core/server/server.cc:1195:3 (liblibserver.so+0x1b97da) #25 grpc_server_shutdown_and_notify /proc/self/cwd/src/core/server/server.cc:1829:37 (liblibserver.so+0x1bf212) #26 grpc_core::CoreEnd2endTest::ShutdownServerAndNotify(int) /proc/self/cwd/./test/core/end2end/end2end_tests.h:459:5 (libtest_Score_Send2end_Slibconnectivity_Ulibrary.so+0x33370) #27 grpc_core::(anonymous namespace)::CoreEnd2endTest_RetryHttp2Test_ConnectivityWatch::RunTest() /proc/self/cwd/test/core/end2end/tests/connectivity.cc:74:3 (libtest_Score_Send2end_Slibconnectivity_Ulibrary.so+0x2ee8d) #28 grpc_core::(anonymous namespace)::CoreEnd2endTest_RetryHttp2Test_ConnectivityWatch::TestBody() /proc/self/cwd/test/core/end2end/tests/connectivity.cc:32:1 (libtest_Score_Send2end_Slibconnectivity_Ulibrary.so+0x2dc96) #29 void testing::internal::HandleSehExceptionsInMethodIfSupported(testing::Test, void (testing::Test::)(), char const) /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2612:10 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x16d2dc) #30 void testing::internal::HandleExceptionsInMethodIfSupported(testing::Test, void (testing::Test::)(), char const) /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2648:14 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x14a51d) #31 testing::Test::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2687:5 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x120458) #32 testing::TestInfo::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2836:11 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x1215f3) #33 testing::TestSuite::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:3015:30 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x12230c) #34 testing::internal::UnitTestImpl::RunAllTests() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:5921:44 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x138142) #35 bool testing::internal::HandleSehExceptionsInMethodIfSupported(testing::internal::UnitTestImpl, bool (testing::internal::UnitTestImpl::)(), char const) /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2612:10 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x17508f) #36 bool testing::internal::HandleExceptionsInMethodIfSupported(testing::internal::UnitTestImpl, bool (testing::internal::UnitTestImpl::)(), char const*) /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2648:14 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x14e5b3) #37 testing::UnitTest::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:5485:10 (libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so+0x13795b) #38 RUN_ALL_TESTS() /proc/self/cwd/external/com_google_googletest/googletest/include/gtest/gtest.h:2316:73 (libtest_Score_Send2end_Slibend2end_Utest_Umain.so+0x8457) #39 main /proc/self/cwd/test/core/end2end/end2end_test_main.cc:50:10 (libtest_Score_Send2end_Slibend2end_Utest_Umain.so+0x77b6) ``` We start the connection outside the critical region and that's where we supply the listener ref to the connection. There is a freak case where the connection can be orphaned due to the listener stopping to serve and the `Orphan()` would also be trying to access the listener ref resulting in a race. Closes #37683 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37683 from yashykt:FixChttp2ServerRace `e3c452984f` PiperOrigin-RevId: 681552145	2 months ago
Craig Tiller	adc7affc2d	[resource-quota] Add metrics for number of calls, connections dropped due to quota exceeded (#37833 ) Closes #37833 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37833 from ctiller:statty `f049916086` PiperOrigin-RevId: 681236201	2 months ago
Craig Tiller	863aca8949	[call-v3] Implement failed_before_recv_message() on {Client,Server}Call (#37810 ) Closes #37810 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37810 from ctiller:CANCELLED `134f49a07b` PiperOrigin-RevId: 681236011	2 months ago
Craig Tiller	d6fd0bd991	[chaotic-good] Annotate key promises for latent-see (#37830 ) Closes #37830 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37830 from ctiller:promising-latent-see `4f7dd1b7c6` PiperOrigin-RevId: 681228010	2 months ago
Mark D. Roth	cd129b49c2	[GCP auth filter] hold ref to service config (#37831 ) This fixes the following crash: https://btx.cloud.google.com/invocations/3a7065d4-db7f-4f01-a239-5376b7f5ee8b/targets/%2F%2Ftest%2Fcpp%2Fend2end%2Fxds:xds_gcp_authn_end2end_test@experiment%3Dwork_serializer_dispatch;config=1a7dc092b28796b045d00aec96c95b85c1d4dc656912e0021a1fc84b3ecb2ac9/log The problem is caused by a race whereby the channel swaps out the service config due to a resolver update while the old dynamic filter stack is still processing calls in another thread. The GCP auth filter was dereferencing the old service config but not holding a ref to it. I've fixed this by having it hold a ref. In the long run, I suspect that we may run into other cases like this, in which case we may want the dynamic filter stack itself to hold a ref to the service config, so that individual filters don't have to. Closes #37831 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37831 from markdroth:gcp_auth_race_fix `f2a0d1dd79` PiperOrigin-RevId: 681221795	2 months ago
Yash Tibrewal	b44de399c1	[Chttp2Server] Remove unnecessary ExecCtx Flush (#37824 ) This seems to be a vestige from around 8 years ago. The code has changed a lot since then, and this makes no sense anymore. (Making this change separately to be safe.) Closes #37824 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37824 from yashykt:RemoveUnnecessaryExecCtxStuff `97a7625033` PiperOrigin-RevId: 681184980	2 months ago
Mark D. Roth	9a12ec91e1	[xds_client_test] change test to use FuzzingEventEngine (#37668 ) Closes #37668 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37668 from markdroth:xds_client_test_fuzzing_ee `e1fe7f56c2` PiperOrigin-RevId: 681150007	2 months ago
Yash Tibrewal	74e640341d	[CSM O11y] Example logs (#37823 ) Closes #37823 PiperOrigin-RevId: 681149244	2 months ago
Arjan Singh Bal	c938a7b564	[interop] Remove python dependencies from Go dockerfile and add v1.67.0 release (#37792 ) <!-- 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. --> ## Why Thanks to @ejona86 for finding this. The python dependencies were wrongly added in `0589e533cd`. And from then on people were just maintaining what was there. Java had it then too. Java removed the extra dockerfile steps at `977e4b1450` ## Testing Passing master branch test: https://source.cloud.google.com/results/invocations/79fc6c6f-c3d9-4692-870d-4d7002f4778a Passing ad-hoc interop matrix build: https://source.cloud.google.com/results/invocations/683d7487-2637-4a69-b65e-95dc82740679 Closes #37792 PiperOrigin-RevId: 681054405	2 months ago
Tanvi Jagtap	229de2530b	[PH2][Refactor] Creating call_tracer_wrapper.h . (#37786 ) [PH2][Refactor] Creating call_tracer_wrapper.h and cc and moving code as-is into it. The content of the functions or classes has not changed. In the next few iterations, more code will come into this file. Closes #37786 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37786 from tanvi-jagtap:ph2_internal_h_split_01 `31af14a4e7` PiperOrigin-RevId: 680861154	2 months ago
Yijie Ma	80b57c909b	[call_creds_util] Add debug checks for metadata accesses (#37822 ) Fix https://github.com/grpc/grpc/issues/37701 <!-- 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. --> Closes #37822 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37822 from yijiem:debug-check-metadata `67df0da89c` PiperOrigin-RevId: 680734161	2 months ago
Esun Kim	daa793860c	[Build] Minimum version of MSVC is now 2022 (#37687 ) Per https://github.com/google/oss-policies-info/blob/main/foundational-cxx-support-matrix.md, we can require msvc 2022 or later. Closes #37687 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37687 from veblush:msvc2022 `32e35ec5b5` PiperOrigin-RevId: 680682655	2 months ago
Esun Kim	a49d450a54	[Build] Bumped the minimum version of cmake (#37702 ) Per https://github.com/google/oss-policies-info/blob/main/foundational-cxx-support-matrix.md, the minimum version of cmake to support is 3.16 so let's change our cmake builds' requirements accordingly. Closes #37702 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37702 from veblush:cmake316 bb4ed1a1be5e9374980c922aac3dc1ccd27d1266 PiperOrigin-RevId: 680639191	2 months ago
Tanvi Jagtap	ff0c2e326f	[PH2][Refactor] Minor changes. 1. Rename write_state_name to get_state_name 2. Adding const where appropriate PiperOrigin-RevId: 680516861	2 months ago
Craig Tiller	331d42140e	[chaotic-good] Fix use after free (#37814 ) Closes #37814 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37814 from ctiller:flake-fightas-15 `cab1017eb7` PiperOrigin-RevId: 680418287	2 months ago
Craig Tiller	6b88612d03	[latent-see] Compilation fix for use_strict_warning=true (#37815 ) Since we don't actually use the `desc` parameter when building without latent_see we had an unused parameter error. More though, I think the `GRPC_LATENT_SEE_METADATA_RAW` macro was flawed, as the inner `metadata` argument would only be created once, and so all uses would get tagged by whatever passed through `Flusher` first. Correct this by passing in the `latent_see::Metadata*` directly to the function. Closes #37815 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37815 from ctiller:woah-bubba `606a6350fe` PiperOrigin-RevId: 680136826	2 months ago
Craig Tiller	3ea3e36e95	[tools] Add a `--fix` argument to check_submodules.sh (#37811 ) IDK how many times I've messed this up over the years accidentally and had to spend thoughts trying to fix it... when really we could do that trivially with code. Closes #37811 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37811 from ctiller:bamboozled `0cfbeae925` PiperOrigin-RevId: 679741188	2 months ago
Vignesh Babu	05d214ee86	[tracing] Add latent see annotations to the filter stack. This would allow us to better attribute time spent in different filters. PiperOrigin-RevId: 679739129	2 months ago
Esun Kim	1b11b92182	[Deps] Updated Abseil to 1.20240722.0 (#37543 ) Closes #37543 PiperOrigin-RevId: 679738302	2 months ago
Yousuk Seung	f1df0f2b5f	[doc] Clarify client callback API comments (#37805 ) Comments say users "should not" call Start* methods after notification callbacks are called with false, but users may still call so it's close to 'not recommended'. But it's less confusing to remove it altogether. <!-- 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. --> Closes #37805 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37805 from yousukseung:client_callback_doc_clarify `38b9371f79` PiperOrigin-RevId: 679709417	2 months ago
Mark D. Roth	e56d766a45	[GCP auth filter] add "blackboard" mechanism for retaining global state, and use it for cache in GCP auth filter (#37646 ) This is the last piece of gRFC A83 (https://github.com/grpc/proposal/pull/438). Note that although this is the first use-case for this "blackboard" mechanism, we will also use it in the future for the xDS rate-limiting filter on the gRPC server side. Closes #37646 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37646 from markdroth:gcp_auth_filter_state `72d0d96c79` PiperOrigin-RevId: 679707134	2 months ago
Craig Tiller	6dbc0bdb10	[chaotic-good] Use a party for internal activities (#37078 ) Allows use of the party <-> party wakeup batching stuff, which reduces threadhops drastically for this transport. Closes #37078 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37078 from ctiller:chaotic-party-3 `75c32e6a64` PiperOrigin-RevId: 679685211	2 months ago
Craig Tiller	40ff7aa617	[call-v3] Inline OperationExecutor methods (#37006 ) These methods are very short and very inlinable. Further more the specializations are very infrequently repeated. (I'm essentially hoisting code out of the .cc file and landing it in the .h file with the appropriate inlining annotations here) Before: ``` ---------------------------------------------------------------------------------------------------- Benchmark Time CPU Iterations ---------------------------------------------------------------------------------------------------- grpc_core::BM_UnaryWithSpawnPerEnd<CallSpineFixture> 1053 ns 1053 ns 2674404 grpc_core::BM_UnaryWithSpawnPerOp<CallSpineFixture> 1191 ns 1191 ns 2337068 grpc_core::BM_ClientToServerStreaming<CallSpineFixture> 243 ns 243 ns 11511112 grpc_core::BM_UnaryWithSpawnPerEnd<ForwardCallFixture> 2723 ns 2724 ns 1033990 grpc_core::BM_UnaryWithSpawnPerOp<ForwardCallFixture> 2930 ns 2931 ns 942815 grpc_core::BM_ClientToServerStreaming<ForwardCallFixture> 455 ns 455 ns 6213422 ``` After: ``` ---------------------------------------------------------------------------------------------------- Benchmark Time CPU Iterations ---------------------------------------------------------------------------------------------------- grpc_core::BM_UnaryWithSpawnPerEnd<CallSpineFixture> 986 ns 986 ns 2867870 grpc_core::BM_UnaryWithSpawnPerOp<CallSpineFixture> 1101 ns 1101 ns 2472611 grpc_core::BM_ClientToServerStreaming<CallSpineFixture> 220 ns 220 ns 12774397 grpc_core::BM_UnaryWithSpawnPerEnd<ForwardCallFixture> 2512 ns 2512 ns 1111667 grpc_core::BM_UnaryWithSpawnPerOp<ForwardCallFixture> 2743 ns 2744 ns 1008676 grpc_core::BM_ClientToServerStreaming<ForwardCallFixture> 420 ns 420 ns 6820805 ``` Closes #37006 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37006 from ctiller:inline-filters `24903bd524` PiperOrigin-RevId: 679666043	2 months ago
Mark D. Roth	6ba3b4a717	[xDS] ORCA to LRS propagation changes (#37467 ) Implements gRFC A85 (https://github.com/grpc/proposal/pull/454). Closes #37467 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37467 from markdroth:orca_lrs_propagation_changes `0c1e889bb7` PiperOrigin-RevId: 679646192	2 months ago
Sreenithi Sridharan	f2e3636303	increased timeout for armv7 artifact build to 2 hours (#37807 ) Building armv7 artifacts after adding Python 3.13 is failing with a timeout. Hence increasing timeout to 2 hours. Closes #37807 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37807 from sreenithi:increase_armv7_artifact_build_timeout `60034378f5` PiperOrigin-RevId: 679608393	2 months ago
Sourabh Singh	992b3aea8a	remove logging_threshold tests (#37806 ) Closes #37806 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37806 from sourabhsinghs:unittest/logging-threshold `d501d38dc0` PiperOrigin-RevId: 679604228	2 months ago
Craig Tiller	2030231e32	[gcc7] Build fix (#37796 ) Closes #37796 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37796 from ctiller:buildy `363eca118a` PiperOrigin-RevId: 679243010	2 months ago
Tanvi Jagtap	f6f13ccaec	[PH2][Refactor][internal.h] Split file. Add stream_lists.h (#37775 ) Moving out related code into a separate file. `internal.h -> stream_lists.h` Closes #37775 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37775 from tanvi-jagtap:ph2_stream_lists_h `13b6030ceb` PiperOrigin-RevId: 679205824	2 months ago
Sreenithi Sridharan	f2ffbb89a2	Add templating and support for Python 3.13 (#37643 ) This PR adds templating for Python versions and updates the maximum supported Python version to 3.13. The following major changes related to templating are added: - Minimum supported Python version and list of supported versions in `setup.py` are fetched using new template generated files called `python_version.py` - Dockerfiles for the different Python Linux builds are now template generated. - The "Supported Python Versions" section from READMEs of ancillary and main packages have been removed Note: All the `python_version.py` files and Linux build `Dockerfiles` except `tools/dockerfile/grpc_artifact_python_linux_armv7/Dockerfile` in the PR are generated from the respective templates. Further non-templated additions to add support for Python 3.13: - install scripts and artifacts for windows, macos and linux are added manually. Later, these can be templated as well. - updated cython bounds to 3.x - updated twine version to solve [cgi module import error](https://github.com/pypa/twine/issues/1046) - the twine update introduces a dependency on cryptography>=2.0. But the cryptography package doesn't support 32-bit Linux images and hence `twine check` has been disabled for x86 manylinux and x86 musllinux artifacts. Closes #37643 PiperOrigin-RevId: 678954495	2 months ago
Xuan Wang	ac31fb0f61	[Python example] Fix version in generated code in example folder (#37665 ) Fix: https://github.com/grpc/grpc/issues/37610 <!-- 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. -->	2 months ago
Paulo Castello da Costa	f188fe4587	[benchmark] Restore python to OSS benchmarks. (#37635 ) Closes #37635 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37635 from paulosjca:python `503f8ed5c8` PiperOrigin-RevId: 678757734	2 months ago
Sourabh Singh	dc12396125	Added excessive logging threshold test (#37274 ) * Added excessive logging threshold tests * Excessive logs sample: ``` Warning: Excessive error output detected (7 lines): WARNING: All log messages before absl::InitializeLog() is called are written to STDERR I0000 00:00:1724852923.140229 45 chttp2_transport.cc:1182] ipv6:%5B::1%5D:45545: Got goaway [2] err=UNAVAILABLE:GOAWAY received; Error code: 2; Debug Text: Cancelling all calls {file:"src/core/ext/transport/chttp2/transport/chttp2_transport.cc", file_line:1171, created_time:"2024-08-28T13:48:43.140192497+00:00", http2_error:2, grpc_status:14} ``` Closes #37274 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37274 from sourabhsinghs:unittest/logging-threshold `09a87693d0` PiperOrigin-RevId: 678742613	2 months ago
erm-g	72c7cf88f9	[security] Change security level classification for local connections (#37266 ) Set security level to NONE for LOCAL_TCP type, Privacy&Integrity for UDS. Closes #37266 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37266 from erm-g:local_creds `2f11125548` PiperOrigin-RevId: 678741239	2 months ago
Sourabh Singh	95e1ab6267	unit testing setup and unit-tests script (#37070 ) 1. Added script to install all dependencies then run all unit tests for python grpcio. Closes #37070 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37070 from sourabhsinghs:feature/developer-setup-script `954892f20a` PiperOrigin-RevId: 678718224	2 months ago
Tanvi Jagtap	2c09f14f0d	[Gpr_To_Absl_Logging] Remove allow listed log. It is not used anymore PiperOrigin-RevId: 678512344	2 months ago
Esun Kim	e4a422ba43	[Deps] Upgrade protobuf to 28.1 (#37682 ) Regular protobuf upgrade from 27.2 to 28.1 along with the required change to accommodate https://github.com/protocolbuffers/protobuf/pull/17285 Closes #37682 PiperOrigin-RevId: 678428936	2 months ago
Esun Kim	169ccf7a70	[CMake] Fixed _gRPC_PROTOBUF_WELLKNOWN_INCLUDE_DIR (#37627 ) gRPC has been passing the wrong directory of `_gRPC_PROTOBUF_WELLKNOWN_INCLUDE_DIR` to protoc. While protoc often works around this by searching its own installation directory for proto files, this isn't always successful. Let's fix this bug to ensure consistent behavior. Fixes: https://github.com/grpc/grpc/issues/37411 Closes #37627 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37627 from veblush:protobuf-cmake-wkt `412f4a6f87` PiperOrigin-RevId: 678428224	2 months ago
Kannan J	e11336112a	Docker files for xds example server and client for C++. (#37778 ) We need to create userguide examples for Xds not involving Observability and currently only the csm observability example has a docker image. Closes #37778 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37778 from kannanjgithub:docker_xds_example_cpp `bc91c8d5da` PiperOrigin-RevId: 678362755	2 months ago
Mark D. Roth	06d3e40d81	[xDS] split LRS client into its own API (#37605 ) Closes #37605 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37605 from markdroth:xds_client_lrs_refactor `f9b86b8f06` PiperOrigin-RevId: 678281548	2 months ago
Mark D. Roth	94b7927def	[doc] add version of C-core for xDS dualstack support (#37662 ) Closes #37662 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37662 from markdroth:xds_features_dualstack `f6b69e3048` PiperOrigin-RevId: 678277887	2 months ago
Tanvi Jagtap	97f22ce295	[PH2][Refactor][is_client] Split functions (#37641 ) [PH2][Refactor][is_client] Split two functions for better readability 1. send_initial_metadata_locked - Moved out the tracing part 2. grpc_chttp2_config_default_keepalive_args - Split the client and server path to reduce cyclomatic complexity. Closes #37641 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37641 from tanvi-jagtap:remove_is_client_02 `bfbc9361db` PiperOrigin-RevId: 678096501	2 months ago
Alisha Nanda	febe0f1458	Get name of connector from local connector rather than from handshaker args. PiperOrigin-RevId: 677992968	2 months ago
Craig Tiller	7379a9fbbe	[call-v3] Outbound request buffer (#37448 ) Basic building block for retries, hedging: buffer outgoing messages & metadata, allow for replay whilst buffered (with a single reader able to read once buffering ends) Closes #37448 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37448 from ctiller:once-again-into-the-breach-my-friends `79cb121054` PiperOrigin-RevId: 677959212	2 months ago
Eugene Ostroukhov	7f550b11b8	[experiments] Use proper delete for array deallocation (#37704 ) Fixes: #37693 Closes #37704 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37704 from grpc:eugeneo-delete-array `98e026c152` PiperOrigin-RevId: 677958660	2 months ago
Eugene Ostroukhov	e88ac072b7	[sample] Client-side flow control (#37604 ) Closes #37604 PiperOrigin-RevId: 677954309	2 months ago
Eugene Ostroukhov	08a777e658	[doc] Add xDS fallback to the feature matrix (#37676 ) Closes #37676 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37676 from grpc:eugeneo-xds-fallback-matrix `34cdf83b86` PiperOrigin-RevId: 677949315	2 months ago
Yijie Ma	c8765eb9c4	[OTel plugin] Do not crash when the gauge in `AddCallback` is different from the gauge in `Report` (#37764 ) Log error message instead of crashing for this API misuse. <!-- 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. --> Closes #37764 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37764 from yijiem:report-different-gauge-wont-crash `1b6e912bfc` PiperOrigin-RevId: 677944595	2 months ago

1 2 3 4 5 ...

55652 Commits (14d15834b96fef8158a08be1145bac5481ed7408) All Branches Search

55652 Commits (14d15834b96fef8158a08be1145bac5481ed7408)

All Branches