puppet-ironic/releasenotes/notes/update-threads-to-match-upstream-6fda517d7f6d1448.yaml
Julia Kreger 037cd2cb5d Correct default number of threads
The default number of threads does not match the default
tested upstream[1], which can return errors for users when
requests begin backing up in the webserver due to a lack
of resources and the nature of the ironic API having some
synchronous requests.

Upstream ironic, as of the Rocky cycle, executed with uwsgi
which is configured for the same process count algorithm,
but enable threds. There does not appear to be an upper bound
to that setting, but accoriding to mod_wsgi documentation[3],
the default is 15 threads. The example shipped with ironic[4]
defaults to 10, which also seems to be a valid configuration.

[1]: https://git.openstack.org/cgit/openstack/ironic/tree/devstack/lib/ironic?h=stable/rocky#n1129
[2]: http://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/apache#n267
[3]: https://modwsgi.readthedocs.io/en/develop/configuration-directives/WSGIDaemonProcess.html
[4]: https://git.openstack.org/cgit/openstack/ironic/tree/etc/apache2/ironic?h=stable/rocky#n21

Story: #2003464
Task: #24724
rhbz: #1613995

Change-Id: I327219f96cfc0a69643ab6369cb8e831251f799c
2018-08-16 15:08:04 -07:00

10 lines
437 B
YAML

---
fixes:
- |
Fixes default configuration where only a single concurrent thread was
permitted per WSGI worker process, which does not match the upstream
configuration where threads are enabled per each spawned worker process.
This allows for greater request concurrency to prevent situations where
the webserver may incorrectly return a 500 error due to the server being
unable to process the request in time.