Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

Fix bit rot in mesos-go API and Consul dependencies #113

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

Conversation

gfv
Copy link

@gfv gfv commented Apr 5, 2017

mesos-go has moved to a different API, but were kind enough to leave the old one in place, so I've monkey-patched this issue.
Consul API requires Go >=1.7 to build, but Alpine 3.1 had Go 1.3.3; I've updated Dockerfile to point towards Alpine 3.5 with Go 1.7.3.
A certain kernel bug in overlayfs does not allow removing a file created in another layer (cf. moby/moby#27214). Added a workaround for that.

@Theaxiom Theaxiom requested a review from ChrisAubuchon April 18, 2017 14:55
@ChrisAubuchon
Copy link
Contributor

This resolves the same issue that #112 does but does it in a different way. Either method works.

@thomasvincent thomasvincent self-requested a review June 7, 2017 09:24
Copy link
Contributor

@thomasvincent thomasvincent left a comment

Choose a reason for hiding this comment

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

LGTM!

@ChrisAubuchon
Copy link
Contributor

glide.yaml will have to be updated to fix the build issues.

@Theaxiom
Copy link
Contributor

Theaxiom commented Jun 8, 2017

#112 was merged in which accomplishes the same thing, we may be able to just close this out.

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

Successfully merging this pull request may close these issues.

4 participants