-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Update to fix pg_rewind can't find postgresql.conf file in data directory via the librepmgr.sh script #53226
base: main
Are you sure you want to change the base?
Conversation
This fixes the issues of the not found postgresql.conf file in the pg data directory. The actual file name there is postgresql.auto.conf. Signed-off-by: xtianus79 <[email protected]>
hi @dgomezleon how are you. Did you get a chance to look at the change request? |
Hi @xtianus79, Have you tested this change? I gave it a try and obtained this:
So it seems it is necessary the full path. Also, the change should be applied to all the supported branches. |
@dgomezleon can you check a proper change. I was not able to test the change but I am assuming you're correct. The auto file is the correct file that is there. |
Applying some changes to this PR we have passed that error. However, we have found other errors, so we have created a task to properly review the issue. |
Thanks @dgomezleon that's great. Do you know how long that might be to patch all errors? As of now I am comfortable with where I'm at and I can plan out a future update down the road. A range of time is sufficient. |
Hi @xtianus79 , After looking over our priorities and due to other issues/initiatives we are working on, the task is still in our backlog list, so I can't give you an ETA. Please feel free to update/test this PR if you are interested. We will update the case as soon as we have any news. |
excuse me, Is there any progress? |
This fixes the issues of the not found postgresql.conf file in the pg data directory. The actual file name there is postgresql.auto.conf.
Description of the change
The change uses the actual file name in the pg data directory. By default it would be postgresql.conf but the actual file name is postgresql.auto.conf
Benefits
prevent the error seens as pg_rewind can't find pg conf file.
Issue seems to be from this code in the repmgr script
librepmgr.sh
Other posts related
bitnami/charts#20998
#52213
bitnami/charts#8933
And probably many more issues where a user is using pg_rewind
Possible drawbacks
It will continue not working so nothing from my perspective.
Applicable issues
The issue is that upon failing a node and it trying to come back as primary it will not know the repmgr_slot_nodeName and will error continuously. The returning node is never put on the right timeline and does not enter back into the HA setup correctly.
Additional information
Dos for pg_rewind