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

a few tweaks to scripts/jenkins-test #459

Merged
merged 2 commits into from Nov 4, 2014

Conversation

Projects
None yet
3 participants
@ryan-williams
Member

ryan-williams commented Nov 4, 2014

No description provided.

ryan-williams added some commits Nov 3, 2014

CYA: pre-remove output files in jenkins-test
while debugging jenkins-test runs, I copied some of the files it
generates into my local ADAM_HOME; these were in turn getting copied
into the tmp dir created by jenkins-test, and causing various commands
to fail when they realized the path they were trying to output to
already exists. This seems like an unnecessary failure mode for the
jenkins-test script to be vulnerable to (namely, the existence in local
ADAM_HOME of files (untracked by git) with the same names as arbitrary
paths that jenkins-test expects to write to), especially since these
particular files are involved in tests but don’t actually live in the
git repo.
@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Nov 4, 2014

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/361/
Test PASSed.

AmplabJenkins commented Nov 4, 2014

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/361/
Test PASSed.

massie added a commit that referenced this pull request Nov 4, 2014

Merge pull request #459 from ryan-williams/jenkins
a few tweaks to scripts/jenkins-test

@massie massie merged commit 714d08a into bigdatagenomics:master Nov 4, 2014

1 check passed

default Merged build finished.
Details
@massie

This comment has been minimized.

Show comment
Hide comment
@massie

massie Nov 4, 2014

Member

Thanks, Ryan!

Member

massie commented Nov 4, 2014

Thanks, Ryan!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment