Bug 632846
Summary: | sa-update.cron needs better status, configurability | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | tlhackque | ||||
Component: | spamassassin | Assignee: | Warren Togami <wtogami> | ||||
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | low | ||||||
Version: | 13 | CC: | jik, kevin, mrunge, nb, wtogami | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | spamassassin-3.3.2-0.2.svn1027144.fc14 | Doc Type: | Bug Fix | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2010-11-06 23:48:23 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
This looks pretty nice from a quick glance here. Don't we need to add /var/log/sa-update.log to logrotate however? Or it will just grow without bound. When DEBUG is set, perhaps add -v to sa-update to note what exact channels were updated? Or might that be worth doing all the time for the log? Thanks for your review. 1) /etc/logrotate.d/sa-update already exists and covers this log. (It's not a new log - it's that it never had any useful content before.) Here's the file as it was installed on my system: /etc/logrotate.d/sa-update ========================== /var/log/sa-update.log { monthly notifempty missingok } I don't think that the improved logging will require changes to logrotate... 2) Sure, you could add -v to sa-update under DEBUG. [ -n "$DEBUG" ] && OPTIONS="$OPTIONS -v". But if you're editing the sysconfig file, you could just as easily add -v to the OPTIONS... I don't have a strong opinion on this. It would be harder to do something different for the log from what gets mailed. You'd have to do something like a sed filter; maybe even changes to sa-update to get everything on stdout/err as appropriate. I didn't look into that; I think going this way would be overkill. Ah right. Sounds good. Do you have a sample /etc/sysconfig/sa-update file as well? Or shall I whip one up here? I don't have any further objections personally... /etc/sysconfig/sa-update is already distributed, but of course didn't describe the options added by this patch. To document all the options, that could be replaced with this: # sa-update configuration # # Note that the opposite of "yes" is the empty string, NOT "no" # Don't run sa-update even if it's in /etc/cron.d/ - as installed #SAUPDATE=no # Run sa-update even if no daemon is detected #SAUPDATE=yes # Default: Run only if a daemon is detected # Options for the actual sa-update command # These are added to the channel configuration from /etc/mail/spamassassin/channel.d/*.conf # OPTIONS=-v # Debug script - send mail even if no update available # DEBUG=yes # Send mail when updates successfully processed # Default: send mail only on error #NOTIFY_UPD=yes spamassassin-3.3.2-0.1.svn1027144.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/spamassassin-3.3.2-0.1.svn1027144.fc14 spamassassin-3.3.2-0.1.svn1027144.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update spamassassin'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/spamassassin-3.3.2-0.1.svn1027144.fc14 /etc/sysconfig/sa-update is broken. Negative karma filed. Sorry, I should have given details... Line 15 is supposed to be a comment but isn't properly commented. Looks like bugzilla wrapped with a hard return; 'from ' and '/etc...' were on the same line when I provided the sample... Yeah. ;( Fixed and pushed a new build. Can you re-test and re-add karma? I can confirm, the new build fixes (at least the issue from comment #7 ff) Update has been created in bodhi, can you please test spamassassin-3.3.2-0.2.svn1027144.fc14 and provide karma in bodhi if this fixes your issue? If we get 2 more positive karma we can go ahead and push it to stable. spamassassin-3.3.2-0.2.svn1027144.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update spamassassin'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/spamassassin-3.3.2-0.2.svn1027144.fc14 spamassassin-3.3.2-0.2.svn1027144.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. |
Created attachment 446655 [details] Patch to provide requested features Description of problem: The spamassassin package installs /etc/cron.d/sa-update and /usr/share/spamassassin/sa-update.cron to automatically update the rulesets. 1) There is no way to disable the automatic update without touching a 'do not edit' file 2) The log file created by the cron job isn't very useful 3) There is no mechanism to obtain notification of successful ruleset updates. 4) For people who want to call sa-update.cron from another script (such as bayes training), it would be useful if valid exit status were produced. The attached patch addresses these issues. /etc/sysconfig/sa-update controls all features: SAUPDATE can take the value "no". This will cause the script to exit without looking for any running daemons. OPTIONS will be passed to sa-update - this can be used to pass -D (debug) or other local choices without having to edit the script. NOTIFY_UPD=yes will cause cron to send mail when an update is successfully applied. DEBUG=yes will also cause cron to send mail when no update is available Any other exit status will be translated to text and unconditionally sent as mail. These are all failures that require attention. Finally, all exit status are timestamped and logged as text in /var/log/sa-update.log, and used as the script's exit status. Version-Release number of selected component (if applicable): spamassassin.x86_64 3.3.1-2.fc13