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

Add NGINX unit image / or use add to nextcloud-fpm #1063

Open
strannick-ru opened this issue Apr 8, 2020 · 9 comments
Open

Add NGINX unit image / or use add to nextcloud-fpm #1063

strannick-ru opened this issue Apr 8, 2020 · 9 comments

Comments

@strannick-ru
Copy link

strannick-ru commented Apr 8, 2020

Hello.

How about use nginx unit in your nextcloud-fpm images?
For debian its really simple — just add nginx unit repo, install it, add config and make little change into entrypoint.sh.

What is for? You get best from two apps — php-fpm for php and nginx for static files.
It will be like nextcloud-apache image, but suitable for heavy loads.

links:
https://hub.docker.com/r/nginx/unit/
https://unit.nginx.org/

@dm3ch
Copy link

dm3ch commented Sep 9, 2022

@J0WI @tilosp Is there any chance that there would be added official docker image using nginx-unit? Not as a replacement of php-fpm, but as a new distribution option (like apache).
If maintainers are ready to merge such contribution, I'm ready to update existing ananace PR or recreate it from scratch.

P.S. Sorry for mentioning

@J0WI
Copy link
Contributor

J0WI commented Oct 2, 2022

Unfortunately no. There is neither support or documentation from Nextcloud nor is this a common setup in the official Docker library. There is also no official unit image and we cannot use third party base image as proposed in #1610.

So far, there was not much demand for this, but feel free to contribute any documentation or examples.

@tippexs
Copy link

tippexs commented Oct 18, 2023

Hello @J0WI sorry for picking up this "old" issue again. But a lot has changed since 2022 and we are now an official Docker Image and NGINX Unit is a true alternative to PHP-FPM.

I have just created a new configuration for NC 27.1.2. I would love to contribute this if we can find a way adding Unit to the NC update script in some sort. If this is not possible is there a way to contribute to your docs? To me a documentation about how to use NGINX Unit for NC is better placed somewhere on your end as on ours. I am happy to share everything we have to make this happen! Just let me know.

Cheers Timo from NGINX :)

@gaigelama
Copy link

Hello @J0WI sorry for picking up this "old" issue again. But a lot has changed since 2022 and we are now an official Docker Image and NGINX Unit is a true alternative to PHP-FPM.

I have just created a new configuration for NC 27.1.2. I would love to contribute this if we can find a way adding Unit to the NC update script in some sort. If this is not possible is there a way to contribute to your docs? To me a documentation about how to use NGINX Unit for NC is better placed somewhere on your end as on ours. I am happy to share everything we have to make this happen! Just let me know.

Cheers Timo from NGINX :)

I was just checking this thread out a couple days ago and was going to comment that there was now an official NGINX Unit image but didn't have time. I circled back today to see your reply and it couldn't get any better than that lol.

If you get something pushed to Docker Hub or another public registry I can test this out on Nextcloud on k8s and provide feedback.

Thanks for taking an interest and getting something started @tippexs .

@tippexs
Copy link

tippexs commented Oct 19, 2023

Thanks @gaigelama sounds great! I will push my updates to a git repository and push the image to docker hub. Will share it later today with you.

@gaigelama
Copy link

I'm not associated with Nextcloud or this repo so I unfortunately can't speak on their behalf. Taking a look at the open/merged PRs and the CoC though, PRs seem to be welcome. Hopefully a maintainer can better assist you soon.

@joshtrichards joshtrichards changed the title Use nginx unit for nextcloud-fpm image Add NGINX unit image / or use add to nextcloud-fpm Oct 24, 2023
@J0WI
Copy link
Contributor

J0WI commented Oct 26, 2023

If this is not possible is there a way to contribute to your docs? To me a documentation about how to use NGINX Unit for NC is better placed somewhere on your end as on ours.

The canonical documentation lives at https://github.com/nextcloud/documentation. That's also the reference for the nginx examples in this repo. Does unit require any change in the Nextcloud image or is this just an alternative for the nginx example?

@tippexs
Copy link

tippexs commented Nov 12, 2023

Hi @J0WI sorry for my late response!

Unit does not require any changes in the Nextcloud image. It is a simpler way to deploy PHP applications with NGINX without having the need of two seperate processes (NGINX and PHP-FPM).

I have just worked on an issue in our repositories. nginx/unit#959
Let me know if you need a template Dockerfile or configuration.

@ananace
Copy link

ananace commented Dec 16, 2023

I pushed some updates to #1610 - it now results in what looks to be a fully functional install, including support for pretty URLs.

I've in fact moved my personal nextcloud instance onto it to further dogfood the image.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants