[ADAM-1038] Eliminate writing to /tmp during CI builds. #1127

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
@fnothaft
Member

fnothaft commented Aug 25, 2016

Resolves #1038. When running scripts/jenkins-test, we make a temp dir where all test files are written. We modify our pom.xmls at test time so that this temp dir is passed as java.io.tmpdir when running scalatest, and then clean the temp dir at the end of the CI build.

CC @shaneknapp

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Aug 25, 2016

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

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

[ADAM-1038] Eliminate writing to /tmp during CI builds.
Resolves #1038. When running `scripts/jenkins-test`, we make a temp dir where all
test files are written. We modify our `pom.xml`s at test time so that this temp
dir is passed as `java.io.tmpdir` when running scalatest, and then clean the temp
dir at the end of the CI build.
@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Aug 25, 2016

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

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

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 25, 2016

Member

LGTM, works locally

Member

heuermh commented Aug 25, 2016

LGTM, works locally

@shaneknapp

This comment has been minimized.

Show comment
Hide comment
@shaneknapp

shaneknapp Aug 25, 2016

Contributor

LGTM!

Contributor

shaneknapp commented Aug 25, 2016

LGTM!

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 25, 2016

Member

Merged via commit 6e28504.

Thank you, @fnothaft!

Member

heuermh commented Aug 25, 2016

Merged via commit 6e28504.

Thank you, @fnothaft!

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