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

Proxy settings are ignored for agent v6 #441

Closed
atayts opened this issue Jun 25, 2018 · 3 comments
Closed

Proxy settings are ignored for agent v6 #441

atayts opened this issue Jun 25, 2018 · 3 comments

Comments

@atayts
Copy link

atayts commented Jun 25, 2018

Proxy settings are completely ignored for the new, v6, agent. They are missing in $_agent_config. Also beware that in the new agent HTTP and HTTPS proxies are independent values, while in the datadog_agent class they are still represented as one.

@ipleten
Copy link

ipleten commented Jul 3, 2018

collect_ec2_tags is missed too.
Update: A lot of settings are missing.

@truthbk
Copy link
Member

truthbk commented Jul 10, 2018

Hi guys,

Please note that you can pass any additional options needed in your datadog.yaml via the agent6_extra_options hash parameter. It was meant to be flexible enough to accomodate for any currently unsupported agent6 options. Because the puppet module is meant to be backward compatible it wasn't straight forward providing a consistent interface for both agents without making breaking changes, hence the presence of parameters that only apply directly to agent5.

We can probably fix collect_ec2_tags, but for proxy I just recommend you pass:

$agent6_extra_options = {
    'collect_ec2_tags' => true,
    'proxy' => {
          'http'    => 'http(s)://user:password@foo:3128',
          'https'  => 'http(s)://user:password@bar:3128',
     }

That should then be enforced.

@truthbk
Copy link
Member

truthbk commented Jul 10, 2018

Adding some deprecation notices here: #446 + adding support for collect_ec2_tags.

@truthbk truthbk closed this as completed Jul 10, 2018
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

3 participants