Summary: | mail-filter/amavisd-new - amavisd init script fails to restart when amavisd has abnormally terminated | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Robin Lutz <robin.m.lutz> |
Component: | Current packages | Assignee: | Net-Mail Packages <net-mail+disabled> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | antivirus, duncan, mjo, nick |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Patch for old amavis.initd
unified patch amavisd.initd-r2 |
Description
Robin Lutz
2014-04-10 23:19:37 UTC
Created attachment 374710 [details, diff]
Patch for old amavis.initd
Comment on attachment 374710 [details, diff]
Patch for old amavis.initd
Please attach unified diffs.
Created attachment 374958 [details, diff]
unified patch
I don't know if my problem is related to yours, but I cannot stop or reload amavisd-new on two of my Gentoo boxes. /etc/init.d/amavisd reload * Reloading amavisd-new ... The amavisd daemon is not running /etc/init.d/amavisd stop * Stopping amavisd-new ... The amavisd daemon is not running * ERROR: amavisd failed to stop BUT: amavisd-new is up and running !! Plz forget my last comment - just found the 'bug'. It seems that my system clock after boot has set to be 1 hour in the future. After setting the clock to 'normal' time, the amavis daemon is no longer able to stop or reload. However if I stop amavis before resetting my time and start it again, reload and stop seems to work... Created attachment 555284 [details]
amavisd.initd-r2
I have a new init script that I think fixes this. I'm going to commit it after I test for a while, along with version bump for amavisd-new itself. I would be interesting in hearing if this solves your problem (I think it does).
I just tested this on our setup, and it does indeed seem to fix this bug as well as #646336 based on testing by killing the master amavis service. With the current mainline init it will show as running in rc-status and refuse to restart until zapped. After switching to the new init script, these behaviours are corrected. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c8fe3982751aa4881b42f89ed080b210c4529c81 commit c8fe3982751aa4881b42f89ed080b210c4529c81 Author: Michael Orlitzky <mjo@gentoo.org> AuthorDate: 2018-11-17 15:09:21 +0000 Commit: Michael Orlitzky <mjo@gentoo.org> CommitDate: 2018-11-18 23:23:57 +0000 mail-filter/amavisd-new: new version 2.11.1. A mostly standard version bump (bug 668494) with some improvements and bug fixes piled on: * The sys-apps/file dependency is replaced by dev-perl/File-LibMagic to improve performance a bit (bug 592802). * New user creation has been moved to pkg_setup(). This allows us to set permissions and ownership properly in src_install(), so that we don't have to "fix" them later and cause big ol' security problems (bug 630836). * The OpenRC service script has been rewritten to use start-stop-daemon. This fixes outstanding bugs 507352, 634860, and 646336. * The systemd service dependencies have been updated (bug 581452). We never really needed spamassassin (spamd) at all, and we shouldn't fail to start if postfix/clamav are absent entirely. * As part of the previous item, removed two failing "sed" calls (that were missing die() statements!) intended to modify a service file that no longer lives where it used to. Thanks are due to, * Marcin Mirosław who reported an OpenRC issue, * Nick Wiltshire for reporting and testing the OpenRC fixes, * Robin Lutz for reporting the OpenRC restart issue, * Timo Rothenpieler for reporting the systemd dependency issue. Bug: https://bugs.gentoo.org/630836 Closes: https://bugs.gentoo.org/507352 Closes: https://bugs.gentoo.org/581452 Closes: https://bugs.gentoo.org/592802 Closes: https://bugs.gentoo.org/634860 Closes: https://bugs.gentoo.org/646336 Closes: https://bugs.gentoo.org/668494 Signed-off-by: Michael Orlitzky <mjo@gentoo.org> Package-Manager: Portage-2.3.51, Repoman-2.3.11 mail-filter/amavisd-new/Manifest | 1 + ...-2.11.0-r4.ebuild => amavisd-new-2.11.1.ebuild} | 68 ++++++++++++---------- mail-filter/amavisd-new/files/amavisd.initd-r2 | 42 +++++++++++++ mail-filter/amavisd-new/files/amavisd.service-r1 | 21 +++++++ 4 files changed, 101 insertions(+), 31 deletions(-) |