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 Marathon watcher #129

Merged
merged 1 commit into from
Dec 17, 2015
Merged

Add Marathon watcher #129

merged 1 commit into from
Dec 17, 2015

Conversation

ktf
Copy link
Contributor

@ktf ktf commented Dec 17, 2015

Fetches metrics from /metrics endpoint in
Marathon and pushes them to
riemann. This is derived from the elasticsearch watcher.

Fetches metrics from /metrics endpoint in
[Marathon](https://mesosphere.github.io/marathon/) and pushes them to
riemann. This is derived from the elasticsearch watcher.
@aphyr
Copy link
Collaborator

aphyr commented Dec 17, 2015

Seems legit! Thanks ktf. :)

aphyr added a commit that referenced this pull request Dec 17, 2015
@aphyr aphyr merged commit 0ac0305 into riemann:master Dec 17, 2015
@ktf
Copy link
Contributor Author

ktf commented Dec 17, 2015

Thanks. I have a problem with riemann-dash though. It seems to choke whenever I've more that a few dozens of metrics being displayed. Is that a known issue?

@aphyr
Copy link
Collaborator

aphyr commented Dec 17, 2015

I've pushed a few thousand events/sec through the dash, but, as you probably already know, computers. ;-)

@ktf
Copy link
Contributor Author

ktf commented Dec 17, 2015

Let me rephrase is. Is there any known slugginesh issue (i.e. it freezes) whenever you use the log / list views with more than a few dozens entries?

@aphyr
Copy link
Collaborator

aphyr commented Dec 17, 2015

I don't recall anyone else reporting this problem, but it obviously exists, since it's happening to you. Fire up a profiler?

@ktf
Copy link
Contributor Author

ktf commented Dec 17, 2015

Okok, I'll try to investigate more. BTW, there is also #128 which is for a related (but actually different) service.

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.

2 participants