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

Renaming in Docker setup to mbin + Improved bare metal setup configs #204

Merged
merged 9 commits into from
Nov 5, 2023

Conversation

melroy89
Copy link
Member

@melroy89 melroy89 commented Nov 4, 2023

  • Rename now also the /var/www folder in the Docker setup to mbin
  • Auto format Caddyfile (using the caddy fmt)
  • Improve bare metal setup, including additional realpath cache settings (omitting the opcache.preload script, that caused issues with other PHP websites on the same server I have).
  • Ignore metal directory.

Tested Docker setup.. Everything still seems to work fine:

docker ps
CONTAINER ID   IMAGE                          COMMAND                  CREATED         STATUS                   PORTS                                                                                                           NAMES
b940bd3a59fa   mbin                           "./docker-entrypoint…"   6 minutes ago   Up 6 minutes (healthy)   2019/tcp, 9000/tcp, 0.0.0.0:8008->80/tcp, :::8008->80/tcp                                                       docker-www-1
dfe2eaa5e920   mbin                           "./docker-entrypoint…"   6 minutes ago   Up 6 minutes (healthy)   2019/tcp, 9000/tcp                                                                                              docker-messenger-1
de67262e3516   mbin                           "./docker-entrypoint…"   6 minutes ago   Up 6 minutes (healthy)   2019/tcp, 9000/tcp                                                                                              docker-php-1
9850fe198eb7   mbin                           "./docker-entrypoint…"   6 minutes ago   Up 6 minutes (healthy)   2019/tcp, 9000/tcp                                                                                              docker-messenger_ap-1
9f928ac12034   rabbitmq:3-management-alpine   "docker-entrypoint.s…"   6 minutes ago   Up 6 minutes             4369/tcp, 5671-5672/tcp, 15671/tcp, 15691-15692/tcp, 25672/tcp, 0.0.0.0:15672->15672/tcp, :::15672->15672/tcp   docker-rabbitmq-1
ab140deea391   postgres:16-alpine             "docker-entrypoint.s…"   6 minutes ago   Up 6 minutes             5432/tcp                                                                                                        docker-db-1
a11fd473b43e   redis:alpine                   "docker-entrypoint.s…"   6 minutes ago   Up 6 minutes (healthy)   6379/tcp     

image

This opcache.preload config is killing all my other PHP websites. So thanks no thanks. I rollback this proposal.
Copy link
Member

@nobodyatroot nobodyatroot left a comment

Choose a reason for hiding this comment

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

Also verified all the docker changes still work, looks good to me.

@nobodyatroot nobodyatroot added documentation Improvements or additions to documentation docker Issues and pull requests related to docker environment labels Nov 5, 2023
@melroy89 melroy89 merged commit 2e57057 into main Nov 5, 2023
6 checks passed
@melroy89 melroy89 deleted the rename_docker_mbin branch November 5, 2023 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Issues and pull requests related to docker environment documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants