Project

General

Profile

Actions

Bug #24927

closed

Ansible processes might get killed when logrotate runs for smart_proxy_dynflow_core

Added by Adam Ruzicka about 6 years ago. Updated about 6 years ago.


Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1628505

Description of problem:
We used systemctl kill ... to deliver SIGUSR1 to smart_proxy_dynflow_core to make it reopen its logs. While systemctl kill delivered the signal reliably, it delivered the signal to every process in the service's control group. The issue was when the service forked more processes (when running ansible-playbook). pkill signals only the processes with matching name.

Ansible-playbook reacts to SIGUSR{1,2} by printing "User defined signal
1" and exiting, which is undesired.

Version-Release number of selected component (if applicable):
6.4

How reproducible:
Always

Steps to Reproduce:
1. Run sleep 1000 using the ansible command template
2. When the job runs, run logrotate -f /etc/logrotate.conf -v
3. (optional) Watch the logrotate output as it may not rotate the logs if they were rotated recently

Actual results:
Logs are rotated, ansible processes die. Possible IOError shown in the jobs output

Expected results:
Logs are rotated, running ansible processes are left intact.

Additional info:

Actions #1

Updated by Adam Ruzicka about 6 years ago

  • Subject changed from Ansible processes might get killed when logrotate runs for smart_proxy_dynflow_core to Ansible processes might get killed when logrotate runs for smart_proxy_dynflow_core
  • Status changed from New to Ready For Testing
  • Assignee set to Adam Ruzicka
  • Pull request https://github.com/theforeman/foreman-packaging/pull/2939, https://github.com/theforeman/foreman-packaging/pull/2940 added
Actions #2

Updated by Ivan Necas about 6 years ago

  • Status changed from Ready For Testing to Closed
Actions #3

Updated by The Foreman Bot about 6 years ago

  • Pull request https://github.com/theforeman/foreman-packaging/pull/3054 added
Actions #4

Updated by The Foreman Bot about 6 years ago

  • Pull request https://github.com/theforeman/foreman-packaging/pull/3055 added
Actions #5

Updated by Tomer Brisker about 6 years ago

  • Fixed in Releases 1.20.0 added
Actions

Also available in: Atom PDF