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

Use https:// in datadog-reports.yaml #503

Merged
merged 1 commit into from
Mar 15, 2019
Merged

Use https:// in datadog-reports.yaml #503

merged 1 commit into from
Mar 15, 2019

Conversation

cabrinha
Copy link
Contributor

Without specifying https://, I get this error:

2019-03-07T05:37:16.801Z ERROR [qtp418436983-72] [puppetserver] Puppet Report processor failed: Connection refused - Failed to open TCP connection to :80 (Connection refused - connect(2) for nil port 80)

In #488 one user claimed to have success by changing this URL to use HTTPS: krux#1

I've also tested this myself and found it resolves the error shown above.

The puppet versions I'm running are:

# Puppet agent
puppet --version
6.3.0

# Puppetserver
puppetserver --version
puppetserver version: 6.2.1

# JRuby
puppetserver ruby --version
jruby 9.2.0.0 (2.5.0) 2018-05-24 81156a8 OpenJDK 64-Bit Server VM 25.191-b12 on 1.8.0_191-8u191-b12-2ubuntu0.16.04.1-b12 +jit [linux-x86_64]

@truthbk please have a look!

Copy link
Member

@truthbk truthbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a regression and is consistent with the client docs here: https://github.com/DataDog/dogapi-rb

Thank you so much! 🙇

@truthbk truthbk merged commit bd65406 into DataDog:master Mar 15, 2019
@truthbk truthbk added this to the 2.5.0 milestone Mar 25, 2019
@cabrinha cabrinha deleted the reports branch April 17, 2019 20:52
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Apr 6, 2020
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

Successfully merging this pull request may close these issues.

2 participants