Skip to content

Improved documentation for SA_KILL#1233

Merged
erik-wramner merged 2 commits intodocker-mailserver:masterfrom
jamesfryer:master
Aug 21, 2019
Merged

Improved documentation for SA_KILL#1233
erik-wramner merged 2 commits intodocker-mailserver:masterfrom
jamesfryer:master

Conversation

@jamesfryer
Copy link
Copy Markdown
Contributor

No description provided.

@erik-wramner
Copy link
Copy Markdown
Contributor

Thanks!

Unfortunately I don't think the path is correct. It depends on several things. The mail state is only used with ONE_DIR=1, otherwise the files are in other locations. Furthermore the exact path that docker uses for volumes is likely to differ. Docker can provision volumes with many drivers.

I think you should document the paths that are used inside the container. There will be one path with ONE_DIR=1 and another with ONE_DIR=0.

@jamesfryer
Copy link
Copy Markdown
Contributor Author

jamesfryer commented Aug 20, 2019

I believe the paths inside the container will be:

ONE_DIR=1: /var/mail-state/lib-amavis/virusmails/
ONE_DIR=0: /var/lib/amavis/virusmails/

Does that look right to you?

@erik-wramner
Copy link
Copy Markdown
Contributor

Yes, that should be correct. The start script sets the state directory for amavis to /var/lib/amavis or /var/mail-state/lib-amavis depending on ONE_DIR.

@jamesfryer
Copy link
Copy Markdown
Contributor Author

I have changed it as you suggest. I now worry it is turning into an essay.

@erik-wramner erik-wramner merged commit 47a9ad9 into docker-mailserver:master Aug 21, 2019
@erik-wramner
Copy link
Copy Markdown
Contributor

Well, it is a bit long, but at least it is informative. Thanks!

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