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

Marathon Watcher #87

Closed
wants to merge 5 commits into from
Closed

Marathon Watcher #87

wants to merge 5 commits into from

Conversation

deverton
Copy link

This adds a service watcher that uses data from the Marathon REST API V2 to locate services. It's based on a combination of the Docker and EC2 watchers.

Dan Everton added 4 commits August 11, 2014 11:26
Pull service information from Marathon's task API to allow using it for
service discovery.
@igor47
Copy link
Collaborator

igor47 commented Aug 18, 2014

hey thanks for the PR. i'm going have a chance to look through this and other PRs when i get back from vacation in september.

Now checks for errors from the Marathon client and logs them.
@mrud
Copy link

mrud commented Oct 29, 2014

Any update on this?

@acmcelwee
Copy link

Here's another series of commits that I needed to get this working w/ marathon 0.7.3 https://github.com/acmcelwee/synapse/commits/marathon. Unfortunately, those are also blocked until 0.3.0 of the marathon_client is published to rubygems.org, issue #14 over there. For now, I've hacked in the updated marathon_client in my env, and that, plus the commits referenced above got me all set w/ this new watcher.

@tdooner tdooner mentioned this pull request Nov 27, 2014
@tdooner
Copy link
Contributor

tdooner commented Nov 27, 2014

+1 for this functionality. I accidentally implemented the same thing earlier today in #101. Anyway, merging one of these would be awesome.

@jolynch jolynch closed this Nov 3, 2015
@jolynch
Copy link
Collaborator

jolynch commented Nov 3, 2015

Merged #101 which should cover this functionality

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.

6 participants