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

Simplify docker compose config #244

Merged
merged 11 commits into from
Apr 18, 2023
Merged

Simplify docker compose config #244

merged 11 commits into from
Apr 18, 2023

Conversation

delano
Copy link
Collaborator

@delano delano commented Apr 14, 2023

Changes

  • Fixed Dockerfile to correctly build from the multi-stage image container
  • Added the current working directory as a volume for the app container (so that local changes are reflected in the running container at the sam time)
  • Added persistent volume for redis:/data
  • Added .env.example
  • Updated docker compose config to now:
    • Load env vars for the app container from .env
    • Properly use the entrypoint.sh script, with the actual command now being passed through from the command: setting in the docker-compose.yml file.
    • stdin: and tty: are now both set to true. This makes it possible to attach and interact with a running container.
  • Updated bundler to latest
  • Removed Gemfile.lock from repo and .dockerignore
  • Simplified redis.conf

to properly install system utilities and use the build container

Signed-off-by: delano <delano@onetimesecret.com>
Signed-off-by: delano <delano@onetimesecret.com>
To properly use bin/entrypoint.sh and to mount the cwd inside the container to skip having to rebuild for every change

Signed-off-by: delano <delano@onetimesecret.com>
Signed-off-by: delano <delano@onetimesecret.com>
@delano delano added bug improvement dependencies Pull requests that update a dependency file labels Apr 14, 2023
@delano delano self-assigned this Apr 14, 2023
Signed-off-by: delano <delano@onetimesecret.com>
And added a `bundle install` to the entrypoint script so that we only
need to restart the container rather than a full rebuild.

Signed-off-by: delano <delano@onetimesecret.com>
Signed-off-by: delano <delano@onetimesecret.com>
* Switched to thin
* Run the command via bundle exec
* Mount the current working directory as $CODE_HOME
* Add container names
* Added redis volume to persist data

Signed-off-by: delano <delano@onetimesecret.com>
sendgrid-ruby

Signed-off-by: delano <delano@onetimesecret.com>
Signed-off-by: delano <delano@onetimesecret.com>
Signed-off-by: delano <delano@onetimesecret.com>
@delano delano force-pushed the delano/20230414-simplify-dc branch from 416f553 to 78a07d9 Compare April 18, 2023 04:27
@delano delano merged commit f5ef0e7 into develop Apr 18, 2023
@delano delano deleted the delano/20230414-simplify-dc branch April 18, 2023 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug dependencies Pull requests that update a dependency file improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant