Bug #21295
closedSlow start of rails environment when starting passenger or foreman-tasks
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1493046
Description of problem:
In satellite 6.3, the time for everything to be ready to respond
to http requests or handle tasks is significantly longer than in 6.2.
This leads to potential initial timeouts when trying to register
a host after the installation finishes or after restarting of services.
Steps to Reproduce (a):
1. systemctl restart httpd
2. wait until the /users/login is available
Actual results:
It takes about 2 minutes to be ready
Expected results:
On similar setup on satellite 6.2, it takes under a minute
Steps to Reproduce (b):
1. systemctl restart foreman-tasks
2. watch /var/log/foreman/dynflow_executor.output for containing 'Everything ready for world'
Actual results:
It takes about 3.5 minutes to be ready
Expected results:
On similar setup on satellite 6.2, it takes under a minute
Additional info:
I've narrowed it down to /opt/rh/rh-ror42/root/usr/share/gems/gems/railties-4.2.6/lib/rails/application/routes_reloader.rb , that runs twice during the boot time and seems to cause the significant slow down at startup. It seems that
something changed between rails 4.1 and 4.2 to cause the delays.
The fix should make sure we don't start reloading the routes twice at the initialization, as that's what causes the long time to start.
Setting the severity on high, as it has impact not only on response times after the installation finishes, but also when the passenger scales up the processes.
Updated by Adam Ruzicka over 7 years ago
- Category set to Foreman plugin
- Assignee set to Ivan Necas
- Target version set to 226
Updated by Ivan Necas over 7 years ago
- Project changed from foreman-tasks to Foreman
- Category changed from Foreman plugin to Rails
Updated by The Foreman Bot about 7 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4908 added
Updated by Marek Hulán about 7 years ago
- Translation missing: en.field_release set to 296
Updated by Ivan Necas about 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset c2e58b1b275e0107de4089e15adc8b6203e696b1.
Updated by Ivan Necas almost 7 years ago
- Related to Bug #22743: Update failed at migrate_foreman: "can't modify frozen Array" added