Skip to content
This repository was archived by the owner on Apr 24, 2026. It is now read-only.

Add Dockerfile - #163

Merged
aravindavk merged 2 commits into
gluster:masterfrom
smuth4:dockerfile
Jul 11, 2019
Merged

aravindavk merged 2 commits into
gluster:masterfrom
smuth4:dockerfile

Conversation

@smuth4

@smuth4 smuth4 commented Jul 9, 2019

Copy link
Copy Markdown
Contributor

Uses Debian in order to align build and run images on the same supported platform. Note that GLUSTER_VERSION must be set to a number rather than LATEST due to http://download.gluster.org/pub/gluster/glusterfs/LATEST/rsa.pub being a 404 as well as differences in directory structure.
I also noticed that despite 567cf6d removing gometalinter, the scripts/ directory was never touched, so it's still required to be installed for the make commands to work.

@aravindavk

Copy link
Copy Markdown
Member

Thanks for pointing to gometalinter issue in scripts directory. I will send PR to address that issue.

@aravindavk

Copy link
Copy Markdown
Member

Please move the Dockerfile to extras directory. May be with name debian.Dockerfile

@aravindavk
aravindavk merged commit 9eacc41 into gluster:master Jul 11, 2019
@adampl

adampl commented Nov 20, 2019 •

Copy link
Copy Markdown

Hi, how to run this container to make it work with a Gluster cluster?

I asked in #73 but now I see you created your own image, but the issue remains unresolved.

By the way, this Dockerfile doesn't build, I'm getting this:

E: Failed to fetch https://download.gluster.org/pub/gluster/glusterfs/6/LATEST/Debian/9/amd64/apt/pool/main/g/glusterfs/glusterfs-server_6.6-1_amd64.deb
GnuTLS recv error (-9): A TLS packet with unexpected length was received.

@smuth4

smuth4 commented Dec 18, 2019 •

Copy link
Copy Markdown
Contributor Author

I am unable to reproduce this on the latest code, and it's odd that it would allow adding/updating the repo but bail on the actual package repo. Is it consistent? If not, it might be related to https://bugs.launchpad.net/ubuntu/+source/gnutls26/+bug/1111882, if so, you may want to double check your network's setting,

Edit: also, you can ignore your cached images by adding the --no-cache --pull flags temporarily.

@adampl

adampl commented Dec 18, 2019

Copy link
Copy Markdown

Maybe I was building the image in a wrong context. I will check it once it's merged. Thanks :)

@smuth4

smuth4 commented Dec 19, 2019

Copy link
Copy Markdown
Contributor Author

That PR only adds documentation, it doesn't change anything about the build itself.

@adampl

adampl commented Dec 19, 2019

Copy link
Copy Markdown

Ah, sorry, you're right xD

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants