Bug 2230250
Summary: | "foreman-rake audits:expire" cron job exited without running | ||
---|---|---|---|
Product: | Red Hat Satellite | Reporter: | Hao Chang Yu <hyu> |
Component: | Audit Log | Assignee: | Adam Lazik <alazik> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Satellite QE Team <sat-qe-bz-list> |
Severity: | high | Docs Contact: | |
Priority: | unspecified | ||
Version: | 6.13.3 | CC: | ahumbe, alazik, apetrova, aruzicka |
Target Milestone: | Unspecified | Keywords: | Documentation, Triaged |
Target Release: | Unused | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2024-02-22 14:33:25 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Hao Chang Yu
2023-08-09 07:10:10 UTC
The cron and the setting were added as part of https://bugzilla.redhat.com/show_bug.cgi?id=1703496 . Digging through history a bit this sounds intentional, or at least per https://projects.theforeman.org/issues/27024#note-4 . The infrastructure is there in case anyone needs it, in which case they can just set a setting instead of fiddling with crons. Would it be ok if we updated the documentation instead? (In reply to Adam Ruzicka from comment #1) > The cron and the setting were added as part of > https://bugzilla.redhat.com/show_bug.cgi?id=1703496 . Digging through > history a bit this sounds intentional, or at least per > https://projects.theforeman.org/issues/27024#note-4 . The infrastructure is > there in case anyone needs it, in which case they can just set a setting > instead of fiddling with crons. > > Would it be ok if we updated the documentation instead? There is a risk. The customer expect it to run and do the clean up because it has been like that for long time. If this script stop running suddenly, customers might not notice that and the database size can blow up slowly until the storage size is full. What has been like that for a long time? (In reply to Adam Ruzicka from comment #4) > What has been like that for a long time? the 90 days default Now this is a bit tricky. If I'm reading things right, before 6.12 there was a default cleanup interval, but it wasn't set up by us to run periodically. The users had to go in and set it up themselves. With 6.12 the situation flipped - it is set up to run periodically ootb, but the interval is unset so it doesn't do anything. Is this assessment correct or have I missed anything? (In reply to Adam Ruzicka from comment #6) > Now this is a bit tricky. If I'm reading things right, before 6.12 there was Before 6.11 > a default cleanup interval In 6.10 (foreman-2.5.2.21-1.el7sat.noarch), /usr/share/foreman/lib/tasks/audits.rake ~~~ def before_date @before_date ||= ENV['days'] ? ENV['days'].to_i.days.ago : 90.days.ago end ~~~ >, but it wasn't set up by us to run periodically. > The users had to go in and set it up themselves. Yes, I think you are right. The cron job is not in 6.10. In this case, it is probably fine to just correct the documentation? Alright, thank you. Hi! I prepared a draft of the new documentation that clarifies the automatic and manual deletion of audits. This draft will go through an SME review, and as such, the contents are subject to change. https://github.com/theforeman/foreman-documentation/pull/2738 The new documentation has been published: 6.14: https://access.redhat.com/documentation/en-us/red_hat_satellite/6.14/html-single/administering_red_hat_satellite/index#Deleting_Audit_Records_Automatically_admin 6.13: https://access.redhat.com/documentation/en-us/red_hat_satellite/6.13/html-single/administering_red_hat_satellite/index#Deleting_Audit_Records_Automatically_admin 6:12: https://access.redhat.com/documentation/en-us/red_hat_satellite/6.12/html-single/administering_red_hat_satellite/index#Deleting_Audit_Records_Automatically_admin |