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

Wrong docker-compose definition using container name instead of service name? (Update docker-compose content inside README.md) #56

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

Conversation

schnillerman
Copy link

@schnillerman schnillerman commented Jun 4, 2024

Doesn't DB_HOST need to be set to the service name instead of the container name?

At least for me, firing up the containers only worked after I changed this to service name (DB_HOST=db instead of, e.g., DB_HOST=xwiki-mariadb-db).

Before, I would get some like 25.000 java exceptions and errors on initial container start-up.

Doesn't DB_HOST need to be the service name instead of the container name?
@schnillerman schnillerman changed the title Update README.md Update README.md: Wrong docker-compose definition? Jun 4, 2024
@schnillerman schnillerman changed the title Update README.md: Wrong docker-compose definition? Wrong docker-compose definition using container name instead of service name? (Update docker-compose content inside README.md) Jun 4, 2024
Copy link
Author

@schnillerman schnillerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works better for me ...

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.

1 participant