-
Notifications
You must be signed in to change notification settings - Fork 18
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
enable web and nginx slaves #20
Conversation
- use swarm plugin 2.0 - use omero-install and fix Dockerfile
- make nginx a jankins node - allow linking nginx config and statics - add redis node
ENV OMERO_INSTALL /tmp/omero-install/linux | ||
|
||
RUN yum install -y git && \ | ||
git clone -b java-parameters https://github.com/jburel/omero-install.git /tmp/omero-install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sbesson had a good idea: down the road, we should likely turn this into a build-arg, but I think that will require a newer version of docker-compose.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
args are in V 2.0 https://docs.docker.com/compose/compose-file/#args
Few comments added, but generally 👍 Discussing with @aleksandra-tarkowska just now, the general next steps for this week look to be:
|
@joshmoore unfortunately I had no success on deployment. All my new systemd containers do not start correctly via compose (although they do start via docker run). I think this is due to compose version 1.3 being too old. On Mac I use 1.6 and have no problem. As compose is installed in your homedir I wasn't sure how to upgrade |
I followed your strategy with miniconda and installed compose 1.6.2
looks like there is no difference, systemd is not starting up. I am not sure what is wrong
|
Can you try a minimal docker-compose file that just starts up omero-ssh-systemd, and play around with the parameters? |
@manics thank you for you suggestions looks like we were missing
@joshmoore do we want centos7 compose file to override default ? |
0159646
to
1f2b012
Compare
58f16f1
to
0918f3b
Compare
Ah, I wonder if that's happening because the plugin is not installed. We can clean all of that up after the fact. In general, if you don't know why a change shows up in the diff, probably best not to commit it. |
I removed previous commits and will play with user id in another PR |
Tested deployment on an existing devspace. As the existing customization created large conflicts, I went for the following approach (after backing up the whole devspace directory):
Got Jenkins up and running with the expected set of jobs. OMERO-test-integration has retained its history as expected. Started a Trigger job to test the entire workflow. At this point, a couple of documentations PRs should follow:
|
I think it should be possible to do it on live container, runing
All files which are located in the user’s home directory will have the file UID changed automatically. Files outside user’s home directory need to be changed manually.
and then |
The above is only needed for new systemd containers, jenkins will pick it up after restart. @manics @joshmoore how do you think? |
Tested a full Trigger workflow. Everything worked as expected (barring failing robot tests which are also happening on the mainline devspace). Also hooked an OMERO-import job to the |
thanks everyone for helping and testing |
Notes on my process for upgrading:
|
This PR improves existing containers infrastructure
NOTE:
MORE TODO: