You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kudu.apache.org by "ASF subversion and git services (Jira)" <ji...@apache.org> on 2022/04/26 16:51:00 UTC

[jira] [Commented] (KUDU-3361) UT SocketTest.TestRecvReset should fix correct

    [ https://issues.apache.org/jira/browse/KUDU-3361?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17528295#comment-17528295 ] 

ASF subversion and git services commented on KUDU-3361:
-------------------------------------------------------

Commit d216e417f7fa7daef59ec7eb2c15aadfd4e1c563 in kudu's branch refs/heads/master from shenxingwuying
[ https://gitbox.apache.org/repos/asf?p=kudu.git;h=d216e417f ]

[unit test] KUDU-3361 Fix SocketTest.TestRecvReset

The case SocketTest.TestRecvReset is not steady, and can cause
jenkins -1, and need rerun it. And I found using a fixed timeout
parameter is not good enough.

eg: Set timeout 1ms coredump happened.
Check failure stack trace: ***
Aborted at 1650445865 (unix time) try "date -d @1650445865" if you are
using GNU date ***
PC: @     0x7fd63d0092e7 __GI_raise
SIGABRT (@0x198309) received by PID 1671945 (TID 0x7fd640cb7980) from
PID 1671945; stack trace: ***
    @   0x7fd63e8881d1 google::(anonymous namespace)::FailureSignalHandler()
    @     0x7fd63d390140 (unknown)
    @     0x7fd63d0092e7 __GI_raise
    @     0x7fd63d00a6c8 __GI_abort
    @     0x7fd63e878419 google::logging_fail()
    @     0x7fd63e87be9d google::LogMessage::Fail()
    @     0x7fd63e87daae google::LogMessage::SendToLog()
    @     0x7fd63e87ba20 google::LogMessage::Flush()
    @     0x7fd63e87e399 google::LogMessageFatal::~LogMessageFatal()
    @           0x21ff23 kudu::SocketTest::ConnectToListeningServer()
    @           0x21cc4a kudu::SocketTest::DoTestServerDisconnects()
    @           0x21b29c kudu::SocketTest_TestRecvReset_Test::TestBody()
    @     0x7fd63e219f2e testing::internal::HandleExceptionsInMethodIfSupported<>()
    @     0x7fd63e20fa2b testing::Test::Run()
    @     0x7fd63e20fb95 testing::TestInfo::Run()
    @     0x7fd63e20fc85 testing::TestSuite::Run()
    @     0x7fd63e210144 testing::internal::UnitTestImpl::RunAllTests()
    @     0x7fd63e21a46e testing::internal::HandleExceptionsInMethodIfSupported<>()
    @     0x7fd63e210340 testing::UnitTest::Run()
    @     0x7fd640e933c1 RUN_ALL_TESTS()
    @     0x7fd640e92316 main
    @     0x7fd63cff5ac5 __libc_start_main
    @           0x219029 (unknown)
    Aborted (core dumped)

eg: Set timeout 50ms test failed.
Value of: s.message().ToString()
    Expected: contains regular expression "recv error from
    127.0.0.1:[0-9]+: Resource temporarily unavailable"
    Actual: "recv error from unknown peer: Transport endpoint is not
    connected" (of type std::string)
    WARNING: Logging before InitGoogleLogging() is written to STDERR

Change-Id: Ie85f36532510db53c8ac167f9d20689da0c76d33
Reviewed-on: http://gerrit.cloudera.org:8080/18431
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <al...@apache.org>


> UT SocketTest.TestRecvReset should fix correct
> ----------------------------------------------
>
>                 Key: KUDU-3361
>                 URL: https://issues.apache.org/jira/browse/KUDU-3361
>             Project: Kudu
>          Issue Type: Test
>            Reporter: shenxingwuying
>            Assignee: shenxingwuying
>            Priority: Trivial
>
> SocketTest.TestRecvReset running not steady, and can cause the whole jenkins task rerun.
>  
> I change the time param, 
> 1ms, coredump
> F0420 17:11:05.293507 1671945 socket-test.cc:118] Check failed: _s.ok() Bad status: Network error: connect(2) error: Connection reset by peer (error 104)
> *** Check failure stack trace: ***
> *** Aborted at 1650445865 (unix time) try "date -d @1650445865" if you are using GNU date ***
> PC: @     0x7fd63d0092e7 __GI_raise
> *** SIGABRT (@0x198309) received by PID 1671945 (TID 0x7fd640cb7980) from PID 1671945; stack trace: ***
>     @     0x7fd63e8881d1 google::(anonymous namespace)::FailureSignalHandler()
>     @     0x7fd63d390140 (unknown)
>     @     0x7fd63d0092e7 __GI_raise
>     @     0x7fd63d00a6c8 __GI_abort
>     @     0x7fd63e878419 google::logging_fail()
>     @     0x7fd63e87be9d google::LogMessage::Fail()
>     @     0x7fd63e87daae google::LogMessage::SendToLog()
>     @     0x7fd63e87ba20 google::LogMessage::Flush()
>     @     0x7fd63e87e399 google::LogMessageFatal::~LogMessageFatal()
>     @           0x21ff23 kudu::SocketTest::ConnectToListeningServer()
>     @           0x21cc4a kudu::SocketTest::DoTestServerDisconnects()
>     @           0x21b29c kudu::SocketTest_TestRecvReset_Test::TestBody()
>     @     0x7fd63e219f2e testing::internal::HandleExceptionsInMethodIfSupported<>()
>     @     0x7fd63e20fa2b testing::Test::Run()
>     @     0x7fd63e20fb95 testing::TestInfo::Run()
>     @     0x7fd63e20fc85 testing::TestSuite::Run()
>     @     0x7fd63e210144 testing::internal::UnitTestImpl::RunAllTests()
>     @     0x7fd63e21a46e testing::internal::HandleExceptionsInMethodIfSupported<>()
>     @     0x7fd63e210340 testing::UnitTest::Run()
>     @     0x7fd640e933c1 RUN_ALL_TESTS()
>     @     0x7fd640e92316 main
>     @     0x7fd63cff5ac5 __libc_start_main
>     @           0x219029 (unknown)
> Aborted (core dumped)
>  
> 50ms, failed.
>  
>  
> Failure
> Value of: s.message().ToString()
> Expected: contains regular expression "recv error from 127.0.0.1:[0-9]+: Resource temporarily unavailable"
>   Actual: "recv error from unknown peer: Transport endpoint is not connected" (of type std::string)
> WARNING: Logging before InitGoogleLogging() is written to STDERR
>  
> So, Simple use the time parameter is no good enough, should improve it.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)