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.

Sign up for GitHub

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

Jump to bottom

fix examples/consul so it works #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix examples/consul so it works #57

wants to merge 1 commit into from

Conversation

tekumara
Copy link

@tekumara tekumara commented Oct 6, 2018

  • use latest autopilotpattern/consul in dev mode (ie: single consul)
  • don't use the default bridge network so the alias consul is created
  • expose consul ui to the host (so it can be accessed as per the readme)

@tekumara
fix examples/consul so it works
690ee4d 
* use latest autopilotpattern/consul in dev mode (ie: single consul)
* don't use the default bridge network so the alias consul is created
* expose consul ui to the host (so it can be accessed as per the readme)
@tekumara
Copy link
Author

@tgross would you mind taking a look at this PR?

@tgross
Copy link
Contributor

tgross commented Aug 8, 2019

@tekumara I'm not really involved at all in this project anymore and don't have a commit bit. This org is mostly a Joyent project, so you might want to reach out to anyone who you might know there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tekumara @tgross