Skip to content

Commit

Permalink
Add Puma via govuk_app_config
Browse files Browse the repository at this point in the history
This commit adds `Puma` via govuk_app_config. Puma was previously installed indirectly through the dependency on `govuk_test` as part of the `dev` and `test` groups; given that we want to run Puma in production, we are instead configuring it via govuk_app_config.

Co-authored-by: Chris Banks <chris.banks@digital.cabinet-office.gov.uk>
  • Loading branch information
Karl Baker and sengi committed Oct 20, 2021
1 parent 7cdd3e0 commit cf4c799
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 1 addition & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ RUN apt-get update -qq && apt-get upgrade -y
RUN curl -sL https://deb.nodesource.com/setup_9.x | bash -
RUN apt-get update && apt-get install -y nodejs && apt-get install npm -y
RUN npm install -g phantomjs-prebuilt@2 --unsafe-perm
RUN gem install foreman

# This image is only intended to be able to run this app in a production RAILS_ENV
ENV RAILS_ENV production
Expand All @@ -28,4 +27,4 @@ RUN GOVUK_APP_DOMAIN=www.gov.uk GOVUK_WEBSITE_ROOT=www.gov.uk bundle exec rails

HEALTHCHECK CMD curl --silent --fail localhost:$PORT || exit 1

CMD foreman run web
CMD bundle exec puma
2 changes: 2 additions & 0 deletions config/puma.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
require "govuk_app_config/govuk_puma"
GovukPuma.configure(self)

0 comments on commit cf4c799

Please sign in to comment.