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

Use a local HTTP server to provide a place for "remote" jars to download in tests #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use a local HTTP server to provide a place for "remote" jars to download in tests #29

wants to merge 1 commit into from

Conversation

mccheah
Copy link
Contributor

@mccheah mccheah commented Jan 10, 2018

No description provided.

mccheah
mccheah commented Jan 10, 2018
View reviewed changes
e2e/runner.sh
cd $SPARK_REPO_ROOT/dist

if [[ $DEPLOY_MODE == cloud ]] ;
then
docker build -t $FILE_SERVER_IMAGE "$TEST_ROOT/integration-test/docker-file-server"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the current setup we're building Docker images both in Scala code if the user just runs mvn but also here as part of the script. We should fix this at a later time though ideally earlier rather than later; there's a bit of room for error right now.

@mccheah
Copy link
Contributor Author

mccheah commented Jan 10, 2018

Requires #28 to merge first.

@mccheah mccheah changed the base branch from use-git-rev-parse-for-root to master January 11, 2018 01:12
@mccheah mccheah mentioned this pull request Jan 12, 2018
Merged
@mccheah
Copy link
Contributor Author

mccheah commented Jan 12, 2018

Messed up some merging - fixing

@mccheah
Copy link
Contributor Author

mccheah commented Jan 12, 2018

Looks ok now

@mccheah mccheah mentioned this pull request Jan 12, 2018
Merged
@mccheah
Copy link
Contributor Author

mccheah commented Jan 12, 2018

Jenkins, retest this please.

@kimoonkim
Copy link
Member

rerun integration tests please.

@SparkQA
Copy link

SparkQA commented Mar 20, 2018

Kubernetes integration test (test of tests) starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-test-integration-tests/2/

@SparkQA
Copy link

SparkQA commented Mar 20, 2018

Kubernetes integration test (test of tests) status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-test-integration-tests/2/

@AmplabJenkins
Copy link

Build finished. Test FAILed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-test-integration-tests/2/
Test FAILed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mccheah @kimoonkim @SparkQA @AmplabJenkins