-
Notifications
You must be signed in to change notification settings - Fork 159
Dockerfile for new minimal base image based on alpine #317
Dockerfile for new minimal base image based on alpine #317
Conversation
@@ -0,0 +1,4 @@ | |||
FROM alpine:latest | |||
|
|||
RUN apk add --update git openssh-client |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this run apk update
too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the --update flag fetches the current package index before adding the package.
For reference: https://github.com/gliderlabs/docker-alpine/blob/master/docs/usage.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ImJasonH, pivotal-nader-ziada The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Related to #179
Proposed Changes
.ko.yaml
file should be updated to point to itRelease Note