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

Fix Postgres connect, incorrect port value #5618

Merged
merged 1 commit into from Mar 12, 2019

Conversation

@Jaxmetalmax
Copy link
Contributor

Jaxmetalmax commented Mar 12, 2019

Summary

Fix postgres connection, port value was incorrect, it was taking host value instead of port.

Component Name

collectors/python.d.plugin/postgres

Additional Information
@Jaxmetalmax Jaxmetalmax requested a review from ilyam8 as a code owner Mar 12, 2019
@CLAassistant

This comment has been minimized.

Copy link

CLAassistant commented Mar 12, 2019

CLA assistant check
All committers have signed the CLA.

@ilyam8
ilyam8 approved these changes Mar 12, 2019
@ilyam8

This comment has been minimized.

Copy link
Member

ilyam8 commented Mar 12, 2019

i tested using unix socket 😅

@ilyam8 ilyam8 merged commit 263ec64 into netdata:master Mar 12, 2019
11 of 12 checks passed
11 of 12 checks passed
Codacy/PR Quality Review Hang in there, Codacy is reviewing your Pull request.
Details
Header rules - netdata No header rules processed
Details
Pages changed - netdata 2 new files uploaded
Details
Redirect rules - netdata No redirect rules processed
Details
LGTM analysis: C/C++ No code changes detected
Details
LGTM analysis: JavaScript No code changes detected
Details
LGTM analysis: Python No new or fixed alerts
Details
Mixed content - netdata No mixed content detected
Details
Travis CI - Pull Request Build Passed
Details
WIP Ready for review
Details
license/cla Contributor License Agreement is signed.
Details
netlify/netdata/deploy-preview Deploy preview ready!
Details
@ilyam8

This comment has been minimized.

Copy link
Member

ilyam8 commented Mar 12, 2019

thx @Jaxmetalmax

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.