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.

Sign up for GitHub

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

Jump to bottom

Is vm release_x64 standalone/io/EchoServerStreamTest flaky? #2262

Closed
peter-ahe-google opened this issue Mar 23, 2012 · 3 comments
Closed

Is vm release_x64 standalone/io/EchoServerStreamTest flaky? #2262

peter-ahe-google opened this issue Mar 23, 2012 · 3 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io

Comments

@peter-ahe-google
Copy link
Contributor

Looks like this a flaky test. I cannot see how my change (https://chromiumcodereview.appspot.com//9836022) could cause this problem:

FAILED: vm release_x64 standalone/io/EchoServerStreamTest
Expected: pass
Actual: fail

stderr:
Error Accept: Resource temporarily unavailable
Unhandled exception:
Expect.fail('Server socket error SocketIOException: Accept failed (OS Error: Resource temporarily unavailable, errno = 35)')
 0. Function: 'Expect._fail@24787773' url: 'bootstrap' line:1496 col:5
 1. Function: 'Expect.fail' url: 'bootstrap' line:1283 col:10
 2. Function: 'TestingServer.errorHandlerServer' url: 'file:///b/build/slave/vm-mac-release/build/dart/tests/standalone/src/io/TestingServer.dart' line:15 col:18
 3. Function: '_SocketBase@19ffb8b9.doReportError' url: 'dart:io' line:6594 col:34
 4. Function: '_SocketBase@19ffb8b9._reportError@19ffb8b9' url: 'dart:io' line:6604 col:20
 5. Function: '_ServerSocket@19ffb8b9._connectionHandler@19ffb8b9' url: 'dart:io' line:6695 col:21
 6. Function: '_SocketBase@19ffb8b9._multiplex@19ffb8b9' url: 'dart:io' line:6471 col:25
 7. Function: '_SocketBase@19ffb8b9.function' url: 'dart:io' line:6584 col:59
 8. Function: '_ReceivePortImpl@38422f21._handleMessage@38422f21' url: 'dart:isolate' line:288 col:22

Command line: xcodebuild/Release_x64/dart --short_socket_write --ignore-unrecognized-flags --enable_asserts --enable_type_checks /b/build/slave/vm-mac-release/build/dart/tests/standalone/src/io/EchoServerStreamTest.dart
@madsager
Copy link
Contributor

Yes, it is flaky. Based on the new error reporting that Søren has put in it is pretty clear what is going on. I'm working on the fix.

Added Started label.

@madsager
Copy link
Contributor

Added Fixed label.

@kevmoo
Copy link
Member

kevmoo commented May 14, 2014

Removed Area-IO label.
Added Area-Library, Library-IO labels.

@peter-ahe-google peter-ahe-google added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io labels May 14, 2014
copybara-service bot pushed a commit that referenced this issue Oct 24, 2023
@devoncarew
[deps] rev dartdoc, lints, matcher, protobuf, tools, webdev
540b3a8 
Revisions updated by `dart tools/rev_sdk_deps.dart`.

dartdoc (https://github.com/dart-lang/dartdoc/compare/f7e9b17..53da3e1):
  53da3e1d  2023-10-23  dependabot[bot]  Bump actions/checkout from 4.1.0 to 4.1.1 (#3544)
  62d5469d  2023-10-23  Sam Rawlins  Support extension types in sidebars and categories (#3537)
  69abd0f0  2023-10-22  Sam Rawlins  Refactor file-gathering logic (#3539)
  e9c61d1d  2023-10-22  Sam Rawlins  Convert some late final fields to getters in Accessor, Annotation, Constructor (#3534)
  adcdc8b7  2023-10-20  Parker Lougheed  Update package:lints to v3 (#3542)
  41f02622  2023-10-19  Sam Rawlins  Greatly simplify ModelCommentReference (#3541)
  50e4b679  2023-10-19  Sam Rawlins  Remove use of NodeLocator2 (#3538)
  b2de813c  2023-10-19  Sam Rawlins  Remove support for deprecated leading new in comment references (#3529)
  b350c688  2023-10-19  Sam Rawlins  Bump to 7.0.1 (#3540)
  5256e2fb  2023-10-19  Sam Rawlins  Remove unused warnings (#3533)

lints (https://github.com/dart-lang/lints/compare/975c687..2cf8403):
  2cf8403  2023-10-23  Devon Carew  add no_wildcard_variable_uses; rev to a new major version (#165)
  7b0f556  2023-10-23  Parker Lougheed  Remove mention of no_wildcard_variable_uses from changelog (#164)

matcher (https://github.com/dart-lang/matcher/compare/356e5f6..7512f80):
  7512f80  2023-10-23  Michael Goderbauer  Specify language in neverCalled docs for nicer formatting in docs (#230)

protobuf (https://github.com/dart-lang/protobuf/compare/050c162..3528fad):
  3528fad  2023-10-24  Ömer Sinan Ağacan  Use `setRange` when copying output chunks to the final buffer in `CodedBufferWriter` (#887)

tools (https://github.com/dart-lang/tools/compare/15cc9c7..da6bb18):
  da6bb18  2023-10-24  Elias Yishak  Enum + event constructors added for doctor events (#178)
  e3dd149  2023-10-24  Elias Yishak  Use futures list internally to manage send events (#184)

webdev (https://github.com/dart-lang/webdev/compare/1bd434b..6e324af):
  6e324afb  2023-10-24  Ben Konyi  Add dependency on `package:vm_service_interface` (#2262)
  8429a79f  2023-10-20  Elliott Brooks  Only notify chatroom when daily stable testing fails (#2259)
  3463d169  2023-10-19  Elliott Brooks  Remove Chrome 115 extension error (#2258)
  fdebc06e  2023-10-19  Elliott Brooks  Reset Webdev to 3.2.0-wip after release (#2260)
  9cffb896  2023-10-19  Elliott Brooks  Reset DWDS to 22.1.0-wip after release (#2256)
  07c70c6c  2023-10-19  Elliott Brooks  Prepare Webdev for release to 3.1.0 (#2255)

Change-Id: Ieae3aadcc804a270867d7935b702987cf1f6d51c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/332060
Reviewed-by: Elias Yishak <eliasyishak@google.com>
Commit-Queue: Devon Carew <devoncarew@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kevmoo @madsager @peter-ahe-google