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

Update to current LTS Version #40

Open
SvenLie opened this issue May 6, 2020 · 12 comments
Open

Update to current LTS Version #40

SvenLie opened this issue May 6, 2020 · 12 comments

Comments

@SvenLie
Copy link

SvenLie commented May 6, 2020

Hey,

Is it possible to update your image to 3.22.15 LTS and tag it as latest or provide a way to use the new version 4?

Thank you

@crramirez
Copy link
Owner

Hello,

"Is it possible to update your image to 3.22.15 LTS". Yes, give me until the weekend.

And about LimeSurvey 4 I am working on that. Limesurvey 4 requires PHP 7 and it takes some work to build the container with this.

Regards,
Carlos

@stefangweichinger
Copy link

Anything I can do to help with LS-4.x?

@stefangweichinger
Copy link

Trying to come up with a Dockerfile. To me it seems that the image you use (tutum/lamp) does not really work well when someone wants to install and use PHP7, right?

I'd prefer debian:buster or php:7.4.0-apache but these don't bring mariadb/mysql.

Do you have any ideas or drafts already?

@crramirez
Copy link
Owner

Hello @stefangweichinger ,

Sorry for not answering before, I was looking for time. Please check this:

https://github.com/crramirez/lamp

and this:

https://hub.docker.com/repository/docker/crramirez/lamp

You can rebase the Dockerfile from tutum/lamp to crramirez/lamp. Last time I tried the app didn't start, but this is in what I've been working.

@crramirez
Copy link
Owner

"I'd prefer debian:buster or php:7.4.0-apache but these don't bring mariadb/mysql." I agree. And I always end with two containers, and the majority are fine with just one container with everything, this is why I stick with the lamp

@stefangweichinger
Copy link

The docker philosopy says "one container per service" ;-)
I test with another lamp-image now.
Personally I could live with a docker-compose stack of one app-container with PHP7-FPM and the app, and a 2nd container with mariadb.

@crramirez
Copy link
Owner

We maybe this is the way now and I can make a script to migrate users from the old container to the newer, but they will have to learn docker-compose. There are other LimeSurvey containers with this split approach. If I separate the containers it is not clear for me which could be the differentiator from other containers.

@stefangweichinger
Copy link

What about this image as a base:

https://hub.docker.com/r/mattrayner/lamp

?

@crramirez
Copy link
Owner

Looks good

@stefangweichinger
Copy link

I have a very raw draft of a Dockerfile that prepares the git-tag checkout from upstream in an intermediate container.
The second container is based on above image and so far I only unpack the code etc
This is raw and not yet ready for a PR. But you might be faster in seeing which steps of your v3-Dockerfile are still necessary and which not. Interest?

@crramirez
Copy link
Owner

Yes,

You can create a draft PR

Thank you

@stefangweichinger
Copy link

see #41

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

No branches or pull requests

3 participants