-
Notifications
You must be signed in to change notification settings - Fork 498
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
Release: 5.x #315
Comments
What do you mean here? Are you removing NGINX from the image entirely? If yes, this will break the image for existing users who depend on the SSL support provided by the image and we should have clear instructions so that existing users can migrate their existing deployments properly.
I see that you intend to move the image to use the official redmine image as a base. I really don't see any advantage of doing so and also see a lot of effort from you to migrate the feature set of this image. There is no clear advantage of taking on this effort and there is no real advantage to the user. I suggest you hold on to merging these changes and I'm sorry if i've missed any discussions on this matter. If you are really sold on the advantage of this migration, then my suggestion is to fork this image, make releases of the fork with these changes. Once any issues are ironed out and the new image is reasonably stable then I'd be happy to merge in the changes.
|
@sameersbn there will be no diffrence the only diff between this image and the other one is that we have imap scripts and nginx + ssl then we have a lot of issues with nginx and ssl . So to solve all that issues and keep this compatible we simply split nginx out but don't the settings i will supply a clear migration path. Pro of this merge is:
Cons
|
@sameersbn befor i merge that in i will discusses that with you this issue is still in preparation but i think it will close near all open issues |
IMHO it would seem logical to base on the official image. |
I'm no expert but these changes to structure sound ideal. Any updates on this? |
Sorry about not looking into this. It would be very helpful if you guys could help test this PR and validate it works for you. Thanks! |
Just noticed that the PR is WIP and there are a couple of TODO tasks in the pipeline. |
@sameersbn yes we will try our best on that at present that is a lot of work and we could need repo access or your assistance for the setup of automated tests as your the repo owner that is blocking |
Notes for my Self for this week
|
Redmine-Version 3.4.4 was released yesterday. |
http://www.redmine.org/news/116. Is there any plans to upgrade to 3.3.6 and 3.4.4 ? |
frank has been working on a rework of the docker image. In the meantime, if someone would be willing to create a pull request updating the installed version of redmine. I'd be willing to test it real quick and merge it in. |
Sorry, I couldn't wait, I used the official repo and made the following compose file work:
Please delete the file dump.sql after the first Docker Compose-run. If you have any advantages with this repo here: please let us know. Thanks. |
Created updates up through 3.2.9, 3.3.6, and 3.4.4 |
I also could not wait and made the switch. Here is what I think are the interesting part:
version: "3.1"
services:
web:
image: redmine:3.4-passenger
restart: always
command: passenger start --ssl --ssl-certificate /run/secrets/ssl-certificate --ssl-certificate-key /run/secrets/ssl-key --ssl-port 3443
environment:
REDMINE_DB_MYSQL_FILE: /run/secrets/database-hostname
REDMINE_DB_DATABASE_FILE: /run/secrets/database-name
REDMINE_DB_USERNAME_FILE: /run/secrets/database-username
REDMINE_DB_PASSWORD_FILE: /run/secrets/database-password
REDMINE_PLUGINS_MIGRATE: 1
ports:
- '80:3000'
- '443:3443'
depends_on:
- database
secrets:
- source: redmine-configuration
target: /usr/src/redmine/config/configuration.yml
- ssl-certificate
- ssl-key
- database-hostname
- database-username
- database-password
- database-name
volumes:
- /etc/localtime:/etc/localtime:ro
- ./data/files:/usr/src/redmine/files
- ./initializers:/usr/src/redmine/config/initializers/customization
- ./plugins:/usr/src/redmine/plugins
- ./themes:/usr/src/redmine/public/themes
labels:
deck-chores.reminder-emails.command: 'rake redmine:reminder_plugin:send_notifications >> log/due_date_reminder.log 2>&1'
deck-chores.reminder-emails.interval: daily
database:
image: mysql:5.7
restart: always
environment:
MYSQL_ROOT_PASSWORD_FILE: /run/secrets/database-password
secrets:
- database-password
volumes:
- /etc/localtime:/etc/localtime:ro
- ./data/mysql:/var/lib/mysql
officer:
image: funkyfuture/deck-chores
restart: always
environment:
TIMEZONE: Europe/Zurich
volumes:
- /etc/localtime:/etc/localtime:ro
- /var/run/docker.sock:/var/run/docker.sock
secrets:
redmine-configuration:
file: ./data/secrets/redmine/configuration.yml
ssl-certificate:
file: ./data/secrets/ssl/redmine.pem
ssl-key:
file: ./data/secrets/ssl/redmine.key
database-hostname:
file: ./data/secrets/database/hostname
database-username:
file: ./data/secrets/database/username
database-password:
file: ./data/secrets/database/password
database-name:
file: ./data/secrets/database/name |
Will use https://docs.docker.com/develop/develop-images/multistage-build/ |
Goals
turn this into a docker app that you can run it makes no sense anymore to integrate the components into a single container. we should only supply a sidecar container that does the config.
Changelog
Closes:
TODO
3.4.3 Features
The text was updated successfully, but these errors were encountered: