Project

General

Profile

Bug #12776

Flaky test case: TestThread#test_thread_name

Added by vo.x (Vit Ondruch) almost 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
ruby -v:
uby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]
[ruby-core:77323]

Associated revisions

Revision 69f86f79
Added by nagachika (Tomoyuki Chikanaga) almost 3 years ago

merge revision(s) 53520: [Backport #12776]

    test_thread.rb: wait for the thread to sleep

    * test/ruby/test_thread.rb (test_thread_name): wait for the status

of the subject thread to fix, so that the status does not change
between two inspect methods.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_3@56440 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 56440
Added by nagachika (Tomoyuki Chikanaga) almost 3 years ago

merge revision(s) 53520: [Backport #12776]

test_thread.rb: wait for the thread to sleep

* test/ruby/test_thread.rb (test_thread_name): wait for the status

of the subject thread to fix, so that the status does not change
between two inspect methods.

History

Updated by vo.x (Vit Ondruch) almost 3 years ago

  • Backport changed from 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: UNKNOWN to 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: REQUIRED
  • ruby -v set to uby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]
  • Subject changed from Flaky test case:TestThread#test_thread_name to Flaky test case: TestThread#test_thread_name

In Fedora CI, I observe from time to time this test failure:

  1) Failure:
TestThread#test_thread_name [/builddir/build/BUILD/ruby-2.3.1/test/ruby/test_thread.rb:1061]:
<"#<TestThread::Thread:0x0055db68186ef8@/builddir/build/BUILD/ruby-2.3.1/test/ruby/test_thread.rb:1054 run>"> expected but was
<"#<TestThread::Thread:0x0055db68186ef8@/builddir/build/BUILD/ruby-2.3.1/test/ruby/test_thread.rb:1054 sleep>">.

It seems that this test fails on rubyci.org from time to time as well:

http://www.rubyist.net/~akr/chkbuild/debian/ruby-2.3/log/20160619T204000Z.diff.html.gz

It seems that r53520 may be the cure? I have not tested this patch though.

Updated by shyouhei (Shyouhei Urabe) almost 3 years ago

  • Status changed from Open to Closed

Seems OK, please backport.

Updated by nagachika (Tomoyuki Chikanaga) almost 3 years ago

  • Backport changed from 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: REQUIRED to 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: DONE

ruby_2_3 r56440 merged revision(s) 53520.

Also available in: Atom PDF