Conversation
|
Hi @pmalek-sumo, thanks for the PR. We have a backlog of new plugins but @ssoroka or I will try to review this soon. |
|
@reimda Awesome. Thanks! |
|
@pmalek-sumo If you'd like to submit this as external plugin in the meantime that would be great. You and others would be able to start using the plugin with Telegraf sooner without waiting for review. Here's more information on external plugins. |
|
Hey @pmalek-sumo. If you're an employee of Sumo Logic, do you think you and any other employees involved in building things for Influx can sign the Corporate CLA? |
0db3fb5 to
de60be8
Compare
de60be8 to
76aa252
Compare
| URL string `toml:"url"` | ||
| Timeout internal.Duration `toml:"timeout"` | ||
| Method string `toml:"method"` | ||
| MaxRequstBodySize int `toml:"max_request_body_size"` |
There was a problem hiding this comment.
There is a config.Size type that might be nice for this.
There was a problem hiding this comment.
Oh, I didn't see this. Nice! Changed
|
Any idea what's going on with ci macdeps test? it seems like it's running against SumoLogic CI instead of InfluxData? |
|
Yeah - all of the CircleCI tests were run against |
228c99b to
fd5a48c
Compare
|
CircleCI project was disconnected from Sumo Logic's repository. I can see the jobs are running now under influx's org. |
|
@sjwang90 we've just sent signed CCLA to community@influxdata.com |
This PR adds an output plugin to send data to Sumo Logic
Required for all PRs: