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

Dockerfile: switch to alpine base image #12

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

Conversation

thaJeztah
Copy link

follow-up to https://github.com/shin-/dockerpy-creds/pull/11, but the pass package is not yet in the "stable" Alpine package index, so perhaps you didn't want that.

The alpine image makes the image quite a bit smaller;

REPOSITORY            TAG       IMAGE ID        CREATED             SIZE
docker/pycreds-test   alpine    2451e1c13a9e    3 seconds ago       134MB
docker/pycreds-test   latest    70526611619f    10 minutes ago      951MB

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
To optimize for caching; only re-install dependencies
if the list of dependencies changed.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
So that it doesn't run as subprocess of `/bin/sh`

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Author

@shin-

Because it's a bit smaller;

    REPOSITORY            TAG       IMAGE ID        CREATED             SIZE
    docker/pycreds-test   alpine    2451e1c13a9e    3 seconds ago       134MB
    docker/pycreds-test   latest    70526611619f    10 minutes ago      951MB

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
curl

# TODO remove once pass is in the main repository
RUN apk add --no-cache --repository http://dl-cdn.alpinelinux.org/alpine/edge/testing pass
Copy link
Author

Choose a reason for hiding this comment

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

I should probably note here that edge packages are not guaranteed to work with stable, so while this one currently does, it's possible that this may break on a future update of stable (or the edge package)

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.

None yet

1 participant