Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

(SUP-2779) Fix the bug - telegraf postgres metrics unable to obtain data #200

Merged
merged 1 commit into from
Nov 3, 2021

Conversation

henrywangpuppet
Copy link
Contributor

Before this commit, the connection request from telegraf to PostgreSQL is not able to be authenticated due to SSL connection issue.
With this commit, now the authentication is successful and the Telegraf Postgre Metrics can get data to display.

Before this commit, the connection request from telegraf to PostgreSQL is not able to be authenticated due to SSL connection issue.
With this commit, now the authentication is successful and the Telegraf Postgre Metrics can get data to display.
@henrywangpuppet henrywangpuppet requested a review from a team as a code owner November 3, 2021 03:28
@puppet-community-rangefinder
Copy link

puppet_metrics_dashboard::profile::master::postgres is a type

that may have no external impact to Forge modules.

This module is declared in 0 of 578 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

@henrywangpuppet henrywangpuppet added the bug Something isn't working label Nov 3, 2021
Copy link
Contributor

@MartyEwings MartyEwings left a comment

Choose a reason for hiding this comment

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

Looks good to me

@MartyEwings MartyEwings merged commit e1a7576 into puppetlabs:main Nov 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants