Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1750030 - Deleted log files kept open until the respective service is restarted
Summary: Deleted log files kept open until the respective service is restarted
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Foreman Proxy
Version: 6.5.0
Hardware: All
OS: Linux
high
high
Target Milestone: 6.7.0
Assignee: Lukas Zapletal
QA Contact: Lukas Pramuk
URL:
Whiteboard: hotfix_delivered
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-09-07 14:52 UTC by Sayan Das
Modified: 2023-03-24 15:24 UTC (History)
12 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1769906 (view as bug list)
Environment:
Last Closed: 2020-04-14 13:25:37 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
Patch for Satellite 6.5 foreman-proxy daemon (4.24 KB, application/mbox)
2019-10-29 10:15 UTC, Lukas Zapletal 		no flags Details
hotfix RPM (156.26 KB, application/x-rpm)
2019-10-30 19:06 UTC, wclark 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 25244 0 Normal Closed Better logging of exceptions 2020-06-25 18:23:47 UTC
Foreman Issue Tracker 27351 0 Normal Closed Turn off rolling log appender by default 2020-06-25 18:23:47 UTC
Foreman Issue Tracker 27509 0 Normal Closed Smart-proxy USR1 reopen does not work 2020-06-25 18:23:47 UTC
Red Hat Product Errata RHSA-2020:1454 0 None None None 2020-04-14 13:25:49 UTC

Description Sayan Das 2019-09-07 14:52:13 UTC 
Description of problem:

When logrotate happens, OS is deleting the files but actually those are present in the list of open files and marked as deleted untill the process is being killed or the respective service is being restarted.


Version-Release number of selected component (if applicable):
Satellite 6.X


How reproducible:
Always


Steps to Reproduce:

1. Wait for the logrotate to happen on foreman-proxy logs.

2. Check by command 'lsof | grep deleted | grep "\/var\/log"' and you will be able to see list of files marked as deleted eating up space(even if it's very less).

3. Restart the service i.e. foreman-proxy in this case and re-execute the command on Step 2 and now the files will have vanished as expected.


Actual results:
When a log file is being deleted by logrotate, It leaves the file descriptor open.


Expected results:
When a log file is being deleted by logrotate, It should not leave a file descriptor open and list it in the list of open files.


Additional info:
NA
Comment 6 Lukas Zapletal 2019-09-09 06:59:07 UTC 
I have a suspicion that this is actually SELinux bug, let's see what customer reports back: https://projects.theforeman.org/issues/19053
Comment 12 Lukas Zapletal 2019-10-29 10:15:25 UTC 
Created attachment 1630091 [details]
Patch for Satellite 6.5 foreman-proxy daemon
Comment 13 wclark 2019-10-30 19:06:06 UTC 
Created attachment 1630713 [details]
hotfix RPM

Hotfix is delivered for Satellite 6.5. Installation instructions:

1. Take a backup of affected Satellites and/or Capsules

2. Download hotfix RPM from this BZ and send it to affected Satellites and/or Capsules

3. yum localinstall /path/to/hotfix/rpm

4. restart services
Comment 17 Lukas Zapletal 2019-11-06 17:57:08 UTC 
Hello, I can confirm that those patches fixes an issue that is related to this problem. When foreman-proxy received a signal, it was supposed to reopen logging file /var/log/foreman-proxy/proxy.log but due to bug it was not doing this at all. I have tested the upstream change, chances are that the original patch was incorrect, or backport has some other issue.

However keep in mind that you must test this with SELinux disabled or in permissive mode because there is another BZ we track related to logrotate - SELinux prevents logrotated from sending a signal to foreman-proxy.

Workaround until this is resolved - set logging level to WARNING in proxy.yml to avoid excessive amount of data in proxy.log.
Comment 20 Lukas Pramuk 2019-11-27 10:43:28 UTC 
VERIFIED.

@satellite-6.7.0-4.beta.el7sat.noarch
foreman-proxy-1.24.0-0.4.RC2.el7sat.noarch

by the following manual reproducer:

1) Check that foreman-proxy is running

# ps -efH | grep foreman\\-proxy
foreman+ 18267     1  0 Nov26 ?        00:00:07   ruby /usr/share/foreman-proxy/bin/smart-proxy --no-daemonize
foreman+ 18556     1  0 Nov26 ?        00:00:13   ruby /usr/bin/smart_proxy_dynflow_core -d -p /var/run/foreman-proxy/smart_proxy_dynflow_core.pid

2) Check that there are already logrotated files (as Satellite is running for couple of days)

# ll /var/log/foreman-proxy/proxy*.gz
-rw-r--r--. 1 foreman-proxy foreman-proxy  222 Nov 25 23:51 /var/log/foreman-proxy/proxy.log-20191126.gz
-rw-r--r--. 1 foreman-proxy foreman-proxy 2532 Nov 26 23:51 /var/log/foreman-proxy/proxy.log-20191127.gz

3) Check for presence of logrotated (deleted) log files being kept open

# lsof | grep '/var/log.* (deleted)'
<empty>

>>> the foreman-proxy log files are logrotated properly without keeping deleted files open
Comment 23 errata-xmlrpc 2020-04-14 13:25:37 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:1454
Note You need to log in before you can comment on or make changes to this bug.