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

Added Telegraf component #12199

Open
wants to merge 6 commits into
base: master
from
Open

Added Telegraf component #12199

wants to merge 6 commits into from

Conversation

@olaaun
Copy link
Member

olaaun commented Feb 14, 2020

Based on the config file defined in #12164

@olaaun olaaun added the DO NOT MERGE label Feb 14, 2020
@olaaun olaaun requested a review from gjoranv Feb 14, 2020
metrics-proxy/pom.xml Outdated Show resolved Hide resolved
tls_cert = "${VESPA_CERTIFICATE_PATH}"
tls_key = "${VESPA_KEY_PATH}"
insecure_skip_verify = true
#end

This comment has been minimized.

Copy link
@gjoranv

gjoranv Feb 14, 2020

Contributor

Please add trailing newline.

public class Telegraf extends AbstractComponent {

private static final String TELEGRAF_CONFIG_PATH = "/etc/telegraf/telegraf.conf";
private static final String TELEGRAF_CONFIG_TEMPLATE_PATH = "src/main/resources/templates/cloudwatch_plugin.vm";

This comment has been minimized.

Copy link
@gjoranv

gjoranv Feb 14, 2020

Contributor

We probably need to consider these file paths, especially where to read the template path from.

@olaaun olaaun force-pushed the olaa/telegraf-component branch from 39039b1 to 83c0f76 Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.