public inbox for [email protected]  

help / color / mirror / Atom feed

From: Michael Banck <[email protected]>
To: Kirill <[email protected]>
Cc: [email protected]
Subject: Re: patroni debian bullseye package file inconsistency
Date: Fri, 29 Apr 2022 18:05:20 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>

Hi,

(I'm not on pgsql-pkg-debian, somebody forwarded me your mail)

On Wed, Apr 27, 2022 at 02:37:34PM +0300, Kirill wrote:
> I am using that package:
>  
> $ apt-cache policy patroni
> patroni:
>   Installed: 2.1.3-2.pgdg110+1
>   Candidate: 2.1.3-2.pgdg110+1
>   Version table:
>  *** 2.1.3-2.pgdg110+1 500
>         500 http://apt.postgresql.org/pub/repos/apt bullseye-pgdg/main amd64 Packages
>         100 /var/lib/dpkg/status
>  
> During the usage of patroni I got an error:
>  
> $ patroni --validate-config /etc/patroni-server/config.yaml

[...]

> Traceback (most recent call last):
[...]
>   File "/usr/lib/python3/dist-packages/patroni/validator.py", line 244, in iter_dict
>     validator = self.validator[key]._schema[d]
> KeyError: 'etcd3'

Apparently, running --validate-config on a config using etc3 has been
broken for quite a long time, and nobody noticed.
  
> I started to investigate and found that actual file:
>  
> $ dpkg -S  /usr/lib/python3/dist-packages/patroni/validator.py
> patroni: /usr/lib/python3/dist-packages/patroni/validator.py
>  
> is different from source file:

This is because there is a Debian patch that reverts an upstream change
that adds some RAFT tests which (as Debian does not ship RAFT yet) when
run would make the testsuite fail:
https://salsa.debian.org/postgresql/patroni/-/blob/debian/2.1.3-2/debian/patches/regression_tests_di...

However, that upstream change we reverted also added etcd3 validation,
which is what is now missing for you.

I've reworked the patch[1] so that only the RAFT-specifc changes remain,
and in my testing the above traceback is gone.

I will upload it once CI passes.


Michael

[1] https://salsa.debian.org/postgresql/patroni/-/commit/538cc5f0f9d6bd1704513f7280a0fd35cd193edf

view thread (2+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: patroni debian bullseye package file inconsistency
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox