Get rid of syslogd warnings about kernel logging.#1711
Get rid of syslogd warnings about kernel logging.#1711erik-wramner merged 3 commits intodocker-mailserver:masterfrom
Conversation
MakerMatrix
commented
Dec 9, 2020
MakerMatrix
commented
- Modify Dockerfile to comment out noisy/errant kernel logging line in /etc/rsyslogd.conf
- Add lines in README.md that prompt users to create an alias that captures messages to postmaster.
- Add commented line in /etc/postfix/main.cf to easily set 'myorigin=$mydomain' if desired. Then e.g. postmaster emails will be addressed from "[email protected]" instead of "[email protected]". This may require a regex or an additional or modified virtual alias for postfix to accept mail from such an address. I have not documented that in these patches. If people watch mail.log it will be clear.
|
Requested a review from @erik-wramner and @fbartels because I am not confidently review these changes - more on the script side myself:) |
|
Damn it, the quota tests again. @martin-schulze-vireso this is the second time close after one another where this test fails. Seems like this hasn't been resolved after all... |
|
It looks like a true failure. The second large mail should not have been delivered but we got a quota message on the third. I am not sure if there is some kind of race condition due to sending the mails too fast. However, that would likely be outside of our control to fix. Maybe we should wait for the first mail to be delivered before sending the next one. |
|
@martin-schulze-vireso you're right. But before doing that, I will see what the next test result bring, whether the quota test is "properly" failing or whether this is random as well. |
| sed -i -e 's/^\(POLICYHELPER=\).*/\1/' /usr/sbin/invoke-rc.d && \ | ||
| # Prevent syslog warning about imklog permissions. | ||
| # We won't want to look at that inside a container anyway. | ||
| sed -i -e 's/^module(load=\"imklog\")/#module(load=\"imklog\")/' /etc/rsyslog.conf && \ |
There was a problem hiding this comment.
I think, the quotes do not need to be escaped:
sed -i -e 's/^module(load="imklog")/#module(load="imklog")/' /etc/rsyslog.conf && \
|
I think this looks good. One thing though. Where we add the postmaster alias we could also add an alias for amavis. There is another PR about adding both in the start script. @aendeavor I think we should either remove the postmaster alias from the readme in this PR and implement it in the other or we should add the amavis alias here. We could of course also do both and then remove the aliases from the readme later when the other PR has been released. |
6 participants
