RefugeRestrooms / refugerestrooms

REFUGE restrooms indexes and maps safe restroom locations for trans, intersex, and gender nonconforming individuals.

http://www.refugerestrooms.org

GNU Affero General Public License v3.0

892 stars 262 forks source link

Revert "Enable puma clustered (#654)" #659

Closed DeeDeeG closed 3 years ago

DeeDeeG commented 3 years ago

Context

Revert https://github.com/RefugeRestrooms/refugerestrooms/pull/654 / https://github.com/RefugeRestrooms/refugerestrooms/commit/6c63546e3047bfddcf6d51f06ef51921ba4f15be
Go back to using a single process of Puma.
Documentation at Heroku for this config option, in case anyone is curious: https://devcenter.heroku.com/articles/deploying-rails-applications-with-the-puma-web-server#workers

Summary of Changes

Go back to using a single process of Puma.
- We don't have metrics/traffic numbers to motivate the switch to multiple workers on a speed/latency basis. And the app hasn't been thoroughly tested with the multiple workers.
- We honestly don't understand the potential implications of multiple workers.
- Revert out of an abundance of caution, as we're making a new release to production soon.

Checklist

[ ] Tested Mobile Responsiveness
[ ] Added Unit Tests
[x] CI Passes
[ ] Deploys to Heroku on test Correctly (Maintainers will handle)
[ ] Added Documentation (Service and Code when required)