Skip to content

Conversation

@gaurav-nelson
Copy link

@gaurav-nelson gaurav-nelson commented Apr 15, 2024

Some optimizations for the image. Down from 1.68 GB to just 394 MB. Around 75% smaller! πŸ€¦πŸΌβ€β™‚οΈ Compared with the wrong image. Down from 412 MB to 394 MB.

I tested locally and worked fine for me on Intel Mac. Need more testing before we can merge.

@gaurav-nelson
Copy link
Author

@aireilly PTAL

@wgordon17
Copy link
Contributor

@gaurav-nelson I'm honestly assuming https://github.com/openshift-cs/docs-builder/blob/main/asciibinder/asciibinder.Dockerfile would benefit from a similar exercise. I'm not seeing anything obvious here that would impact the reliability...and having a smaller image is always a win! Assuming @aireilly is onboard (since he did the last edits), then I'm βž•1️⃣ on a merge

@gaurav-nelson
Copy link
Author

Thank you @wgordon17 I'll check that one as well.

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.

2 participants