Skip to content

Fresh install need to restart after pg_hba.conf #236

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

Closed
Poil opened this issue Aug 10, 2018 · 3 comments
Closed

Fresh install need to restart after pg_hba.conf #236

Poil opened this issue Aug 10, 2018 · 3 comments

Comments

@Poil
Copy link

Poil commented Aug 10, 2018

Hi,

I've deployed a new postgresql server, it seems that the postgresql-pg_hba resource is not notifying the postgresql service (PG 10 on CentOS 7)

Best regards

@myii
Copy link
Contributor

myii commented Aug 16, 2018

This was addressed by @RobRuana (with a proposed resolution): #216 (comment).

@noelmcloughlin
Copy link
Contributor

impacts Ubuntu too, see #216 (comment)

I've found that simply reloading the postgres service is not enough to pick up ACL changes in pg_hba.conf. A full restart is required.
This is on ubuntu 18.04 with postgres 10.

@danny-smit
Copy link

I just ran into this with Debian as well. It seems that triggering a reload manually does pick up the changes, but during the run of this formula it does not.

Could it be that it is actually a race condition? As the formula executes the following steps:

  1. update postgresql.conf
  2. restart postgresql service
  3. update pg_hba.conf
  4. reload the postgresql service

Perhaps the reload (bullet 4) is ignored because the restart (of bullet 2) is still in progress?

@aboe76 aboe76 closed this as completed in cce4cbf Jan 19, 2019
aboe76 added a commit that referenced this issue Jan 19, 2019
Fix #236: Restart postgres service after ACL changes in pg_hba.conf
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

No branches or pull requests

4 participants