coopdevs / timeoverflow

🏦 ⌛ A time banking system
https://www.timeoverflow.org
GNU Affero General Public License v3.0
145 stars 67 forks source link

Add Docker support + Upgrade to Ruby 3 and Rails 7 💫 #722

Closed markets closed 7 months ago

markets commented 9 months ago

WHAT

markets commented 7 months ago

@microstudi I pushed some small tweaks (basically this commit: https://github.com/coopdevs/timeoverflow/commit/e90d8e215c2adc0163bc2b4494b5ae9819cab4c8), but it seems it broke the Docker build job 😅 That's because I made :amazon the default for pro right? Hmmm 🤔 let me know what you think, but it seems a sensible default for pro.

microstudi commented 7 months ago

Conceptually, are you sure to use amazonby default in all environmonents? does not that makes it difficult for developing? What is probably happening is that the test docker compose is precisely using local files to run the app.

If you really want to put amazon as a default, just add this env var in both services (app, sidekiq) in the docker-compose.yml file:

STORAGE_PROVIDER=local
markets commented 7 months ago

I just made :amazon the default in production:

https://github.com/coopdevs/timeoverflow/commit/e90d8e215c2adc0163bc2b4494b5ae9819cab4c8#diff-da60b4e96eff2b132991226d308949e23f4ef3aad45ad59edd09cbc32cc6251eL49

In local and test, it makes sense to use local storage, but for staging, pro or any production-like env, it seems a better default I'd say. Moving local files between different servers would be a pain, but pushing them to S3 make it more flexible.

microstudi commented 7 months ago

I just made :amazon the default in production:

e90d8e2#diff-da60b4e96eff2b132991226d308949e23f4ef3aad45ad59edd09cbc32cc6251eL49

In local and test, it makes sense to use local storage, but for staging, pro or any production-like env, it seems a better default I'd say. Moving local files between different servers would be a pain, but pushing them to S3 make it more flexible.

I see, it's ok. But because the docker compose is for testing the production docker it picks up this value. Just adding the ENV var will solve this then. Go ahead!