Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IpVersions/GolangIntegrationTest.DynamicMetadata_Async_Sleep/IPv6 flake #31654

Closed
alyssawilk opened this issue Jan 4, 2024 · 0 comments · Fixed by #32081
Closed

IpVersions/GolangIntegrationTest.DynamicMetadata_Async_Sleep/IPv6 flake #31654

alyssawilk opened this issue Jan 4, 2024 · 0 comments · Fixed by #32081
Assignees

Comments

@alyssawilk
Copy link
Contributor

https://dev.azure.com/cncf/envoy/_build/results?buildId=159064&view=logs&j=767be981-567e-57d8-68c3-2140ede0a0bd&t=2181edf2-f610-59f2-c43a-04bb9d0bca00&l=1032

[ RUN ] IpVersions/GolangIntegrationTest.DynamicMetadata_Async_Sleep/IPv6
[2024-01-04 19:42:03.814][86310][error][golang] [contrib/golang/common/log/cgo.cc:24] [http][basic] log test
[2024-01-04 19:42:03.814][86310][critical][golang] [contrib/golang/common/log/cgo.cc:27] [http][basic] log test
[2024-01-04 19:42:03.814][86310][error][golang] [contrib/golang/common/log/cgo.cc:24] log test
[2024-01-04 19:42:03.814][86310][critical][golang] [contrib/golang/common/log/cgo.cc:27] log test
[2024-01-04 19:42:03.814][86310][error][golang] [contrib/golang/common/log/cgo.cc:24] log test false
[2024-01-04 19:42:03.814][86310][critical][golang] [contrib/golang/common/log/cgo.cc:27] log test false
[2024-01-04 19:42:13.816][85526][critical][assert] [test/integration/http_integration.cc:560] assert failure: 0. Details: Timed out waiting for new connection.
[2024-01-04 19:42:13.816][85526][critical][backtrace] [./source/server/backtrace.h:104] Caught Aborted, suspect faulting address 0x3e900014e16
[2024-01-04 19:42:13.816][85526][critical][backtrace] [./source/server/backtrace.h:91] Backtrace (use tools/stack_decode.py to get line numbers):
[2024-01-04 19:42:13.816][85526][critical][backtrace] [./source/server/backtrace.h:92] Envoy version: 0/1.29.0-dev/test/RELEASE/BoringSSL
[2024-01-04 19:42:13.817][85526][critical][backtrace] [./source/server/backtrace.h:96] #0: __kernel_rt_sigreturn [0xffff926b38fc]
[2024-01-04 19:42:13.817][85526][critical][backtrace] [./source/server/backtrace.h:96] #1: abort [0xffff92428aac]
[2024-01-04 19:42:13.828][85526][critical][backtrace] [./source/server/backtrace.h:96] #2: Envoy::HttpIntegrationTest::waitForNextUpstreamConnection() [0xe73e58]
[2024-01-04 19:42:13.856][85526][critical][backtrace] [./source/server/backtrace.h:96] #3: Envoy::HttpIntegrationTest::waitForNextUpstreamRequest() [0xe735a8]
[2024-01-04 19:42:13.879][85526][critical][backtrace] [./source/server/backtrace.h:96] #4: Envoy::HttpIntegrationTest::waitForNextUpstreamRequest() [0xe740c0]
[2024-01-04 19:42:13.890][85526][critical][backtrace] [./source/server/backtrace.h:96] #5: Envoy::GolangIntegrationTest::testDynamicMetadata() [0xbcf408]
[2024-01-04 19:42:13.916][85526][critical][backtrace] [./source/server/backtrace.h:96] #6: Envoy::GolangIntegrationTest_DynamicMetadata_Async_Sleep_Test::TestBody() [0xbcfe48]
[2024-01-04 19:42:13.932][85526][critical][backtrace] [./source/server/backtrace.h:96] #7: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x2099bb0]
[2024-01-04 19:42:13.953][85526][critical][backtrace] [./source/server/backtrace.h:96] #8: testing::Test::Run() [0x2099a50]
[2024-01-04 19:42:13.964][85526][critical][backtrace] [./source/server/backtrace.h:96] #9: testing::TestInfo::Run() [0x209aba0]
[2024-01-04 19:42:13.975][85526][critical][backtrace] [./source/server/backtrace.h:96] #10: testing::TestSuite::Run() [0x209b90c]
[2024-01-04 19:42:13.987][85526][critical][backtrace] [./source/server/backtrace.h:96] #11: testing::internal::UnitTestImpl::RunAllTests() [0x20a8e14]
[2024-01-04 19:42:13.998][85526][critical][backtrace] [./source/server/backtrace.h:96] #12: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x20a88c8]
[2024-01-04 19:42:14.009][85526][critical][backtrace] [./source/server/backtrace.h:96] #13: testing::UnitTest::Run() [0x20a8724]
[2024-01-04 19:42:14.020][85526][critical][backtrace] [./source/server/backtrace.h:96] #14: Envoy::TestRunner::runTests() [0x1a29fe8]
[2024-01-04 19:42:14.032][85526][critical][backtrace] [./source/server/backtrace.h:96] #15: main [0x1a28e18]
[2024-01-04 19:42:14.032][85526][critical][backtrace] [./source/server/backtrace.h:96] #16: __libc_start_main [0xffff92428e10]

jmarantz pushed a commit that referenced this issue Jan 30, 2024
fix #31654 and #29496

Thanks @spacewander for founding this race:
envoyGoRequestSemaDec may be invoked eariler than r.sema.Wait, then, there is no one to resume the r.sema.Wait.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
Fixes: #31654, #29496


Signed-off-by: doujiang24 <[email protected]>
spacewander pushed a commit to mosn/envoy that referenced this issue Jun 27, 2024
…roxy#32081)

fix envoyproxy#31654 and envoyproxy#29496

Thanks @spacewander for founding this race:
envoyGoRequestSemaDec may be invoked eariler than r.sema.Wait, then, there is no one to resume the r.sema.Wait.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
Fixes: envoyproxy#31654, envoyproxy#29496


Signed-off-by: doujiang24 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants