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

Add Docker release files for mc #2182

Merged
merged 1 commit into from
Jun 15, 2017

Conversation

nitisht
Copy link
Contributor

@nitisht nitisht commented Jun 14, 2017

These Dockerfile(s) will be used for mc release images for various platforms.

Statically built binary is downloaded from dl.minio.io website and used in the docker image

@codecov-io
Copy link

codecov-io commented Jun 14, 2017

Codecov Report

Merging #2182 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #2182   +/-   ##
======================================
  Coverage    9.15%   9.15%           
======================================
  Files          93      93           
  Lines        6906    6906           
======================================
  Hits          632     632           
  Misses       6140    6140           
  Partials      134     134

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d44e8c...1ad9516. Read the comment docs.

@nitisht
Copy link
Contributor Author

nitisht commented Jun 14, 2017

Built locally using https://blog.hypriot.com/post/setup-simple-ci-pipeline-for-arm-images/ works fine.

@@ -0,0 +1,12 @@
FROM alpine:3.5

WORKDIR /go/src/app
Copy link
Member

Choose a reason for hiding this comment

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

You don't need this.. @nitisht also you can follow Dockerfile.release

     apk add --no-cache ca-certificates && \
     apk add --no-cache --virtual .build-deps curl && \

We need ca-certificates as well.

     echo 'hosts: files mdns4_minimal [NOTFOUND=return] dns mdns4' >> /etc/nsswitch.conf && \

curl https://dl.minio.io/client/mc/release/linux-amd64/mc > /usr/bin/mc && \
chmod +x /usr/bin/mc

ENTRYPOINT ["mc"]
Copy link
Member

Choose a reason for hiding this comment

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

Odd newline here.


WORKDIR /go/src/app

COPY . /go/src/app
Copy link
Member

Choose a reason for hiding this comment

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

Same as above.

FROM resin/armhf-alpine:3.5

WORKDIR /go/src/app

Copy link
Member

Choose a reason for hiding this comment

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

Same as above.

@nitisht nitisht force-pushed the dockerfile-update branch from 2ac29e2 to 3faddf3 Compare June 14, 2017 22:28
@nitisht
Copy link
Contributor Author

nitisht commented Jun 14, 2017

Updated as per your comments @harshavardhana

Also as discussed, did not use

echo 'hosts: files mdns4_minimal [NOTFOUND=return] dns mdns4' >> /etc/nsswitch.conf && \

- New Dockerfile will be used for mc release images.
- Statically built binary is downloaded from dl.minio.io website and used in the docker image.
@nitisht nitisht force-pushed the dockerfile-update branch from 3faddf3 to 1ad9516 Compare June 14, 2017 23:27
@harshavardhana harshavardhana merged commit ea05fcb into minio:master Jun 15, 2017
@nitisht nitisht deleted the dockerfile-update branch April 25, 2020 06:58
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.

4 participants