[Pkg-openldap-devel] Bug#614569: RFS: Bug#614569: slapd fails to dump/reload partial replica -- NMU sponsor needed

[Ferenc Wagner]

Ferenc Wagner <wferi at niif.hu> writes:

> I pushed my proposed change at a separate branch:
> http://anonscm.debian.org/cgit/users/wferi-guest/openldap.git/log/?h=bug/614569
> Reproducing it here:
>
> commit 8a769204be90c1eaf7d5070c4e8a2d1eb18a82dc
> Author: Ferenc Wágner <wferi at niif.hu>
> Date:   Wed Dec 3 15:51:58 2014 +0100
>
>     Disable schema checking for reloading the dumped data during upgrades
>     
>     This avoids interrupting the upgrade procedure of partial replicas (#614569).
>
> diff --git a/debian/changelog b/debian/changelog
> index 1fcc7f3..012df5d 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -1,3 +1,12 @@
> +openldap (2.4.40-3.1) unstable; urgency=medium
> +
> +  * Non-maintainer upload.
> +  * Disable schema checking for reloading the dumped data during upgrades,
> +    to avoid interrupting the upgrade procedure of partial replicas.
> +    (Closes: #614569).
> +
> + -- Ferenc Wágner <wferi at niif.hu>  Wed, 03 Dec 2014 15:35:44 +0100
> +
>  openldap (2.4.40-3) unstable; urgency=medium
>  
>    * Remove trailing spaces from slapd.templates.
> diff --git a/debian/slapd.scripts-common b/debian/slapd.scripts-common
> index f56b3b1..8cf7f7b 100644
> --- a/debian/slapd.scripts-common
> +++ b/debian/slapd.scripts-common
> @@ -220,8 +220,11 @@ load_databases() {                                                 # {{{
>                 else
>                         slapadd_opts="-g -F ${SLAPD_CONF}"
>                 fi
> +               # Disable schema checking for the reload of the dumped data.
> +               # Otherwise, reloading partial replicas fails, breaking the
> +               # upgrade process.
>                 capture_diagnostics slapadd ${slapadd_opts} \
> -                       -q -b "$suffix" -l "$file" || failed=1
> +                       -q -b "$suffix" -l "$file" -s || failed=1
>                 if [ "$failed" ]; then
>                         rm -f "$dbdir"/*
>                         echo "failed." >&2
>
> Would anybody please sponsor an NMU with the above change added to
> 2.4.40-3, if I happen to get the change pre-approved by the release
> team?

I got pre-approval on #771962: the upload will be unblocked, provided
it's in unstable by Monday the 8th of December.  People with upload
rights, if you can spare a minute please review the above change and
consider sponsoring the upload!  Actually, review is welcome from
anybody. :)

The package is available at
http://mentors.debian.net/package/openldap

The dsc file can be downloaded from
http://mentors.debian.net/debian/pool/main/o/openldap/openldap_2.4.40-3.1.dsc

Ryan, please note that my upload somehow ended up under your name on
mentors.debian.net.  Thus I can't flip the "Needs a sponsor" flag.

Also, QA information says "Bug #614569 does not belong to this package",
and I only wonder why...
-- 
Thanks,
Feri.