Skip to content

Conversation

@ivlis
Copy link

@ivlis ivlis commented Sep 16, 2016

It is smaller and much faster to build!

Copy link
Member

@stempler stempler left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution Ivan. A version of the image that is based on Alpine will indeed be much smaller. Though I would still like to keep the Ubuntu version around as well, as other images like dockup-mongo may depend on it.
So my suggestion would be to integrate your changes into a new alpine branch, with a corresponding Docker image tag on the hub. As you also installed bash, changes are minimal, so maintaining both should not be a hassle.
What do you think about that?


# RUN echo '@testing http://nl.alpinelinux.org/alpine/edge/testing' >> /etc/apk/repositories && \
# apk add --update --no-cache curl awscli@testing bash grep
# RUN apt-get update && apt-get install -y python-pip curl && pip install awscli
Copy link
Member

Choose a reason for hiding this comment

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

What about these commented lines? I guess they can be removed?

Copy link
Author

Choose a reason for hiding this comment

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

Yeah, these lines are here by mistake

@ivlis
Copy link
Author

ivlis commented Sep 17, 2016

Having a separate branch for alpine is a nice idea. If you create an alpine branch here and link it to the docker hub it would be great.

@ivlis
Copy link
Author

ivlis commented Sep 17, 2016

Another idea, since migration to alpine does not require any modification for the script itself, perhaps it is better to create a separate Dockerfile.alpine file and link it to a separate tag on the Docker Hub. It is easier then maintain two separate branches. I can create a PR for that.

@stempler
Copy link
Member

I was unsure if that's supported by Docker Hub automated builds, but this post suggests it does, so I would like to give your idea a try. Would be great if you create the PR.

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