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

Bosun Support (aka OpenTSDB api) #131

Open
rbucker opened this issue Aug 10, 2015 · 4 comments
Open

Bosun Support (aka OpenTSDB api) #131

rbucker opened this issue Aug 10, 2015 · 4 comments

Comments

@rbucker
Copy link

rbucker commented Aug 10, 2015

The current OpenTSDB seems to implement the socket version of messaging. It's very similar to the Graphite version. Looking at the Bosun documentation (said to support OpenTSDB 2.x) it appears to be limited to the http POST verb. I'm about to fork go-metrics in order to implement the feature but I wanted to get your opinion first.

Is Bosun supported? Is someone working on it? Would you accept a pull request?

@mihasya
Copy link
Collaborator

mihasya commented Aug 10, 2015

@rbucker you come to this repo at an exciting time! Additional clients will not be accepted to the core go-metrics repository, as it creates a support nightmare as external APIs change and the maintainers have to try and make decisions about clients they don't use which potentially break things for people in communities to which the maintainers don't belong. As such, the decision was made to encourage "output" clients to be implemented as standalone repos. You can see the early examples here and here and the related issue at #121.

I'll gladly accept a pull request that updates the README to include bosun as a supported client with a link to the corresponding repository.

@rbucker
Copy link
Author

rbucker commented Aug 10, 2015

It's a crappy answer but I guess I'll have to live with it. I've created the wrapper for the project. I should have a working model tonight.

@rbucker rbucker closed this as completed Aug 10, 2015
@mihasya
Copy link
Collaborator

mihasya commented Aug 10, 2015

Sorry you find the solution "crappy." I look forward to adding a link to your repo when it is ready.

@rbucker
Copy link
Author

rbucker commented Aug 11, 2015

Code Complete here

please update whatever doc needs it.... and close.

@rbucker rbucker reopened this Aug 11, 2015
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

No branches or pull requests

2 participants