Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(105)

Issue 2105553003: Attempt to make probability of test flake in tests/standalone/io/socket_bind_test lower (Closed)

Created:
4 years, 5 months ago by kustermann
Modified:
4 years, 5 months ago
Reviewers:
Bill Hesse
CC:
reviews_dartlang.org, Florian Schneider, Cutch
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Attempt to make probability of test flake in tests/standalone/io/socket_bind_test lower The test is attempting to bind 2 different sockets to the *same* port: * socket 1: (IPv4, port) * socket 2: (IPv6, port) Between making socket 1 and socket 2 there can be other processes on the host grabbing a (IPvX, port) combination thereby making the binding of socket 2 fail. We will now retry the test a few times which will hopefully make the flakiness go away. BUG=https://github.com/dart-lang/sdk/issues/26716 R=whesse@google.com Committed: https://github.com/dart-lang/sdk/commit/75c8b6d74c5522c56cec9163768eb30608494259

Patch Set 1 #

Total comments: 2

Patch Set 2 : Attempt to make probability of test flake in tests/standalone/io/socket_bind_test lower #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -21 lines) Patch
M tests/standalone/io/socket_bind_test.dart View 1 2 chunks +39 lines, -21 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
kustermann
4 years, 5 months ago (2016-06-28 10:21:19 UTC) #3
Bill Hesse
lgtm https://codereview.chromium.org/2105553003/diff/1/tests/standalone/io/socket_bind_test.dart File tests/standalone/io/socket_bind_test.dart (right): https://codereview.chromium.org/2105553003/diff/1/tests/standalone/io/socket_bind_test.dart#newcode125 tests/standalone/io/socket_bind_test.dart:125: print("Retry $i"); Put the print after the retry-catch ...
4 years, 5 months ago (2016-06-28 10:29:28 UTC) #4
kustermann
Thanks https://codereview.chromium.org/2105553003/diff/1/tests/standalone/io/socket_bind_test.dart File tests/standalone/io/socket_bind_test.dart (right): https://codereview.chromium.org/2105553003/diff/1/tests/standalone/io/socket_bind_test.dart#newcode125 tests/standalone/io/socket_bind_test.dart:125: print("Retry $i"); On 2016/06/28 10:29:28, Bill Hesse wrote: ...
4 years, 5 months ago (2016-06-28 10:36:06 UTC) #5
kustermann
4 years, 5 months ago (2016-06-28 10:36:27 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
75c8b6d74c5522c56cec9163768eb30608494259 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698