vitabaks / postgresql_cluster

PostgreSQL High-Availability Cluster (based on "Patroni" and DCS "etcd" or "consul"). Automating with Ansible.
MIT License
1.29k stars 353 forks source link

Patroni dependency missing #626

Closed rcknr closed 1 month ago

rcknr commented 1 month ago

I've used the playbook to deploy a cluster on Ubuntu 22.04 several times now and every time patronictl was missing a dependency - cdiff. Upon installing it manually it worked fine. I'm wondering why is not installed by the playbook?

vitabaks commented 1 month ago

Hello @rcknr

cdiff is optional and ydiff is used by default.

See requirements.txt - https://github.com/vitabaks/postgresql_cluster/blob/master/files/requirements.txt

vitabaks commented 1 month ago

If you are experiencing problems with patronictl, then simply update Patroni to version 3.3.0

rcknr commented 1 month ago

Sure, but without cdiff patronictl wasn't executing complaining about missing dependency.

vitabaks commented 1 month ago

All older Partoni versions are not compatible with ydiff>=1.3.

I don't see cdiff in the required dependencies. I think the update will solve the problem, please try.

rcknr commented 1 month ago

Okay, I see that you have updated Patroni version to 3.3.0 recently. I also see that this issue is addressed in Patroni's release notes.