-
Notifications
You must be signed in to change notification settings - Fork 132
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
Feat(postgresql): Add support for postgresql backend (including tests) #211
base: master
Are you sure you want to change the base?
Feat(postgresql): Add support for postgresql backend (including tests) #211
Conversation
212161f
to
ec65eb5
Compare
You could have kept my commit. |
This comment was marked as off-topic.
This comment was marked as off-topic.
bda11f8
to
d0181ef
Compare
Hi @commonism, Done |
I thought this was MIT-0, but well, can you point me to somewhere to read up on this?
He's settled things already. |
This comment was marked as off-topic.
This comment was marked as off-topic.
MIT-0 is "zero attribution", the license here is MIT. |
@commonism I've created a discussion in a throwaway repo to continue the license talk - there's no need to spam this PR with irrelevant discussion (open source etiquette and all 😉) |
d0181ef
to
f6c7093
Compare
Hi,
I have a use case where I would like to use postgressql as a backend for pdns.
This PR finishes the work started by #104
I've made the following changes:
Minor changes to add pgsql backend #104
Add tests to the postgresSQL ansible code (I've added tests similar to what is tested with MySQL backend)
Added "throttle:1" to both MySQL and PostgreSQL user creation module. Occasionally the tasks failed and I'm assuming it's because of race-conditions. (For both MySQL and PostgreSQL, a user with the same name is created multiple times (foreach distro in the tests) and assigned permissions/privileges)
Added support for PostgreSQL's Host Based Authentication. This can be used when PowerDNS Authoritative server is installed on the same host as the PostgreSQL server. In that case, Ansible does not need a privileged PostgreSQL user to create a user for PowerDNS. It will simply use the unix socket and use
become "postgresql"
. This is unfortunately not covered by any tests.