-
Notifications
You must be signed in to change notification settings - Fork 5
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
Dockerize genomics-workspace #255
Comments
useful tool to check yml file |
yaml can be annoying. Looking at yours, I see a whole bunch of problems, you have to be very careful with the indenting, new lines, note for example
vs
ie, different line, missing the dash. This is why its saying it isnt an array, its not defined as an indented set of -'d lines. |
Sorry... @deming7h777 @bradfordcondon Can I ask why we are rebuilding this ? As far as I know, there is a repo based on docker for genomics-workspace in our gitlab organization. If we want to update to newer version. That will be a good starting point. |
''' and i also push the branch call genomics-docker |
Do you forget to clone the submodule ? |
What kind of submodule you mean ? |
git submodule, check if your |
It's not empty. |
Can you provide the result when you run the |
Ah, I can try it myself first, thanks for your kindness ! |
Got it ! I am not sure what you mean old version here. Do you mean the internal i5k-docker repo ? |
/bin/sh: ./configure: No such file or directory now I'm trying to figure out linux problem when I try to build cutomize image for workspace |
I change the code in Dockerfile from In the current issue, it seems I need to figure out the problem of c compiler |
|
I get an error on step 11: $ docker-compose up
Creating network "genomics-workspace_default" with the default driver
Building workspace
Step 1/13 : FROM centos:6.10
---> 30e66b619e9f
Step 2/13 : LABEL maintainer Deming
---> Using cache
---> 2fda5d15f394
Step 3/13 : ENV PYTHONBUFFERED 1
---> Using cache
---> 52eba52b63c3
Step 4/13 : VOLUME /data
---> Using cache
---> ed9ad883dfbe
Step 5/13 : RUN mkdir /genomics-workspace
---> Using cache
---> 7a04d699d6fe
Step 6/13 : WORKDIR /genomics-workspace
---> Using cache
---> fb278825b72d
Step 7/13 : COPY requirements.txt /genomics-workspace/
---> Using cache
---> c42e9156239c
Step 8/13 : RUN yum groupinstall "Development tools" -y && yum install wget -y
---> Using cache
---> 94e2bc2416b5
Step 9/13 : RUN wget http://www.python.org/ftp/python/2.7.15/Python-2.7.15.tar.xz && tar -xf Python-2.7.15.tar.xz
---> Using cache
---> 7f26b413d79e
Step 10/13 : WORKDIR /Python-2.7.15
---> Using cache
---> a2a50cbaa3d9
Step 11/13 : RUN yum install automake && yum install autoconf && autoreconf -i && ./configure --prefix=/usr/local --enable-unicode=usc4 --enable-shared LDFLAGS="-WL,-rpath /usr/local/lib" && make && make altinstall
---> Running in b8fea31fcd41
Loaded plugins: fastestmirror, ovl
Setting up Install Process
Loading mirror speeds from cached hostfile
* base: mirror.umd.edu
* extras: mirror.umd.edu
* updates: mirror.umd.edu
Package automake-1.11.1-4.el6.noarch already installed and latest version
Nothing to do
Loaded plugins: fastestmirror, ovl
Setting up Install Process
Loading mirror speeds from cached hostfile
* base: mirror.umd.edu
* extras: mirror.umd.edu
* updates: mirror.umd.edu
Package autoconf-2.63-5.1.el6.noarch already installed and latest version
Nothing to do
autoreconf: `configure.ac' or `configure.in' is required
ERROR: Service 'workspace' failed to build: The command '/bin/sh -c yum install automake && yum install autoconf && autoreconf -i && ./configure --prefix=/usr/local --enable-unicode=usc4 --enable-shared LDFLAGS="-WL,-rpath /usr/local/lib" && make && make altinstall' returned a non-zero code: 1 |
|
I tried these two customize dockerfile to build my own genomics-workspace image |
the docker-compose runs without error on my system after:
as port 5432 is in use on my machine.
So I no longer get db connection errors etc from the workspace container, meaning its hooked up right... but, i get |
warning: my knowledge of Django is nonexistant. you need to supply the IP address in the run command, like so
now, instead of an empty response error, i get a routing type error from Django, yay! So from this error i can see that we only define certain paths. so if i try to visit, say, http://localhost:8000/blast/ instead, i get this error that shows me stuff is definitely happening, but perhaps not configured all the way: the admin address also doesnt work:
so looks to me like the django site isnt configured and perhaps additional setup commands or config files need to be done. |
@bradfordcondon You are right. We didn’t define root path and the error message means no datatable has been created. I guess python manage.py migrate hasn’t been included into the setup steps. |
the only command run is the server start command specified in the docker-compose. You all probably need to run a full script that runs each command necessary to setting up and launching the django site. |
I didn't put migrate commend in the dockerfile because I thought we can build the container first then do the migrate commend directly in container. |
That's really good idea so I just need to modify the permission in git history in advance. |
By the way try docker toolbox rather than desktop docker :https://docs.docker.com/toolbox/toolbox_install_mac/ |
a note- docker toolbox is only recommended for machines that dont meet the system requirements for docker desktop. In general its use is discouraged I think. |
my current status:
my version:
theres an update available (not docker-compose, 1.23.2 is the latest full release of compose) |
this looks likely to me. windows stile line endings in the script. edit- nope. i ran dos2unix on all files in the project and that didnt resolve. |
ok got it @deming7h777
as its written, |
For this issue, I solve it by modify the EOL covertion from CR LF to LF |
Most of windows user need to install docker toolbox instead of desktop of docker due to the window's issue, ig, Windows 10 64bit: Pro, Enterprise or Education (1607 Anniversary Update, Build 14393 or later) is required.
@bradfordcondon 's issue
|
Seems to work well for windows, possibly for Mac with some modifications. Closing this until @childers can test later in the year. |
Re-opening - we need to document this properly first. |
@deming7h777 did you change the setup so that run_web.sh is already executable once you clone the repo? If that's not possible, can you change the mac documentation to include instructions to do this: Once that's done, I think you can create a PR for the documentation (sphinx_doc branch) - perhaps @tony006469 or @r06942072 can test the PR. |
I have not only made the change to run_web.sh and waiting for test but also add the instruction "sudo chmod +x ./run_web.sh" |
There are some solutions suggested: you can use a separate tool to do it for you, use a proper combination of user account and umask to set them properly by default or write a git hook yourself to do it. A hook would've to be installed on the user doing the checkout. I make two files called post-checkout and post-merge in genomics-workspace/.git/hooks/ Git is not really a deployment tool and should not be used as such. It is a version control system for source code |
I don't quite understand the point @deming7h777 state here. Do you mean that we should not directly change the permission of the file in git repo ? |
I still change the permission file in git repo used .git/hook/post-checkout and .git/hook/post-merge but I also see some suggestion that disagree to use git repo to do it |
@deming7h777 Can you post your reference here ? |
ah I see, so what' s your opinion for "we can directly modify file permission in git history rather than fixing it every time we want to use them." you just suggest and I think the permission issue I have encounter now is like this one : https://medium.com/@akash1233/change-file-permissions-when-working-with-git-repos-on-windows-ea22e34d5cee reference : |
@deming7h777 Thanks. I traced whole thread again and it seems to me that the fundamental reason of this issue is that on Windows (and only on Windows), Therefore, simply change the execution command into @bradfordcondon What kind of OS you use for testing this ? I think you use Windows, right ? |
hi @hsiaoyi0504 im on OSX.
this is the solution i proposed above, which does work for me on OSX (when I posted it, this branch might have changed since then). as far as a windows solution, i linked this in slack. Seems like if you are using window's git bash, there are tools for changing the file permissions ( |
Hi @bradfordcondon, thanks. Yeah, I just copy your solution :>) By default, |
As far as 592de71, the file permission of |
To solve the layout issue for blast home page |
|
@deming7h777 do you think it's possible to make a PR of the sphinx_docker branch, have someone test it, and merge before AGS? Would be nice to have the docker documentation available by then. |
@mpoelchau I think it is ok, I will work on it |
docker is very useful to rebuild the environment and whole configuration so I want to apply it to our genomics-workspace.
reference : 1. https://github.com/twtrubiks/docker-tutorial
2. https://docs.docker.com/get-started/
The text was updated successfully, but these errors were encountered: