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

Suggestions regarding docker image #1016

Open
DannyBen opened this issue Apr 26, 2021 · 2 comments
Open

Suggestions regarding docker image #1016

DannyBen opened this issue Apr 26, 2021 · 2 comments

Comments

@DannyBen
Copy link

I followed the clear instructions of manually using the docker image, and after running the container, it took a long time to run. After running it in debug mode, I realized that it pulls additional >5GB containers.

To me (and I am sure to some others), it looked like it just "hangs" since the pulling of the additional images takes way too long, and I know my project usually runs in a few seconds on the cloud service.

So, based on this experience, I would like to suggest to update the docker pull instructions to instruct users to pre-download the additional images and not only the base image:

$ docker pull codeclimate/codeclimate
$ docker pull codeclimate/codeclimate-structure
$ docker pull codeclimate/codeclimate-duplication

In addition, I can say that these docker images are the largest I have ever seen, by a huge factor. I am guessing the reason is that they contain all the tools for all the languages. It would be much nicer to have smaller (<1GB) images, on a per-language basis, and even better - to have these images work without the need to use docker-in-docker.

I know the chances of this happening are slim to none, but I thought I should mention it, in case someone else in the code climate team is thinking along these lines.

@Shocktrooper
Copy link

I am with you on the large images. It would be really nice to be able to configure codeclimate to point at alternative engine locations because then they also don't need to be cached on the host

@thelooter
Copy link

Any update on this?

@noelia-lencina noelia-lencina removed their assignment Feb 28, 2024
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

No branches or pull requests

4 participants