Skip to content

Add R and apt packages to container image #1239

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

Merged
merged 2 commits into from
Sep 10, 2021

Conversation

korlaxxalrok
Copy link
Contributor

Description

Add some necessary packages

Changelog

  • Dockerfile: add packages

- Adds apt packages for sending mail and connecting to sftp
Copy link
Contributor

@nmdefries nmdefries left a comment

Choose a reason for hiding this comment

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

Everything seems to install correctly, but sendmail hangs if I try to call it. Is there additional configuration we need to do for sendmail?

@korlaxxalrok
Copy link
Contributor Author

Ah, good catch. I'll check.

- Add the ssmtp package and a simple config file
@korlaxxalrok
Copy link
Contributor Author

@nmdefries I think this will work.

We'll add ssmtp and a simple config to the container that will relay mail to the container host.

We'll have to run the container with the host networking option (--net=host) so that we have simple access to the host via localhost, and then the host will relay again through a specific server based on the email domain of the recipient (automagically configured as best I can tell, but this appears to be the mechanism on our VMs owned by CMU). I've tested this on a bigchunk with success to both a CMU and Google account.

This will also require a simple change to the postfix config on the container host.

It will be hard to test this locally, but if you have access to a bigchunk (or potentially other system that is part of the delphi.cmu.edu domain) you can build the container there and it should work 🤞

This introduces another dependency on the container host, but later on we can find a way to use an external mail relay. That would be one of the best ways to decouple.

Copy link
Contributor

@nmdefries nmdefries left a comment

Choose a reason for hiding this comment

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

Looks good. I'm not able to verify that sendmail works (as noted above, doesn't work on any old machine) but it no longer hangs.

@krivard krivard merged commit ac35bdf into docker-survey-automation Sep 10, 2021
@krivard krivard deleted the prepare-docker-survey-automation branch September 10, 2021 18:39
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.

3 participants