Combined the master and base docker images to create an updated, minimized image #2378
+703
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… and dependency only) image based and the latest Debian images Moved from from tar to cp and shell scripting for consistency and correct racing conditions documented in the shell script Added erlang-base, curl, and ca-certificates Copied ca-certificates, so Curl functions properly (including internally to freeswitch) Fixed sound file downloading Increased the complexity of the default generated password Merged sudo process, but moved to su-exec from gosu, to clear vulnerabilities Update Debian distro before freeswitch install, to keep dependencies fully up to date Updated to the latest Busybox
latest has minimal dependencies python includes python perl includes perl java includes java full includes python perl and java Also added additional openssl dependencies to all images
|
The PR drone seem to consistently fail when another commit is added to the PR, on the notify step, not on an actual test. Anyhow, those unit tests are irrelevant to this PR and no code being tested was updated. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Most of the concepts are taken from the docker images in the master and base_image folders. It's now all done in one Dockerfile, using buildkit and a staging image. It also includes build hooks so the Token can be added as an environment variable, on Docker Hub, for automated builds and security scans.
Other enhancements to the updated docker image:
This is currently being auto-built at https://hub.docker.com/r/dheaps/freeswitch.