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

Runfiles Tests are failing on remote execution #112

Closed
xingao267 opened this issue Aug 8, 2018 · 4 comments · Fixed by #132
Closed

Runfiles Tests are failing on remote execution #112

xingao267 opened this issue Aug 8, 2018 · 4 comments · Fixed by #132
Assignees

Comments

@xingao267
Copy link
Member

A lot of tests are failing on RBE when I tested with #111 where I added the Buildkite config to test against RBE. You can find the set of tests that are failing in the PR (https://github.com/bazelbuild/rules_rust/pull/111/files) where I have to skip them.

We previously tested rules_rust repo at commit ef77316 internally and this set of test targets pass on RBE:

-- @examples//... -@examples//hello_runfiles:hello_runfiles_test

It's likely there is some regression between ef77316 and 1944c8a

@acmcarther
Copy link
Collaborator

acmcarther commented Aug 8, 2018

I missed this codegen-backends directory when I changed the toolchain fetching logic.

I'm putting together a fix now in https://github.com/bazelbuild/rules_rust/compare/acm-fix-remote-exec.


EDIT: That PR didn't address one of the failures. Details copied from that PR:

CI logs from commit 2 of this PR:

Runfiles test stdout:
https://buildkite.com/bazel/rules-rust-rustlang/builds/149#166b2f90-cd68-40d5-9a63-88dbd0cc1215**
Log message from test failure:
https://storage.googleapis.com/bazel-buildkite-artifacts/166b2f90-cd68-40d5-9a63-88dbd0cc1215/xamples/hello_runfiles/hello_runfiles_test/attempt_1.log

Panic site:

File::open(runfiles.join("examples/hello_runfiles/data/sample.txt")).unwrap()

pub fn get_runfiles_dir() -> io::Result<PathBuf> {
let mut path = std::env::current_exe()?;
if cfg!(target_os = "macos") {
path.pop();
} else {
let mut name = path.file_name().unwrap().to_owned();
name.push(".runfiles");
path.pop();
path.push(name);
}
Ok(path)

Symptom is test is trying to load the runfile but not finding it. Some historical discussion on runfiles here: #66, describing some inconsistency in treatment among OSes and languages.

If nothing else, some filesystem exploration within the test would probably help figure out where they're being routed to.

@mfarrugi
Copy link
Collaborator

#122 skipped the (newly added / fixed) doc tests as well, I don't have any visibility into the test failures though.

@xingao267
Copy link
Member Author

@mfarrugi
Copy link
Collaborator

mfarrugi commented Aug 29, 2018 via email

@mfarrugi mfarrugi changed the title Tests are failing on remote execution Runfiles Tests are failing on remote execution Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants