Bug #12295
closedSmart proxy stops listening on HTTPS port after log rotate
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1273785
Description of problem:
The foreman-proxy stops listening on port 9090 after the logrotate was performed:
[crash] root@li-lc-1017:/etc/logrotate.d# grep '04:28' /var/log/cron
Oct 21 04:28:01 li-lc-1017 anacron31376: Job `cron.daily' started
Oct 21 04:28:01 li-lc-1017 run-parts(/etc/cron.daily)[1263]: starting logrotate
Oct 21 04:28:14 li-lc-1017 run-parts(/etc/cron.daily)[1382]: finished logrotate
Oct 21 04:28:14 li-lc-1017 run-parts(/etc/cron.daily)[1263]: starting makewhatis.cron
Oct 21 04:28:18 li-lc-1017 run-parts(/etc/cron.daily)[1555]: finished makewhatis.cron
Oct 21 04:28:18 li-lc-1017 run-parts(/etc/cron.daily)[1263]: starting mlocate.cron
[crash] root@li-lc-1017:/etc/logrotate.d# ls l /var/log/foreman-proxy/. 1 foreman-proxy foreman-proxy 0 Oct 19 12:22 access.log
total 4
-rw-r--r-rw-r--r-. 1 foreman-proxy foreman-proxy 0 Oct 21 04:28 proxy.logrw-r--r-. 1 foreman-proxy foreman-proxy 1366 Oct 21 04:28 proxy.log-20151021.gz
[crash] root@li-lc-1017:/var/log/foreman-proxy# ps aux | grep proxy
root 922 0.0 0.0 103308 832 pts/0 S+ 08:24 0:00 grep proxy
493 1378 0.2 0.1 162960 54032 ? SNl 04:28 0:30 ruby /usr/share/foreman-proxy/bin/smart-proxy
[crash] root@li-lc-1017:/etc/logrotate.d# lsof -P -n | grep 'foreman-proxy.*TCP'
ruby 1378 foreman-proxy 5u IPv4 10120344 0t0 TCP *:8000 (LISTEN)
Checking the logrotate script is does a 'service foreman-proxy condrestart'.
I think that the startup might be too quickly after the stopping so the port 9090 is not released yet for re-use.
Version-Release number of selected component (if applicable):
How reproducible:
Steps to Reproduce:
1. Force logrotate logrotate -f /etc/logrotate.conf
2.
3.
Actual results:
Foreman-proxy is not listening the port 9090
Expected results:
Foreman-proxy is listening again on the port 9090
Additional info:
Updated by Dominic Cleal about 9 years ago
- Status changed from New to Feedback
- Assignee deleted (
Ohad Levy)
Assuming this is Foreman 1.7, please try again with 1.10.0-RC1, the startup code has been refactored since. I can't reproduce any restart failure with condrestart alone.
Updated by Anonymous almost 9 years ago
Apparently the issue is still there. I don't think adding a delay between launching http and https apps is a great approach (slows down startup, 5 seconds may not be sufficient time to for the process to release the socket). Updating 'sigterm' handler to stop http and https apps and then exit smart-proxy would be my preferred approach.
Updated by Dominic Cleal almost 9 years ago
- Subject changed from Capsule stops listening on 9090 after log rotate to Smart proxy stops listening on HTTPS port after log rotate
- Category set to Core
- Priority changed from High to Normal
Or provide another signal handler just for rolling logs, avoiding the need to restart at all.
Updated by Anonymous almost 9 years ago
I like the idea of using a dedicated signal for log rolling even better.
Updated by Anonymous almost 9 years ago
- Status changed from New to Assigned
- Assignee set to Anonymous
Updated by The Foreman Bot almost 9 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/smart-proxy/pull/394 added
Updated by The Foreman Bot almost 9 years ago
- Pull request https://github.com/theforeman/foreman-packaging/pull/1058 added
Updated by The Foreman Bot almost 9 years ago
- Pull request https://github.com/theforeman/foreman-packaging/pull/1060 added
Updated by Anonymous almost 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 7cc71a107f37f1c7dfc12f2791906565e796bed6.
Updated by Dominic Cleal almost 9 years ago
- Translation missing: en.field_release set to 136
Updated by Stephen Benjamin almost 9 years ago
- Related to Bug #14314: Tests are failing due to logging change in smart proxy added
Updated by Stephen Benjamin almost 9 years ago
- Related to Bug #14315: Discovery should no longer use Proxy::Log directly added
Updated by Dominic Cleal over 8 years ago
- Related to Bug #14755: Error during startup, terminating. unsupported signal SIGUSR1 (windows) added
Updated by Ivan Necas over 7 years ago
- Related to Bug #19224: smart_proxy_dynflow_core.log not reopened during logrotate added