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 Dockerfile #163

Merged
merged 2 commits into from
Jul 11, 2019
Merged

Add Dockerfile #163

merged 2 commits into from
Jul 11, 2019

Conversation

smuth4
Copy link
Contributor

@smuth4 smuth4 commented Jul 9, 2019

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
Member

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

@aravindavk
Copy link
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
Copy link

adampl commented Nov 20, 2019

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
Copy link
Contributor Author

smuth4 commented Dec 18, 2019

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
Copy link

adampl commented Dec 18, 2019

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

@smuth4
Copy link
Contributor Author

smuth4 commented Dec 19, 2019

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

@adampl
Copy link

adampl commented Dec 19, 2019

Ah, sorry, you're right xD

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.

3 participants