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

Support for volumes in AWS and GCP #1

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

Conversation

bharatak
Copy link

@bharatak bharatak commented Dec 9, 2017

When trying to mount a volume (disk in GCP) to the docker image and installation of app aborts with the following error

Directory /home/dataiku/dss already exists, but is not empty. Aborting !

The current logic in the wrapper script which is run as entry point to the Dataiku-provided Docker image
https://github.com/dataiku/dataiku-tools/blob/master/dss-docker/run.sh
looks at the provided data directory for a well-known DSS file in order to automatically determine whether it should install a new instance or upgrade an existing one, as a convenience for an easier use.
It should be possible to change that in a production environment to adapt to use a EBS volume or a GCP disk.

@@ -59,7 +59,6 @@ RUN R --slave --no-restore \

# Entry point
WORKDIR /home/dataiku
USER dataiku
Copy link

@jbelafa jbelafa Jan 16, 2018

Choose a reason for hiding this comment

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

Please try to avoid as much as possible execution of run.sh as root
In theDockerFile permissions are already enforced for the data directory
If it's a cloud provider level permission issue we should consider that enforcing container level related permissions must fix the problem.

You can also fork the DockerFile if you are willing to handle a EBS (or equivalent ) within docker image definition.

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 this pull request may close these issues.

2 participants