Docker image improvements

[karlomikus]

• Switch to PHP-FPM image variant
• Include meilisearch instance in base BA image
• Add empty volume for BA data
  • Move uploads folder and db file to this folder
• Add nginx as default reverse proxy
• Add services by path in a another nginx instance
  • /search for meilisearch, /bar for api
  • Update code to handle this changes
• Use nginx to serve static files for salt-rim
• [Optional] Create a single container image based on alpine

This should hopefully simplify deployment.

Related issues: #12 #26 #37 #50 #59

bar-assistant/docker#2

[zhdenny]

This sounds like a pretty big update/upgrade for docker users.

Could you include any special upgrade instructions when this is released. Like path changes or docker compose changes etc.

Awesome work man!

[karlomikus]

Before all this I will iron out application backup and import features.
Ideally this will not be a huge change to user's docker settings but we will see :D

[zhdenny]

Open a separate issue and provide your full docker compose file.

[damian-w]

+1 for the suggestion to a merge everything into a single container image based on alpine

[karlomikus]

+1 for the suggestion to a merge everything into a single container image based on alpine

This was something I planned to do from the start but it kinda goes against the whole containerization principle. But I will definitely simplify redis and meilisearch dependencies.

[tscibilia]

I tried the new docker info you referenced in this PR bar-assistant/docker#2 but it's still not connecting to the bar assistant server. I'm seeing a "Status: Not available" on the frontend login screen

[karlomikus]

I tried the new docker info you referenced in this PR bar-assistant/docker#2 but it's still not connecting to the bar assistant server. I'm seeing a "Status: Not available" on the frontend login screen

That is still work in progress, and I still haven't released any images for that branch.

But that should be done by the end of this month