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

//src/test/shell/bazel:rule_test_test fails on CI (OSX) #2939

Closed
buchgr opened this issue May 4, 2017 · 7 comments
Closed

//src/test/shell/bazel:rule_test_test fails on CI (OSX) #2939

buchgr opened this issue May 4, 2017 · 7 comments
Labels
breakage P2 We'll consider working on this in future. (Assignee optional) under investigation

Comments

@buchgr
Copy link
Contributor

buchgr commented May 4, 2017

The test test_local_rule_test_in_root failed on OSX with

Server terminated abruptly (error code: 14, error message: '', log file: '...')

The error code maps to gRPC's UNAVAILABLE code, so this actually looks like the gRPC server crashed.

http://ci.bazel.io/job/Gerrit-bazel-tests/639/BAZEL_VERSION=latest,PLATFORM_NAME=darwin-x86_64/testReport/junit/(root)/(empty)/test_local_rule_test_in_root/

@buchgr
Copy link
Contributor Author

buchgr commented May 4, 2017

cc @damienmg

@damienmg
Copy link
Contributor

damienmg commented May 4, 2017

Probably a flake. /cc @lberki because gRPC.

@damienmg damienmg added breakage under investigation Release blocker P1 I'll work on this now. (Assignee required) and removed Release blocker labels May 4, 2017
@damienmg
Copy link
Contributor

damienmg commented May 4, 2017

This is irreproducible because we don't have access to the jvm.log file. We should probably output it if we are running under test.

Assigning to Lukacs to evaluate.

@lberki
Copy link
Contributor

lberki commented May 5, 2017

Without a jvm.log there isn't a lot I can do.

@lberki lberki closed this as completed May 5, 2017
@damienmg
Copy link
Contributor

damienmg commented May 5, 2017 via email

@damienmg damienmg added P2 We'll consider working on this in future. (Assignee optional) and removed P1 I'll work on this now. (Assignee required) labels May 5, 2017
@damienmg
Copy link
Contributor

damienmg commented May 5, 2017

As discussed offline, in our test framework we should cat the jvm.log if we get that error.

@haxorz
Copy link
Contributor

haxorz commented May 20, 2017

fyi, this sounds possibly like a bug introduced by 3e5edaf. it was diagnosed by @ericfelly internally. that commit was reverted in c4f271d. @philwo fixed the underlying issue recently with 67cea8f (see the third line of the commit message for the bug).

given that i'm pretty sure bazel.io was affected by another bug with the initial commit (see #2985), it's likely you're also being affected by this bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breakage P2 We'll consider working on this in future. (Assignee optional) under investigation
Projects
None yet
Development

No branches or pull requests

4 participants