Skip to content
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

Clean up Docker containers #706

Merged
merged 28 commits into from
Sep 9, 2022
Merged

Clean up Docker containers #706

merged 28 commits into from
Sep 9, 2022

Conversation

elopez
Copy link
Member

@elopez elopez commented Dec 28, 2021

This PR introduces several changes to the Dockerfile and GitHub Actions used to build the echidna docker container to achieve the following:

  • Single point of distribution: Docker Hub is dropped, and images are only published on GitHub Container Registry now
  • Smaller images: two variants are introduced, ubuntu (~160M) which ships with a shell, apt-get and nvm, and distroless (~100M) which is just echidna and python dependencies. Distroless is currently disabled and left as reference to keep things simple. The current docker image is ~680M.
  • Images with metadata and automatic tagging: by leveraging docker/metadata-action.
  • Caching: building the images now leverages the gha (GitHub Actions) cache type to reduce build times.

The README section on Docker is also updated to account for the changes in this PR.

To keep the same distro we were using previously
@elopez elopez force-pushed the docker-slim branch 3 times, most recently from c8c4b8f to b6b476e Compare August 6, 2022 12:45
Otherwise the final environment is slightly broken if someone tries
to install extra packages
Echidna prints emojis and will throw an error if the locale is not
set correctly.
@elopez elopez marked this pull request as ready for review August 13, 2022 11:16
@ggrieco-tob ggrieco-tob merged commit 1dc209b into crytic:master Sep 9, 2022
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.

None yet

2 participants