Bug 1375328 - Allow us to configure the rails 5 web server
Summary: Allow us to configure the rails 5 web server
Keywords:
Status: CLOSED DUPLICATE of bug 1372439
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: GA
: 5.7.0
Assignee: Dan Clarizio
QA Contact: Dave Johnson
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-12 17:43 UTC by John Prause
Modified: 2016-09-13 05:05 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-09-12 18:53:44 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description John Prause 2016-09-12 17:43:10 UTC
Purpose or Intent

Puma handles each request in a new thread, leading to possible thread issues
with the database connection pool, ActiveSupport::Dependencies interlock and our
locks, or thread unsafe code in our application.

Thin, while it may be run in threads via --threaded, does not use threads by default.
See here

This patch allows for thin 1.7.0+ and gives us an option to use either puma or
thin. Note, earlier versions are not compatible with rack 2.

We still default to puma.
Steps for Testing/QA

Note: Testers/users, the proctitle for UI/Webservice and web socket workers will
look different in ps, top, etc. if you use thin instead of puma. Puma configures
it's own proctitle and we configure the parts that we can. Thin does not, so it
will look like all the other workers.

For example:

thin:
43177 ttys002 0:08.20 MIQ: MiqUiWorker id: 158, uri: http://0.0.0.0:3000

puma:
43871 ttys004 0:00.68 puma 3.3.0 (tcp://0.0.0.0:3000) [MIQ: Web Server Worker]

Comment 2 John Prause 2016-09-12 17:43:44 UTC
https://github.com/ManageIQ/manageiq/pull/10447

Comment 3 Satoe Imaishi 2016-09-12 18:53:44 UTC

*** This bug has been marked as a duplicate of bug 1372439 ***


Note You need to log in before you can comment on or make changes to this bug.