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

Consul env #10

Merged
merged 2 commits into from
Apr 14, 2016
Merged

Consul env #10

merged 2 commits into from
Apr 14, 2016

Conversation

jasondewitt
Copy link
Contributor

Running into problems getting this image to work on Triton because containerbuddy.json has consul statically set rather than pulling from an environment variable.

@tianon
Copy link

tianon commented Apr 14, 2016

On Triton, this is already handled via --link on the mysql containers (so that they can get to consul via this direct hostname).

@misterbisson
Copy link
Contributor

@tianon we've switched to using DNS discovery for Consul in our more recent blueprints. https://github.com/autopilotpattern/jenkins is a good example. The setup.sh there writes out the Consul DNS name to the _env file.

One significant advantage there is using Consul in an HA raft, where every instance of consul shares the same DNS name.

Ideally, @Roxet's change would fully implement that pattern here too and eliminate any use of --link in the Compose file, but this is a start.

@tianon
Copy link

tianon commented Apr 14, 2016

@misterbisson ok, fair enough! 👍

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