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

Migrate to alpine #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Migrate to alpine #1

wants to merge 3 commits into from

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