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 Dockerfile #789

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Dockerfile #789

wants to merge 1 commit into from

Conversation

raffus
Copy link

@raffus raffus commented Dec 17, 2024

Fixing adduser missing file

1. Why is this change necessary?

To build docker image

2. What does this change do, exactly?

Docker build was not able to find adduser file

3. Please link to the relevant issues (if any).

Fixing adduser missing file
@papnoisanjeev
Copy link
Collaborator

@raffus
Please check with the latest one if able To build docker image with php 8.1.
https://github.com/uvdesk/community-skeleton/blob/v1.1.7_micro/Dockerfile

@PatrykPlewaOfficial
Copy link

I had big hopes for this PR because I struggled as well to set up UVDesk on Docker (#787).
Unfortunately, after using your Dockerfile I still get the 500 error. No matter if using your setup on PHP7 or mine on PHP8.

Internal Server Error

The server encountered an internal error or misconfiguration and was unable to complete your request.
Please contact the server administrator at webmaster@localhost to inform them of the time this error occurred, and the actions you performed just before this error.
More information about this error may be available in the server error log.

Apache/2.4.58 (Ubuntu) Server at localhost Port 4003

@raffus
Copy link
Author

raffus commented Dec 18, 2024

I had big hopes for this PR because I struggled as well to set up UVDesk on Docker (#787). Unfortunately, after using your Dockerfile I still get the 500 error. No matter if using your setup on PHP7 or mine on PHP8.

Nice.

Internal Server Error

The server encountered an internal error or misconfiguration and was unable to complete your request.
Please contact the server administrator at webmaster@localhost to inform them of the time this error occurred, and the actions you performed just before this error.
More information about this error may be available in the server error log.

Apache/2.4.58 (Ubuntu) Server at localhost Port 4003

You need to run composer install and other commands to run the database migrations inside the docker container.

@raffus
Copy link
Author

raffus commented Dec 18, 2024

@raffus Please check with the latest one if able To build docker image with php 8.1. https://github.com/uvdesk/community-skeleton/blob/v1.1.7_micro/Dockerfile

The problem was with the "adduser" command that needed to be added in Dockerfile, after apt-update.

@PatrykPlewaOfficial
Copy link

You need to run composer install and other commands to run the database migrations inside the docker container.

Sorry @raffus , but I don't get it. The Dockerfile should be responsible for setting up everything that is required.
I'm getting this error after running the fresh container (image built from your Dockerfile). I haven't yet gotten to the setup wizard where you select the database location.

@raffus
Copy link
Author

raffus commented Dec 18, 2024

You need to run composer install and other commands to run the database migrations inside the docker container.

Sorry @raffus , but I don't get it. The Dockerfile should be responsible for setting up everything that is required. I'm getting this error after running the fresh container (image built from your Dockerfile). I haven't yet gotten to the setup wizard where you select the database location.

You're right. I'll fix it and send a PR.

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

Successfully merging this pull request may close these issues.

3 participants