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

🐳 Docker Updates #101

Merged
merged 26 commits into from
Feb 8, 2022
Merged

🐳 Docker Updates #101

merged 26 commits into from
Feb 8, 2022

Conversation

joshuanianji
Copy link
Owner

@joshuanianji joshuanianji commented Feb 7, 2022

  • Update .dockerignore.

  • Use experimental (and possibly unsafe) scratch-node base image to reduce my docker images. This makes it so my images CVE-2021-3807.

UPDATE: I'm forking the scratch-node repo so I can add dumb-init to it.
UPDATE 2: I ended up building on top of scratch-node to make a new base image scratch-node-dumb-init, since I was waiting upwards of 4 hours for the CI script to build the image (I never even got it to work!).

@netlify
Copy link

netlify bot commented Feb 7, 2022

✔️ Deploy Preview for treehouse-bot ready!

🔨 Explore the source changes: 0cedcab

🔍 Inspect the deploy log: https://app.netlify.com/sites/treehouse-bot/deploys/6202c5e1cbde100007852281

😎 Browse the preview: https://deploy-preview-101--treehouse-bot.netlify.app

@joshuanianji joshuanianji temporarily deployed to dev February 7, 2022 06:19 Inactive
@joshuanianji joshuanianji changed the title 🐳 Update .dockerignore 🐳 Dockerfile Updates Feb 7, 2022
@joshuanianji joshuanianji changed the title 🐳 Dockerfile Updates 🐳 Docker Updates Feb 7, 2022
@joshuanianji joshuanianji linked an issue Feb 8, 2022 that may be closed by this pull request
@joshuanianji
Copy link
Owner Author

When I use a distroless Node base image, it won't have npm, so #5 will be fixed.

Image size now down to `46.97MB` for bot, `46.17MB` for server!
@joshuanianji joshuanianji temporarily deployed to dev February 8, 2022 09:15 Inactive
Might save up to 1:30m!
@joshuanianji joshuanianji temporarily deployed to dev February 8, 2022 16:14 Inactive
@joshuanianji joshuanianji temporarily deployed to dev February 8, 2022 19:22 Inactive
@joshuanianji joshuanianji temporarily deployed to dev February 8, 2022 19:36 Inactive
@joshuanianji joshuanianji merged commit 7037359 into main Feb 8, 2022
@joshuanianji joshuanianji deleted the update-dockerignore branch February 8, 2022 19:37
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.

CVE-2021-3807 Vulnerability
2 participants