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

ENH: Add option to mount additional drives #428

Merged
merged 1 commit into from
Mar 22, 2016

Conversation

sandhujasmine
Copy link

Some of the blaze tests populate postgres data from CSV file in the
blaze/ project's directory. Use this option to mount it in docker so
posgres can read data when running those tests.

Some of the blaze tests populate postgres data from CSV file in the
blaze/ project's directory. Use this option to mount it in docker so
posgres can read data when running those tests.
@sandhujasmine
Copy link
Author

This allows us to define another mount point using -m so psql can find and load data for blaze tests.

To run blaze tests in test_postgresql_compute.py, let's fire up the postgres docker container and mount the test data location as well as the temp location to write output of tests:

provisioning.sh -t ~/tmp/docker/ -m ${BLAZE_PROJ_PATH}/blaze/blaze/examples/data restart

Next, set the env variables in the terminal. provisioning script writes a docker_info.txt file in ~/tmp/docker - copy & paste first 3 lines in terminal:

export POSTGRES_IP=192.168.99.100
export MONGO_IP=192.168.99.100
export POSTGRES_TMP_DIR=/Users/jsandhu/tmp/docker/

The following will now run tests that were previously skipping:
py.test -v test_postgresql_compute.py

@kwmsmith
Copy link
Member

+1 Sorry that this one slipped through the cracks.

@kwmsmith kwmsmith added this to the 0.5.0 milestone Mar 22, 2016
kwmsmith added a commit that referenced this pull request Mar 22, 2016
ENH: Add option to mount additional drives
@kwmsmith kwmsmith merged commit 6a0cc89 into blaze:master Mar 22, 2016
@sandhujasmine sandhujasmine deleted the add-mnt-opt branch March 22, 2016 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants