-
-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updates to allow ssmtp to work out of the box #40
Conversation
Can an admin please whisper to jenkins to test this PR. |
You have to find another way to get the ssmtp in the /etc folder. Symlinking to a volume mount is not something we want. Symlinks inside the container is ok. |
Jenkins test this please |
Sorry. I didn't remember that I commented here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you change it to a copy only if exist?
Same format as the symlink part in the 30-config.
@tobbenb done |
9c53a9c
to
e64f8ce
Compare
LGTM |
sendmail
command in pathnames so it utilizes ssmtpWithout these changes I was not able to the docker container to send email notifications.