-
Notifications
You must be signed in to change notification settings - Fork 230
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
(PDB-4764) Agent SSL certificates are used for communication with PostgreSQL #322
Conversation
puppetdb_conn_validator is a typeBreaking changes to this file MAY impact these 1 modules (near match):puppetdb::database::postgresql is a classthat may have no external impact to Forge modules. puppetdb is a classBreaking changes to this file WILL impact these 5 modules (exact match):Breaking changes to this file MAY impact these 3 modules (near match):puppetdb::params is a classthat may have no external impact to Forge modules. This module is declared in 33 of 576 indexed public
|
4758df1
to
cb44ace
Compare
fd2cccb
to
310fe29
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To document the settings I had to add for client authentication I wanted to add these here
pg_hba.conf
# Rule Name: Allow certificate mapped connections to pe-puppetdb as pe-puppetdb (ipv4)
# Description: none
# Order: 0
hostssl puppetdb puppetdb 0.0.0.0/0 cert map=puppetdb-puppetdb-map clientcert=1
# Rule Name: Allow certificate mapped connections to pe-puppetdb as pe-puppetdb (ipv6)
# Description: none
# Order: 1
hostssl puppetdb puppetdb ::/0 cert map=puppetdb-puppetdb-map clientcert=1
This is a file I forgot about, but pg_ident.conf defines the certificate name to database username mappings
# This file is managed by Puppet. DO NOT EDIT.
puppetdb-puppetdb-map duller-artery.delivery.puppetlabs.net puppetdb
A lot of my configuration to get it working was done manually, so in case I missed anything I've extended the lifetime of duller-artery.delivery.puppetlabs.net in case you want to dig around in the settings.
90ec746
to
4e2210d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, I think it could use a couple of basic tests for the important settings, like pg_ident/pg_hba rules, the subname config setting, and the pk8 and then it should be ready to go.
98c8214
to
b46e611
Compare
b46e611
to
bb043f2
Compare
Updated the manifests for SSL configuration of PostgreSQL database configuration file.