Skip to content

Commit 56a4a05

Browse files
WarheadsSEstanhu
andcommitted
Merge branch 'sh-fix-patroni' into 'master'
Fix patroni no longer working with update to ydiff 1.3 See merge request https://gitlab.com/gitlab-org/omnibus-gitlab/-/merge_requests/7535 Merged-by: Jason Plum <[email protected]> Approved-by: Jason Plum <[email protected]> Approved-by: João Alexandre Cunha <[email protected]> Co-authored-by: Stan Hu <[email protected]>
2 parents 42e34c1 + a17b4c8 commit 56a4a05

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

config/software/patroni.rb

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,5 +35,13 @@
3535
# https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/5701
3636
command "#{install_dir}/embedded/bin/pip3 install prettytable==0.7.2", env: env
3737

38+
# Pin ydiff to 1.2 since 1.3 requires cdiff:
39+
# https://github.com/zalando/patroni/blob/634b44ee0586f063033074806a42b534a354cbff/docs/releases.rst
40+
# According to Patroni 3.3.0 release notes, upgrading to this version
41+
# should be enough to fix this problem. We should test unpinning
42+
# this library once we update to Patroni 3.3.0.
43+
# See: https://github.com/zalando/patroni/blob/634b44ee0586f063033074806a42b534a354cbff/docs/releases.rst
44+
command "#{install_dir}/embedded/bin/pip3 install ydiff==1.2", env: env
45+
3846
command "#{install_dir}/embedded/bin/pip3 install patroni[consul]==#{version}", env: env
3947
end

0 commit comments

Comments
 (0)