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

Switch the base image from Debian Bullseye to Alpine LInux 3.16 #55

Merged

Conversation

mcdonnnj
Copy link
Member

@mcdonnnj mcdonnnj commented Sep 20, 2022

🗣 Description

This pull request changes the base image from the 3.10.7-bullseye and 3.10.7-slim-bullseye tags to the 3.10.7-alpine3.16 tag. All necessary changes to support this change are made.

💭 Motivation and context

Alpine Linux is especially suited for use in Docker images when possible due to its minimalist approach. Since everything this project needs is available for Alpine Linux 3.16 it made sense to update the image. The image size went from 748MiB to 457MiB so the space savings are significant.

🧪 Testing

Automated tests pass. I ran the new image locally and compared it to the output of the Debian-based image and there were no real differences (a single organization had a different hash but that could be the result of many things).

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated
    to reflect the changes in this PR.
  • Tests have been added and/or modified to cover the changes in this PR.
  • All new and existing tests pass.

✅ Pre-merge checklist

  • Update version.

Change the operating system of the base image from Debian Bullseye to
Alpine Linux 3.16. All dependencies and the `vdp_scanner.py` script are
updated appropriately. The documentation and GitHub Actions workflow
are also updated to support this change.
@mcdonnnj mcdonnnj added the improvement This issue or pull request will add or improve functionality, maintainability, or ease of use label Sep 20, 2022
@mcdonnnj mcdonnnj self-assigned this Sep 20, 2022
Copy link
Member

@dav3r dav3r left a comment

Choose a reason for hiding this comment

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

Love it! 🚀

.github/workflows/build.yml Outdated Show resolved Hide resolved
mcdonnnj and others added 2 commits September 20, 2022 12:22
@mcdonnnj mcdonnnj force-pushed the improvement/switch_to_alpine_linux branch from e5d825d to c2f34b3 Compare September 20, 2022 16:25
@mcdonnnj mcdonnnj merged commit 40aede4 into improvement/project_updates Sep 20, 2022
@mcdonnnj mcdonnnj deleted the improvement/switch_to_alpine_linux branch September 20, 2022 16:37
@mcdonnnj mcdonnnj mentioned this pull request Sep 20, 2022
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants