-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 <[email protected]>
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 <[email protected]>
f41c625
to
063ce38
Compare
Signed-off-by: Vishnu kannan <[email protected]>
@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 <[email protected]>
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.