Skip to content

Fix for #552#738

Merged
johansmitsnl merged 7 commits intodocker-mailserver:masterfrom
kamuri:issue-552
Oct 10, 2017
Merged

Fix for #552#738
johansmitsnl merged 7 commits intodocker-mailserver:masterfrom
kamuri:issue-552

Conversation

@kamuri
Copy link
Copy Markdown
Contributor

@kamuri kamuri commented Oct 7, 2017

No description provided.

@kamuri kamuri closed this Oct 7, 2017
@kamuri kamuri reopened this Oct 7, 2017
@kamuri
Copy link
Copy Markdown
Contributor Author

kamuri commented Oct 7, 2017

Ok tested everything. All works now as expected.

postfix restart via supervisorctl is buggy.
see.: docker-mailserver#552 (comment)
Comment thread target/check_for_changes.sh Outdated


#supervisorctl restart postfix # Buggy causes error while testing.
postfix reload
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is my only concern because the process might lose the pid. Is the reload nesesary?
When we use the postmap a reload is not needed (http://www.postfix.org/postmap.1.html)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants