Riemann sink #687
Riemann sink #687
Conversation
|
We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm. |
1 similar comment
|
We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm. |
Support a Riemann sink (derived from the influxdb sink). Map Heapster events into Riemann: * Map Heapster event structures to Riemann events. * Coerce Int64 metrics to Int for transport to Riemann. * Support user-provided Riemann tags and event state, with defaults. Includes basic documentation and sample Riemann config.
Signed-off-by: Vishnu kannan <vishnuk@google.com>
Some repos no longer exist. Update kube dependency to current v1.1 branch. Updated coreos fleet support to match godeps. Signed-off-by: Vishnu kannan <vishnuk@google.com>
vishh
force-pushed
the
jfoy-riemann_sink
branch
from
f41c625
to
063ce38
Compare
Signed-off-by: Vishnu kannan <vishnuk@google.com>
|
@jfoy: I'd appreciate a PR in the future to fix the docs section I have added for Riemann sink. |
|
Will do! Thanks for updating this. |
|
I tested this sink myself. This should be good to go. |
|
LGTM, just 2 nits. I've taken a look only into the last commit. Also please bear in mind #692 |
Signed-off-by: Vishnu kannan <vishnuk@google.com>
vishh
force-pushed
the
jfoy-riemann_sink
branch
from
261393e
to
22a8ea0
Compare
|
@piosz: Thanks for the review. Added comments. |
Credits to @jfoy!
I fixed godeps while adding the required dependencies for riemann!
@jfoy can you test this PR once?
ping @mwielgus @piosz for review.